Merge pull request #539 from ueokande/webext-on-install-event

Use browser.runtime.onInstalled event

1 files changed, 7 insertions, 22 deletions

diff --git a/src/background/usecases/VersionUseCase.js b/src/background/usecases/VersionUseCase.js
index 704c808..ed5112b 100644
--- a/src/background/usecases/VersionUseCase.js
+++ b/src/background/usecases/VersionUseCase.js
@@ -1,40 +1,25 @@
 import manifest from '../../../manifest.json';
-import VersionRepository from '../repositories/VersionRepository';
 import TabPresenter from '../presenters/TabPresenter';
-import Notifier from '../infrastructures/Notifier';
+import NotifyPresenter from '../presenters/NotifyPresenter';
 
 export default class VersionUseCase {
   constructor() {
-    this.versionRepository = new VersionRepository();
     this.tabPresenter = new TabPresenter();
-    this.notifier = new Notifier();
+    this.notifyPresenter = new NotifyPresenter();
   }
 
-  async notifyIfUpdated() {
-    if (!await this.checkUpdated()) {
-      return;
-    }
-
-    let title = 'Vim Vixen ' + manifest.version + ' has been installed';
+  notify() {
+    let title = `Vim Vixen ${manifest.version} has been installed`;
     let message = 'Click here to see release notes';
-    this.notifier.notify(title, message, () => {
-      let url = this.releaseNoteUrl(manifest.version);
+    let url = this.releaseNoteUrl(manifest.version);
+    this.notifyPresenter.notify(title, message, () => {
       this.tabPresenter.create(url);
     });
-    this.versionRepository.update(manifest.version);
-  }
-
-  async checkUpdated() {
-    let prev = await this.versionRepository.get();
-    if (!prev) {
-      return true;
-    }
-    return manifest.version !== prev;
   }
 
   releaseNoteUrl(version) {
     if (version) {
-      return 'https://github.com/ueokande/vim-vixen/releases/tag/' + version;
+      return `https://github.com/ueokande/vim-vixen/releases/tag/${version}`;
     }
     return 'https://github.com/ueokande/vim-vixen/releases/';
   }