Merge pull request #544 from ueokande/refactor-background

Refactor background

5 files changed, 47 insertions, 7 deletions

diff --git a/src/background/infrastructures/ConsoleClient.js b/src/background/infrastructures/ConsoleClient.js
new file mode 100644
index 0000000..f691515
--- /dev/null
+++ b/src/background/infrastructures/ConsoleClient.js
@@ -0,0 +1,37 @@
+import messages from '../../shared/messages';
+
+export default class ConsoleClient {
+  showCommand(tabId, command) {
+    return browser.tabs.sendMessage(tabId, {
+      type: messages.CONSOLE_SHOW_COMMAND,
+      command,
+    });
+  }
+
+  showFind(tabId) {
+    return browser.tabs.sendMessage(tabId, {
+      type: messages.CONSOLE_SHOW_FIND
+    });
+  }
+
+  showInfo(tabId, message) {
+    return browser.tabs.sendMessage(tabId, {
+      type: messages.CONSOLE_SHOW_INFO,
+      text: message,
+    });
+  }
+
+  showError(tabId, message) {
+    return browser.tabs.sendMessage(tabId, {
+      type: messages.CONSOLE_SHOW_ERROR,
+      text: message,
+    });
+  }
+
+  hide(tabId) {
+    return browser.tabs.sendMessage(tabId, {
+      type: messages.CONSOLE_HIDE,
+    });
+  }
+}
+
diff --git a/src/background/infrastructures/content-message-client.js b/src/background/infrastructures/ContentMessageClient.js
index 7e7e602..0fab5a3 100644
--- a/src/background/infrastructures/content-message-client.js
+++ b/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,
       });
diff --git a/src/background/infrastructures/content-message-listener.js b/src/background/infrastructures/ContentMessageListener.js
index aae07c0..1179a8c 100644
--- a/src/background/infrastructures/content-message-listener.js
+++ b/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() {
diff --git a/src/background/infrastructures/memory-storage.js b/src/background/infrastructures/MemoryStorage.js
index 3a7e4f2..3a7e4f2 100644
--- a/src/background/infrastructures/memory-storage.js
+++ b/src/background/infrastructures/MemoryStorage.js
diff --git a/src/background/infrastructures/notifier.js b/src/background/infrastructures/Notifier.js
index 1eccc47..1eccc47 100644
--- a/src/background/infrastructures/notifier.js
+++ b/src/background/infrastructures/Notifier.js