Merge pull request #544 from ueokande/refactor-background

Refactor background

.eslintrc

@@ -40,6 +40,7 @@
     "no-alert": "off",
     "no-bitwise": "off",
     "no-console": ["error", { "allow": ["warn", "error"] }],
+    "no-continue": "off",
     "no-empty-function": "off",
     "no-magic-numbers": "off",
     "no-mixed-operators": "off",

src/background/controllers/AddonEnabledController.js

@@ -0,0 +1,11 @@
+import AddonEnabledUseCase from '../usecases/AddonEnabledUseCase';
+
+export default class AddonEnabledController {
+  constructor() {
+    this.addonEnabledUseCase = new AddonEnabledUseCase();
+  }
+
+  indicate(enabled) {
+    return this.addonEnabledUseCase.indicate(enabled);
+  }
+}

src/background/controllers/CommandController.js

@@ -1,6 +1,6 @@
-import CompletionsInteractor from '../usecases/completions';
-import CommandInteractor from '../usecases/command';
-import Completions from '../domains/completions';
+import CompletionsUseCase from '../usecases/CompletionsUseCase';
+import CommandUseCase from '../usecases/CommandUseCase';
+import Completions from '../domains/Completions';
 
 const trimStart = (str) => {
   // NOTE String.trimStart is available on Firefox 61
@@ -9,8 +9,8 @@ const trimStart = (str) => {
 
 export default class CommandController {
   constructor() {
-    this.completionsInteractor = new CompletionsInteractor();
-    this.commandIndicator = new CommandInteractor();
+    this.completionsUseCase = new CompletionsUseCase();
+    this.commandIndicator = new CommandUseCase();
   }
 
   getCompletions(line) {
@@ -18,7 +18,7 @@ export default class CommandController {
     let words = trimmed.split(/ +/);
     let name = words[0];
     if (words.length === 1) {
-      return this.completionsInteractor.queryConsoleCommand(name);
+      return this.completionsUseCase.queryConsoleCommand(name);
     }
     let keywords = trimStart(trimmed.slice(name.length));
     switch (words[0]) {
@@ -28,22 +28,22 @@ export default class CommandController {
     case 'tabopen':
     case 'w':
     case 'winopen':
-      return this.completionsInteractor.queryOpen(name, keywords);
+      return this.completionsUseCase.queryOpen(name, keywords);
     case 'b':
     case 'buffer':
-      return this.completionsInteractor.queryBuffer(name, keywords);
+      return this.completionsUseCase.queryBuffer(name, keywords);
     case 'bd':
     case 'bdel':
     case 'bdelete':
     case 'bdeletes':
-      return this.completionsInteractor.queryBdelete(name, keywords);
+      return this.completionsUseCase.queryBdelete(name, keywords);
     case 'bd!':
     case 'bdel!':
     case 'bdelete!':
     case 'bdeletes!':
-      return this.completionsInteractor.queryBdeleteForce(name, keywords);
+      return this.completionsUseCase.queryBdeleteForce(name, keywords);
     case 'set':
-      return this.completionsInteractor.querySet(name, keywords);
+      return this.completionsUseCase.querySet(name, keywords);
     }
     return Promise.resolve(Completions.empty());
   }

src/background/controllers/FindController.js

@@ -0,0 +1,15 @@
+import FindUseCase from '../usecases/FindUseCase';
+
+export default class FindController {
+  constructor() {
+    this.findUseCase = new FindUseCase();
+  }
+
+  getKeyword() {
+    return this.findUseCase.getKeyword();
+  }
+
+  setKeyword(keyword) {
+    return this.findUseCase.setKeyword(keyword);
+  }
+}

src/background/controllers/LinkController.js

@@ -0,0 +1,15 @@
+import LinkUseCase from '../usecases/LinkUseCase';
+
+export default class LinkController {
+  constructor() {
+    this.linkUseCase = new LinkUseCase();
+  }
+
+  openToTab(url, tabId) {
+    this.linkUseCase.openToTab(url, tabId);
+  }
+
+  openNewTab(url, openerId, background) {
+    this.linkUseCase.openNewTab(url, openerId, background);
+  }
+}

src/background/controllers/MarkController.js

@@ -0,0 +1,15 @@
+import MarkUseCase from '../usecases/MarkUseCase';
+
+export default class MarkController {
+  constructor() {
+    this.markUseCase = new MarkUseCase();
+  }
+
+  setGlobal(key, x, y) {
+    this.markUseCase.setGlobal(key, x, y);
+  }
+
+  jumpGlobal(key) {
+    this.markUseCase.jumpGlobal(key);
+  }
+}

src/background/controllers/OperationController.js

@@ -0,0 +1,77 @@
+import operations from '../../shared/operations';
+import FindUseCase from '../usecases/FindUseCase';
+import ConsoleUseCase from '../usecases/ConsoleUseCase';
+import TabUseCase from '../usecases/TabUseCase';
+import TabSelectUseCase from '../usecases/TabSelectUseCase';
+import ZoomUseCase from '../usecases/ZoomUseCase';
+
+export default class OperationController {
+  constructor() {
+    this.findUseCase = new FindUseCase();
+    this.consoleUseCase = new ConsoleUseCase();
+    this.tabUseCase = new TabUseCase();
+    this.tabSelectUseCase = new TabSelectUseCase();
+    this.zoomUseCase = new ZoomUseCase();
+  }
+
+  // eslint-disable-next-line complexity, max-lines-per-function
+  exec(operation) {
+    switch (operation.type) {
+    case operations.TAB_CLOSE:
+      return this.tabUseCase.close(false);
+    case operations.TAB_CLOSE_RIGHT:
+      return this.tabUseCase.closeRight();
+    case operations.TAB_CLOSE_FORCE:
+      return this.tabUseCase.close(true);
+    case operations.TAB_REOPEN:
+      return this.tabUseCase.reopen();
+    case operations.TAB_PREV:
+      return this.tabSelectUseCase.selectPrev(1);
+    case operations.TAB_NEXT:
+      return this.tabSelectUseCase.selectNext(1);
+    case operations.TAB_FIRST:
+      return this.tabSelectUseCase.selectFirst();
+    case operations.TAB_LAST:
+      return this.tabSelectUseCase.selectLast();
+    case operations.TAB_PREV_SEL:
+      return this.tabSelectUseCase.selectPrevSelected();
+    case operations.TAB_RELOAD:
+      return this.tabUseCase.reload(operation.cache);
+    case operations.TAB_PIN:
+      return this.tabUseCase.setPinned(true);
+    case operations.TAB_UNPIN:
+      return this.tabUseCase.setPinned(false);
+    case operations.TAB_TOGGLE_PINNED:
+      return this.tabUseCase.togglePinned();
+    case operations.TAB_DUPLICATE:
+      return this.tabUseCase.duplicate();
+    case operations.PAGE_SOURCE:
+      return this.tabUseCase.openPageSource();
+    case operations.PAGE_HOME:
+      return this.tabUseCase.openHome(operation.newTab);
+    case operations.ZOOM_IN:
+      return this.zoomUseCase.zoomIn();
+    case operations.ZOOM_OUT:
+      return this.zoomUseCase.zoomOut();
+    case operations.ZOOM_NEUTRAL:
+      return this.zoomUseCase.zoomNutoral();
+    case operations.COMMAND_SHOW:
+      return this.consoleUseCase.showCommand();
+    case operations.COMMAND_SHOW_OPEN:
+      return this.consoleUseCase.showOpenCommand(operation.alter);
+    case operations.COMMAND_SHOW_TABOPEN:
+      return this.consoleUseCase.showTabopenCommand(operation.alter);
+    case operations.COMMAND_SHOW_WINOPEN:
+      return this.consoleUseCase.showWinopenCommand(operation.alter);
+    case operations.COMMAND_SHOW_BUFFER:
+      return this.consoleUseCase.showBufferCommand();
+    case operations.COMMAND_SHOW_ADDBOOKMARK:
+      return this.consoleUseCase.showAddbookmarkCommand(operation.alter);
+    case operations.FIND_START:
+      return this.findUseCase.findStart();
+    case operations.CANCEL:
+      return this.consoleUseCase.hideConsole();
+    }
+  }
+}
+

src/background/controllers/SettingController.js

@@ -0,0 +1,18 @@
+import SettingUseCase from '../usecases/SettingUseCase';
+import ContentMessageClient from '../infrastructures/ContentMessageClient';
+
+export default class SettingController {
+  constructor() {
+    this.settingUseCase = new SettingUseCase();
+    this.contentMessageClient = new ContentMessageClient();
+  }
+
+  getSetting() {
+    return this.settingUseCase.get();
+  }
+
+  async reload() {
+    await this.settingUseCase.reload();
+    this.contentMessageClient.broadcastSettingsChanged();
+  }
+}

src/background/controllers/VersionController.js

@@ -0,0 +1,11 @@
+import VersionUseCase from '../usecases/VersionUseCase';
+
+export default class VersionController {
+  constructor() {
+    this.versionUseCase = new VersionUseCase();
+  }
+
+  notifyIfUpdated() {
+    this.versionUseCase.notifyIfUpdated();
+  }
+}

src/background/controllers/addon-enabled.js

@@ -1,11 +0,0 @@
-import AddonEnabledInteractor from '../usecases/addon-enabled';
-
-export default class AddonEnabledController {
-  constructor() {
-    this.addonEnabledInteractor = new AddonEnabledInteractor();
-  }
-
-  indicate(enabled) {
-    return this.addonEnabledInteractor.indicate(enabled);
-  }
-}

src/background/controllers/find.js

@@ -1,15 +0,0 @@
-import FindInteractor from '../usecases/find';
-
-export default class FindController {
-  constructor() {
-    this.findInteractor = new FindInteractor();
-  }
-
-  getKeyword() {
-    return this.findInteractor.getKeyword();
-  }
-
-  setKeyword(keyword) {
-    return this.findInteractor.setKeyword(keyword);
-  }
-}

src/background/controllers/link.js

@@ -1,15 +0,0 @@
-import LinkInteractor from '../usecases/link';
-
-export default class LinkController {
-  constructor() {
-    this.linkInteractor = new LinkInteractor();
-  }
-
-  openToTab(url, tabId) {
-    this.linkInteractor.openToTab(url, tabId);
-  }
-
-  openNewTab(url, openerId, background) {
-    this.linkInteractor.openNewTab(url, openerId, background);
-  }
-}

src/background/controllers/mark.js

@@ -1,15 +0,0 @@
-import MarkInteractor from '../usecases/mark';
-
-export default class MarkController {
-  constructor() {
-    this.markInteractor = new MarkInteractor();
-  }
-
-  setGlobal(key, x, y) {
-    this.markInteractor.setGlobal(key, x, y);
-  }
-
-  jumpGlobal(key) {
-    this.markInteractor.jumpGlobal(key);
-  }
-}

src/background/controllers/operation.js

@@ -1,69 +0,0 @@
-import operations from '../../shared/operations';
-import OperationInteractor from '../usecases/operation';
-
-export default class OperationController {
-  constructor() {
-    this.operationInteractor = new OperationInteractor();
-  }
-
-  // eslint-disable-next-line complexity, max-lines-per-function
-  exec(operation) {
-    switch (operation.type) {
-    case operations.TAB_CLOSE:
-      return this.operationInteractor.close(false);
-    case operations.TAB_CLOSE_RIGHT:
-      return this.operationInteractor.closeRight();
-    case operations.TAB_CLOSE_FORCE:
-      return this.operationInteractor.close(true);
-    case operations.TAB_REOPEN:
-      return this.operationInteractor.reopen();
-    case operations.TAB_PREV:
-      return this.operationInteractor.selectPrev(1);
-    case operations.TAB_NEXT:
-      return this.operationInteractor.selectNext(1);
-    case operations.TAB_FIRST:
-      return this.operationInteractor.selectFirst();
-    case operations.TAB_LAST:
-      return this.operationInteractor.selectLast();
-    case operations.TAB_PREV_SEL:
-      return this.operationInteractor.selectPrevSelected();
-    case operations.TAB_RELOAD:
-      return this.operationInteractor.reload(operation.cache);
-    case operations.TAB_PIN:
-      return this.operationInteractor.setPinned(true);
-    case operations.TAB_UNPIN:
-      return this.operationInteractor.setPinned(false);
-    case operations.TAB_TOGGLE_PINNED:
-      return this.operationInteractor.togglePinned();
-    case operations.TAB_DUPLICATE:
-      return this.operationInteractor.duplicate();
-    case operations.PAGE_SOURCE:
-      return this.operationInteractor.openPageSource();
-    case operations.PAGE_HOME:
-      return this.operationInteractor.openHome(operation.newTab);
-    case operations.ZOOM_IN:
-      return this.operationInteractor.zoomIn();
-    case operations.ZOOM_OUT:
-      return this.operationInteractor.zoomOut();
-    case operations.ZOOM_NEUTRAL:
-      return this.operationInteractor.zoomNutoral();
-    case operations.COMMAND_SHOW:
-      return this.operationInteractor.showCommand();
-    case operations.COMMAND_SHOW_OPEN:
-      return this.operationInteractor.showOpenCommand(operation.alter);
-    case operations.COMMAND_SHOW_TABOPEN:
-      return this.operationInteractor.showTabopenCommand(operation.alter);
-    case operations.COMMAND_SHOW_WINOPEN:
-      return this.operationInteractor.showWinopenCommand(operation.alter);
-    case operations.COMMAND_SHOW_BUFFER:
-      return this.operationInteractor.showBufferCommand();
-    case operations.COMMAND_SHOW_ADDBOOKMARK:
-      return this.operationInteractor.showAddbookmarkCommand(operation.alter);
-    case operations.FIND_START:
-      return this.operationInteractor.findStart();
-    case operations.CANCEL:
-      return this.operationInteractor.hideConsole();
-    }
-  }
-}
-

src/background/controllers/setting.js

@@ -1,18 +0,0 @@
-import SettingInteractor from '../usecases/setting';
-import ContentMessageClient from '../infrastructures/content-message-client';
-
-export default class SettingController {
-  constructor() {
-    this.settingInteractor = new SettingInteractor();
-    this.contentMessageClient = new ContentMessageClient();
-  }
-
-  getSetting() {
-    return this.settingInteractor.get();
-  }
-
-  async reload() {
-    await this.settingInteractor.reload();
-    this.contentMessageClient.broadcastSettingsChanged();
-  }
-}

src/background/controllers/version.js

@@ -1,11 +0,0 @@
-import VersionInteractor from '../usecases/version';
-
-export default class VersionController {
-  constructor() {
-    this.versionInteractor = new VersionInteractor();
-  }
-
-  notifyIfUpdated() {
-    this.versionInteractor.notifyIfUpdated();
-  }
-}

src/background/index.js

@@ -1,6 +1,6 @@
-import ContentMessageListener from './infrastructures/content-message-listener';
-import SettingController from './controllers/setting';
-import VersionController from './controllers/version';
+import ContentMessageListener from './infrastructures/ContentMessageListener';
+import SettingController from './controllers/SettingController';
+import VersionController from './controllers/VersionController';
 
 new SettingController().reload();
 new VersionController().notifyIfUpdated();

src/background/infrastructures/ConsoleClient.js

@@ -1,6 +1,6 @@
 import messages from '../../shared/messages';
 
-export default class ConsolePresenter {
+export default class ConsoleClient {
   showCommand(tabId, command) {
     return browser.tabs.sendMessage(tabId, {
       type: messages.CONSOLE_SHOW_COMMAND,
@@ -34,3 +34,4 @@ export default class ConsolePresenter {
     });
   }
 }
+

src/background/infrastructures/ContentMessageClient.js

@@ -4,6 +4,9 @@ export default class ContentMessageClient {
   async broadcastSettingsChanged() {
     let tabs = await browser.tabs.query({});
     for (let tab of tabs) {
+      if (tab.url.startsWith('about:')) {
+        continue;
+      }
       browser.tabs.sendMessage(tab.id, {
         type: messages.SETTINGS_CHANGED,
       });

src/background/infrastructures/ContentMessageListener.js

@@ -1,11 +1,11 @@
 import messages from '../../shared/messages';
-import CommandController from '../controllers/command';
-import SettingController from '../controllers/setting';
-import FindController from '../controllers/find';
-import AddonEnabledController from '../controllers/addon-enabled';
-import LinkController from '../controllers/link';
-import OperationController from '../controllers/operation';
-import MarkController from '../controllers/mark';
+import CommandController from '../controllers/CommandController';
+import SettingController from '../controllers/SettingController';
+import FindController from '../controllers/FindController';
+import AddonEnabledController from '../controllers/AddonEnabledController';
+import LinkController from '../controllers/LinkController';
+import OperationController from '../controllers/OperationController';
+import MarkController from '../controllers/MarkController';
 
 export default class ContentMessageListener {
   constructor() {

src/background/presenters/TabPresenter.js

@@ -1,4 +1,4 @@
-import MemoryStorage from '../infrastructures/memory-storage';
+import MemoryStorage from '../infrastructures/MemoryStorage';
 
 const CURRENT_SELECTED_KEY = 'tabs.current.selected';
 const LAST_SELECTED_KEY = 'tabs.last.selected';

src/background/repositories/BrowserSettingRepository.js

@@ -0,0 +1,8 @@
+import * as urls from '../../shared/urls';
+
+export default class BrowserSettingRepository {
+  async getHomepageUrls() {
+    let { value } = await browser.browserSettings.homepageOverride.get({});
+    return value.split('|').map(urls.normalizeUrl);
+  }
+}

src/background/repositories/FindRepository.js

@@ -1,4 +1,4 @@
-import MemoryStorage from '../infrastructures/memory-storage';
+import MemoryStorage from '../infrastructures/MemoryStorage';
 
 const FIND_KEYWORD_KEY = 'find-keyword';
 

src/background/repositories/MarkRepository.js

@@ -1,5 +1,5 @@
-import MemoryStorage from '../infrastructures/memory-storage';
-import GlobalMark from 'background/domains/global-mark';
+import MemoryStorage from '../infrastructures/MemoryStorage';
+import GlobalMark from '../domains/GlobalMark';
 
 const MARK_KEY = 'mark';
 

src/background/repositories/PersistentSettingRepository.js

@@ -1,10 +1,6 @@
-import Setting from '../domains/setting';
+import Setting from '../domains/Setting';
 
 export default class SettingRepository {
-  save(settings) {
-    return browser.storage.local.set({ settings: settings.serialize() });
-  }
-
   async load() {
     let { settings } = await browser.storage.local.get('settings');
     if (!settings) {

src/background/repositories/SettingRepository.js

@@ -1,4 +1,4 @@
-import MemoryStorage from '../infrastructures/memory-storage';
+import MemoryStorage from '../infrastructures/MemoryStorage';
 
 const CACHED_SETTING_KEY = 'setting';
 

src/background/usecases/AddonEnabledUseCase.js

@@ -1,8 +1,8 @@
-import IndicatorPresenter from '../presenters/indicator';
-import TabPresenter from '../presenters/tab';
-import ContentMessageClient from '../infrastructures/content-message-client';
+import IndicatorPresenter from '../presenters/IndicatorPresenter';
+import TabPresenter from '../presenters/TabPresenter';
+import ContentMessageClient from '../infrastructures/ContentMessageClient';
 
-export default class AddonEnabledInteractor {
+export default class AddonEnabledUseCase {
   constructor() {
     this.indicatorPresentor = new IndicatorPresenter();
 

src/background/usecases/CommandUseCase.js

@@ -1,11 +1,11 @@
 import * as parsers from './parsers';
 import * as urls from '../../shared/urls';
-import TabPresenter from '../presenters/tab';
-import WindowPresenter from '../presenters/window';
-import SettingRepository from '../repositories/setting';
-import BookmarkRepository from '../repositories/bookmark';
-import ConsolePresenter from '../presenters/console';
-import ContentMessageClient from '../infrastructures/content-message-client';
+import TabPresenter from '../presenters/TabPresenter';
+import WindowPresenter from '../presenters/WindowPresenter';
+import SettingRepository from '../repositories/SettingRepository';
+import BookmarkRepository from '../repositories/BookmarkRepository';
+import ConsoleClient from '../infrastructures/ConsoleClient';
+import ContentMessageClient from '../infrastructures/ContentMessageClient';
 import * as properties from 'shared/settings/properties';
 
 export default class CommandIndicator {
@@ -14,7 +14,7 @@ export default class CommandIndicator {
     this.windowPresenter = new WindowPresenter();
     this.settingRepository = new SettingRepository();
     this.bookmarkRepository = new BookmarkRepository();
-    this.consolePresenter = new ConsolePresenter();
+    this.consoleClient = new ConsoleClient();
 
     this.contentMessageClient = new ContentMessageClient();
   }
@@ -105,7 +105,7 @@ export default class CommandIndicator {
     let tab = await this.tabPresenter.getCurrent();
     let item = await this.bookmarkRepository.create(title, tab.url);
     let message = 'Saved current page: ' + item.url;
-    return this.consolePresenter.showInfo(tab.id, message);
+    return this.consoleClient.showInfo(tab.id, message);
   }
 
   async set(keywords) {

src/background/usecases/CompletionsUseCase.js

@@ -1,19 +1,19 @@
-import CompletionItem from '../domains/completion-item';
-import CompletionGroup from '../domains/completion-group';
-import Completions from '../domains/completions';
-import CommandDocs from '../domains/command-docs';
-import CompletionRepository from '../repositories/completions';
+import CompletionItem from '../domains/CompletionItem';
+import CompletionGroup from '../domains/CompletionGroup';
+import Completions from '../domains/Completions';
+import CommandDocs from '../domains/CommandDocs';
+import CompletionsRepository from '../repositories/CompletionsRepository';
 import * as filters from './filters';
-import SettingRepository from '../repositories/setting';
-import TabPresenter from '../presenters/tab';
+import SettingRepository from '../repositories/SettingRepository';
+import TabPresenter from '../presenters/TabPresenter';
 import * as properties from '../../shared/settings/properties';
 
 const COMPLETION_ITEM_LIMIT = 10;
 
-export default class CompletionsInteractor {
+export default class CompletionsUseCase {
   constructor() {
     this.tabPresenter = new TabPresenter();
-    this.completionRepository = new CompletionRepository();
+    this.completionsRepository = new CompletionsRepository();
     this.settingRepository = new SettingRepository();
   }
 
@@ -86,7 +86,7 @@ export default class CompletionsInteractor {
         tabs = [tab];
       }
     } else {
-      tabs = await this.completionRepository.queryTabs(keywords, false);
+      tabs = await this.completionsRepository.queryTabs(keywords, false);
     }
     const flag = (tab) => {
       if (tab.active) {
@@ -153,7 +153,7 @@ export default class CompletionsInteractor {
   }
 
   async queryTabs(name, excludePinned, args) {
-    let tabs = await this.completionRepository.queryTabs(args, excludePinned);
+    let tabs = await this.completionsRepository.queryTabs(args, excludePinned);
     let items = tabs.map(tab => new CompletionItem({
       caption: tab.title,
       content: name + ' ' + tab.title,
@@ -177,7 +177,7 @@ export default class CompletionsInteractor {
   }
 
   async queryHistoryItems(name, keywords) {
-    let histories = await this.completionRepository.queryHistories(keywords);
+    let histories = await this.completionsRepository.queryHistories(keywords);
     histories = [histories]
       .map(filters.filterBlankTitle)
       .map(filters.filterHttp)
@@ -194,7 +194,7 @@ export default class CompletionsInteractor {
   }
 
   async queryBookmarkItems(name, keywords) {
-    let bookmarks = await this.completionRepository.queryBookmarks(keywords);
+    let bookmarks = await this.completionsRepository.queryBookmarks(keywords);
     return bookmarks.slice(0, COMPLETION_ITEM_LIMIT)
       .map(page => new CompletionItem({
         caption: page.title,

src/background/usecases/ConsoleUseCase.js

@@ -0,0 +1,61 @@
+import TabPresenter from '../presenters/TabPresenter';
+import ConsoleClient from '../infrastructures/ConsoleClient';
+
+export default class ConsoleUseCase {
+  constructor() {
+    this.tabPresenter = new TabPresenter();
+    this.consoleClient = new ConsoleClient();
+  }
+
+  async showCommand() {
+    let tab = await this.tabPresenter.getCurrent();
+    return this.consoleClient.showCommand(tab.id, '');
+  }
+
+  async showOpenCommand(alter) {
+    let tab = await this.tabPresenter.getCurrent();
+    let command = 'open ';
+    if (alter) {
+      command += tab.url;
+    }
+    return this.consoleClient.showCommand(tab.id, command);
+  }
+
+  async showTabopenCommand(alter) {
+    let tab = await this.tabPresenter.getCurrent();
+    let command = 'tabopen ';
+    if (alter) {
+      command += tab.url;
+    }
+    return this.consoleClient.showCommand(tab.id, command);
+  }
+
+  async showWinopenCommand(alter) {
+    let tab = await this.tabPresenter.getCurrent();
+    let command = 'winopen ';
+    if (alter) {
+      command += tab.url;
+    }
+    return this.consoleClient.showCommand(tab.id, command);
+  }
+
+  async showBufferCommand() {
+    let tab = await this.tabPresenter.getCurrent();
+    let command = 'buffer ';
+    return this.consoleClient.showCommand(tab.id, command);
+  }
+
+  async showAddbookmarkCommand(alter) {
+    let tab = await this.tabPresenter.getCurrent();
+    let command = 'addbookmark ';
+    if (alter) {
+      command += tab.title;
+    }
+    return this.consoleClient.showCommand(tab.id, command);
+  }
+
+  async hideConsole() {
+    let tab = await this.tabPresenter.getCurrent();
+    return this.consoleClient.hide(tab.id);
+  }
+}

src/background/usecases/FindUseCase.js

@@ -0,0 +1,24 @@
+import FindRepository from '../repositories/FindRepository';
+import TabPresenter from '../presenters/TabPresenter';
+import ConsoleClient from '../infrastructures/ConsoleClient';
+
+export default class FindUseCase {
+  constructor() {
+    this.tabPresenter = new TabPresenter();
+    this.findRepository = new FindRepository();
+    this.consoleClient = new ConsoleClient();
+  }
+
+  getKeyword() {
+    return this.findRepository.getKeyword();
+  }
+
+  setKeyword(keyword) {
+    return this.findRepository.setKeyword(keyword);
+  }
+
+  async findStart() {
+    let tab = await this.tabPresenter.getCurrent();
+    return this.consoleClient.showFind(tab.id);
+  }
+}

src/background/usecases/LinkUseCase.js

@@ -1,7 +1,7 @@
-import SettingRepository from '../repositories/setting';
-import TabPresenter from '../presenters/tab';
+import SettingRepository from '../repositories/SettingRepository';
+import TabPresenter from '../presenters/TabPresenter';
 
-export default class LinkInteractor {
+export default class LinkUseCase {
   constructor() {
     this.settingRepository = new SettingRepository();
     this.tabPresenter = new TabPresenter();

src/background/usecases/MarkUseCase.js

@@ -1,14 +1,14 @@
-import GlobalMark from '../domains/global-mark';
-import TabPresenter from '../presenters/tab';
-import MarkRepository from '../repositories/mark';
-import ConsolePresenter from '../presenters/console';
-import ContentMessageClient from '../infrastructures/content-message-client';
+import GlobalMark from '../domains/GlobalMark';
+import TabPresenter from '../presenters/TabPresenter';
+import MarkRepository from '../repositories/MarkRepository';
+import ConsoleClient from '../infrastructures/ConsoleClient';
+import ContentMessageClient from '../infrastructures/ContentMessageClient';
 
-export default class MarkInteractor {
+export default class MarkUseCase {
   constructor() {
     this.tabPresenter = new TabPresenter();
     this.markRepository = new MarkRepository();
-    this.consolePresenter = new ConsolePresenter();
+    this.consoleClient = new ConsoleClient();
     this.contentMessageClient = new ContentMessageClient();
   }
 
@@ -23,7 +23,7 @@ export default class MarkInteractor {
 
     let mark = await this.markRepository.getMark(key);
     if (!mark) {
-      return this.consolePresenter.showError(current.id, 'Mark is not set');
+      return this.consoleClient.showError(current.id, 'Mark is not set');
     }
 
     return this.contentMessageClient.scrollTo(

src/background/usecases/SettingUseCase.js

@@ -1,17 +1,14 @@
-import Setting from '../domains/setting';
-import PersistentSettingRepository from '../repositories/persistent-setting';
-import SettingRepository from '../repositories/setting';
+import Setting from '../domains/Setting';
+// eslint-disable-next-line max-len
+import PersistentSettingRepository from '../repositories/PersistentSettingRepository';
+import SettingRepository from '../repositories/SettingRepository';
 
-export default class SettingInteractor {
+export default class SettingUseCase {
   constructor() {
     this.persistentSettingRepository = new PersistentSettingRepository();
     this.settingRepository = new SettingRepository();
   }
 
-  save(settings) {
-    this.persistentSettingRepository.save(settings);
-  }
-
   get() {
     return this.settingRepository.get();
   }

src/background/usecases/TabSelectUseCase.js

@@ -0,0 +1,51 @@
+import TabPresenter from '../presenters/TabPresenter';
+
+export default class TabSelectUseCase {
+  constructor() {
+    this.tabPresenter = new TabPresenter();
+  }
+
+  async selectPrev(count) {
+    let tabs = await this.tabPresenter.getAll();
+    if (tabs.length < 2) {
+      return;
+    }
+    let tab = tabs.find(t => t.active);
+    if (!tab) {
+      return;
+    }
+    let select = (tab.index - count + tabs.length) % tabs.length;
+    return this.tabPresenter.select(tabs[select].id);
+  }
+
+  async selectNext(count) {
+    let tabs = await this.tabPresenter.getAll();
+    if (tabs.length < 2) {
+      return;
+    }
+    let tab = tabs.find(t => t.active);
+    if (!tab) {
+      return;
+    }
+    let select = (tab.index + count) % tabs.length;
+    return this.tabPresenter.select(tabs[select].id);
+  }
+
+  async selectFirst() {
+    let tabs = await this.tabPresenter.getAll();
+    return this.tabPresenter.select(tabs[0].id);
+  }
+
+  async selectLast() {
+    let tabs = await this.tabPresenter.getAll();
+    return this.tabPresenter.select(tabs[tabs.length - 1].id);
+  }
+
+  async selectPrevSelected() {
+    let tabId = await this.tabPresenter.getLastSelectedId();
+    if (tabId === null || typeof tabId === 'undefined') {
+      return;
+    }
+    this.tabPresenter.select(tabId);
+  }
+}

src/background/usecases/TabUseCase.js

@@ -0,0 +1,77 @@
+import TabPresenter from '../presenters/TabPresenter';
+import BrowserSettingRepository from '../repositories/BrowserSettingRepository';
+
+export default class TabUseCase {
+  constructor() {
+    this.tabPresenter = new TabPresenter();
+    this.browserSettingRepository = new BrowserSettingRepository();
+  }
+
+  async close(force) {
+    let tab = await this.tabPresenter.getCurrent();
+    if (!force && tab.pinned) {
+      return;
+    }
+    return this.tabPresenter.remove([tab.id]);
+  }
+
+  async closeRight() {
+    let tabs = await this.tabPresenter.getAll();
+    tabs.sort((t1, t2) => t1.index - t2.index);
+    let index = tabs.findIndex(t => t.active);
+    if (index < 0) {
+      return;
+    }
+    for (let i = index + 1; i < tabs.length; ++i) {
+      let tab = tabs[i];
+      if (!tab.pinned) {
+        this.tabPresenter.remove(tab.id);
+      }
+    }
+  }
+
+  reopen() {
+    return this.tabPresenter.reopen();
+  }
+
+  async reload(cache) {
+    let tab = await this.tabPresenter.getCurrent();
+    return this.tabPresenter.reload(tab.id, cache);
+  }
+
+  async setPinned(pinned) {
+    let tab = await this.tabPresenter.getCurrent();
+    return this.tabPresenter.setPinned(tab.id, pinned);
+  }
+
+  async togglePinned() {
+    let tab = await this.tabPresenter.getCurrent();
+    return this.tabPresenter.setPinned(tab.id, !tab.pinned);
+  }
+
+  async duplicate() {
+    let tab = await this.tabPresenter.getCurrent();
+    return this.tabPresenter.duplicate(tab.id);
+  }
+
+  async openPageSource() {
+    let tab = await this.tabPresenter.getCurrent();
+    let url = 'view-source:' + tab.url;
+    return this.tabPresenter.create(url);
+  }
+
+  async openHome(newTab) {
+    let tab = await this.tabPresenter.getCurrent();
+    let urls = await this.browserSettingRepository.getHomepageUrls();
+    if (urls.length === 1 && urls[0] === 'about:home') {
+      // eslint-disable-next-line max-len
+      throw new Error('Cannot open Firefox Home (about:home) by WebExtensions, set your custom URLs');
+    }
+    if (urls.length === 1 && !newTab) {
+      return this.tabPresenter.open(urls[0], tab.id);
+    }
+    for (let url of urls) {
+      this.tabPresenter.create(url);
+    }
+  }
+}

src/background/usecases/VersionUseCase.js

@@ -1,9 +1,9 @@
 import manifest from '../../../manifest.json';
-import VersionRepository from '../repositories/version';
-import TabPresenter from '../presenters/tab';
-import Notifier from '../infrastructures/notifier';
+import VersionRepository from '../repositories/VersionRepository';
+import TabPresenter from '../presenters/TabPresenter';
+import Notifier from '../infrastructures/Notifier';
 
-export default class VersionInteractor {
+export default class VersionUseCase {
   constructor() {
     this.versionRepository = new VersionRepository();
     this.tabPresenter = new TabPresenter();

src/background/usecases/ZoomUseCase.js

@@ -0,0 +1,35 @@
+import TabPresenter from '../presenters/TabPresenter';
+
+const ZOOM_SETTINGS = [
+  0.33, 0.50, 0.66, 0.75, 0.80, 0.90, 1.00,
+  1.10, 1.25, 1.50, 1.75, 2.00, 2.50, 3.00
+];
+
+export default class ZoomUseCase {
+  constructor() {
+    this.tabPresenter = new TabPresenter();
+  }
+
+  async zoomIn(tabId) {
+    let tab = await this.tabPresenter.getCurrent();
+    let current = await this.tabPresenter.getZoom(tab.id);
+    let factor = ZOOM_SETTINGS.find(f => f > current);
+    if (factor) {
+      return this.tabPresenter.setZoom(tabId, factor);
+    }
+  }
+
+  async zoomOut(tabId) {
+    let tab = await this.tabPresenter.getCurrent();
+    let current = await this.tabPresenter.getZoom(tab.id);
+    let factor = [].concat(ZOOM_SETTINGS).reverse().find(f => f < current);
+    if (factor) {
+      return this.tabPresenter.setZoom(tabId, factor);
+    }
+  }
+
+  zoomNutoral(tabId) {
+    return this.tabPresenter.setZoom(tabId, 1);
+  }
+
+}

src/background/usecases/find.js

@@ -1,15 +0,0 @@
-import FindRepository from '../repositories/find';
-
-export default class FindInteractor {
-  constructor() {
-    this.findRepository = new FindRepository();
-  }
-
-  getKeyword() {
-    return this.findRepository.getKeyword();
-  }
-
-  setKeyword(keyword) {
-    return this.findRepository.setKeyword(keyword);
-  }
-}

src/background/usecases/operation.js

@@ -1,208 +0,0 @@
-import TabPresenter from '../presenters/tab';
-import ConsolePresenter from '../presenters/console';
-import * as urls from '../../shared/urls';
-
-const ZOOM_SETTINGS = [
-  0.33, 0.50, 0.66, 0.75, 0.80, 0.90, 1.00,
-  1.10, 1.25, 1.50, 1.75, 2.00, 2.50, 3.00
-];
-
-export default class OperationInteractor {
-  constructor() {
-    this.tabPresenter = new TabPresenter();
-    this.consolePresenter = new ConsolePresenter();
-  }
-
-  async close(force) {
-    let tab = await this.tabPresenter.getCurrent();
-    if (!force && tab.pinned) {
-      return;
-    }
-    return this.tabPresenter.remove([tab.id]);
-  }
-
-  async closeRight() {
-    let tabs = await this.tabPresenter.getAll();
-    tabs.sort((t1, t2) => t1.index - t2.index);
-    let index = tabs.findIndex(t => t.active);
-    if (index < 0) {
-      return;
-    }
-    for (let i = index + 1; i < tabs.length; ++i) {
-      let tab = tabs[i];
-      if (!tab.pinned) {
-        this.tabPresenter.remove(tab.id);
-      }
-    }
-  }
-
-  reopen() {
-    return this.tabPresenter.reopen();
-  }
-
-  async selectPrev(count) {
-    let tabs = await this.tabPresenter.getAll();
-    if (tabs.length < 2) {
-      return;
-    }
-    let tab = tabs.find(t => t.active);
-    if (!tab) {
-      return;
-    }
-    let select = (tab.index - count + tabs.length) % tabs.length;
-    return this.tabPresenter.select(tabs[select].id);
-  }
-
-  async selectNext(count) {
-    let tabs = await this.tabPresenter.getAll();
-    if (tabs.length < 2) {
-      return;
-    }
-    let tab = tabs.find(t => t.active);
-    if (!tab) {
-      return;
-    }
-    let select = (tab.index + count) % tabs.length;
-    return this.tabPresenter.select(tabs[select].id);
-  }
-
-  async selectFirst() {
-    let tabs = await this.tabPresenter.getAll();
-    return this.tabPresenter.select(tabs[0].id);
-  }
-
-  async selectLast() {
-    let tabs = await this.tabPresenter.getAll();
-    return this.tabPresenter.select(tabs[tabs.length - 1].id);
-  }
-
-  async selectPrevSelected() {
-    let tabId = await this.tabPresenter.getLastSelectedId();
-    if (tabId === null || typeof tabId === 'undefined') {
-      return;
-    }
-    this.tabPresenter.select(tabId);
-  }
-
-  async reload(cache) {
-    let tab = await this.tabPresenter.getCurrent();
-    return this.tabPresenter.reload(tab.id, cache);
-  }
-
-  async setPinned(pinned) {
-    let tab = await this.tabPresenter.getCurrent();
-    return this.tabPresenter.setPinned(tab.id, pinned);
-  }
-
-  async togglePinned() {
-    let tab = await this.tabPresenter.getCurrent();
-    return this.tabPresenter.setPinned(tab.id, !tab.pinned);
-  }
-
-  async duplicate() {
-    let tab = await this.tabPresenter.getCurrent();
-    return this.tabPresenter.duplicate(tab.id);
-  }
-
-  async openPageSource() {
-    let tab = await this.tabPresenter.getCurrent();
-    let url = 'view-source:' + tab.url;
-    return this.tabPresenter.create(url);
-  }
-
-  async zoomIn(tabId) {
-    let tab = await this.tabPresenter.getCurrent();
-    let current = await this.tabPresenter.getZoom(tab.id);
-    let factor = ZOOM_SETTINGS.find(f => f > current);
-    if (factor) {
-      return this.tabPresenter.setZoom(tabId, factor);
-    }
-  }
-
-  async zoomOut(tabId) {
-    let tab = await this.tabPresenter.getCurrent();
-    let current = await this.tabPresenter.getZoom(tab.id);
-    let factor = [].concat(ZOOM_SETTINGS).reverse().find(f => f < current);
-    if (factor) {
-      return this.tabPresenter.setZoom(tabId, factor);
-    }
-  }
-
-  zoomNutoral(tabId) {
-    return this.tabPresenter.setZoom(tabId, 1);
-  }
-
-  async showCommand() {
-    let tab = await this.tabPresenter.getCurrent();
-    return this.consolePresenter.showCommand(tab.id, '');
-  }
-
-  async showOpenCommand(alter) {
-    let tab = await this.tabPresenter.getCurrent();
-    let command = 'open ';
-    if (alter) {
-      command += tab.url;
-    }
-    return this.consolePresenter.showCommand(tab.id, command);
-  }
-
-  async showTabopenCommand(alter) {
-    let tab = await this.tabPresenter.getCurrent();
-    let command = 'tabopen ';
-    if (alter) {
-      command += tab.url;
-    }
-    return this.consolePresenter.showCommand(tab.id, command);
-  }
-
-  async showWinopenCommand(alter) {
-    let tab = await this.tabPresenter.getCurrent();
-    let command = 'winopen ';
-    if (alter) {
-      command += tab.url;
-    }
-    return this.consolePresenter.showCommand(tab.id, command);
-  }
-
-  async showBufferCommand() {
-    let tab = await this.tabPresenter.getCurrent();
-    let command = 'buffer ';
-    return this.consolePresenter.showCommand(tab.id, command);
-  }
-
-  async showAddbookmarkCommand(alter) {
-    let tab = await this.tabPresenter.getCurrent();
-    let command = 'addbookmark ';
-    if (alter) {
-      command += tab.title;
-    }
-    return this.consolePresenter.showCommand(tab.id, command);
-  }
-
-  async findStart() {
-    let tab = await this.tabPresenter.getCurrent();
-    return this.consolePresenter.showFind(tab.id);
-  }
-
-  async hideConsole() {
-    let tab = await this.tabPresenter.getCurrent();
-    return this.consolePresenter.hide(tab.id);
-  }
-
-  async openHome(newTab) {
-    let tab = await this.tabPresenter.getCurrent();
-    let result = await browser.browserSettings.homepageOverride.get({});
-    let us = urls.homepageUrls(result.value);
-    if (us.length === 1 && us[0] === 'about:home') {
-      // eslint-disable-next-line max-len
-      throw new Error('Cannot open Firefox Home (about:home) by WebExtensions, set your custom URLs');
-    }
-    if (us.length === 1 && !newTab) {
-      return this.tabPresenter.open(us[0], tab.id);
-    }
-    for (let u of us) {
-      this.tabPresenter.create(u);
-    }
-  }
-}
-

src/shared/urls.js

@@ -40,8 +40,4 @@ const normalizeUrl = (url) => {
   return 'http://' + url;
 };
 
-const homepageUrls = (value) => {
-  return value.split('|').map(normalizeUrl);
-};
-
-export { searchUrl, normalizeUrl, homepageUrls };
+export { searchUrl, normalizeUrl };

test/background/domains/GlobalMark.test.js

@@ -1,4 +1,4 @@
-import GlobalMark from 'background/domains/global-mark';
+import GlobalMark from 'background/domains/GlobalMark';
 
 describe('background/domains/global-mark', () => {
   describe('constructor and getter', () => {

test/background/infrastructures/MemoryStorage.test.js

@@ -1,4 +1,4 @@
-import MemoryStorage from 'background/infrastructures/memory-storage';
+import MemoryStorage from 'background/infrastructures/MemoryStorage';
 
 describe("background/infrastructures/memory-storage", () => {
   it('stores values', () => {

test/background/repositories/Mark.test.js

@@ -1,5 +1,5 @@
-import MarkRepository from 'background/repositories/mark';
-import GlobalMark from 'background/domains/global-mark';
+import MarkRepository from 'background/repositories/MarkRepository';
+import GlobalMark from 'background/domains/GlobalMark';
 
 describe('background/repositories/mark', () => {
   let repository;

test/background/repositories/Version.js

@@ -1,4 +1,4 @@
-import VersionRepository from 'background/repositories/version';
+import VersionRepository from 'background/repositories/Version';
 
 describe("background/repositories/version", () => {
   let versionRepository;

test/shared/urls.test.js

@@ -44,14 +44,5 @@ describe("shared/commands/parsers", () => {
         .to.equal('http://google.com');
     });
   });
-
-  describe('#homepageUrls', () => {
-    it('split urls', () => {
-      expect(parsers.homepageUrls('https://google.com/'))
-        .to.deep.equal(['https://google.com/']);
-      expect(parsers.homepageUrls('yahoo.com|https://i-beam.org/'))
-        .to.deep.equal(['http://yahoo.com', 'https://i-beam.org/']);
-    });
-  });
 });