this addresses #184

Author: gorhill

background.html

@@ -29,6 +29,7 @@
 <script src="js/useragent.js"></script>
 <script src="js/contentscripthandlers.js"></script>
 <script src="js/start.js"></script>
+<script src="js/commands.js"></script>
 <script src="js/install.js"></script>
 </body>
 </html>

js/commands.js

@@ -0,0 +1,60 @@
+/*******************************************************************************
+
+    httpswitchboard - a Chromium browser extension to black/white list requests.
+    Copyright (C) 2014  Contributors to HTTP Switchboard
+
+    This program is free software: you can redistribute it and/or modify
+    it under the terms of the GNU General Public License as published by
+    the Free Software Foundation, either version 3 of the License, or
+    (at your option) any later version.
+
+    This program is distributed in the hope that it will be useful,
+    but WITHOUT ANY WARRANTY; without even the implied warranty of
+    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+    GNU General Public License for more details.
+
+    You should have received a copy of the GNU General Public License
+    along with this program.  If not, see {http://www.gnu.org/licenses/}.
+
+    Home: https://github.com/gorhill/httpswitchboard
+*/
+
+/******************************************************************************/
+
+(function() {
+
+/******************************************************************************/
+
+var gotoDashboard = function(which) {
+    chrome.runtime.sendMessage({
+        what: 'gotoExtensionURL',
+        url: 'dashboard.html#' + which
+    });
+};
+
+/******************************************************************************/
+
+var onCommand = function(command) {
+    switch ( command ) {
+    case 'open-statistics':
+        gotoDashboard('statistics');
+        break;
+    case 'open-ubiquitous-rules':
+        gotoDashboard('ubiquitous-rules');
+        break;
+    case 'open-scoped-rules':
+        gotoDashboard('scoped-rules');
+        break;
+    case 'open-settings':
+        gotoDashboard('settings');
+        break;
+    }
+};
+
+/******************************************************************************/
+
+chrome.commands.onCommand.addListener(onCommand);
+
+/******************************************************************************/
+
+})();

js/start.js

@@ -173,17 +173,3 @@ function clearBrowserCacheCallback() {
 HTTPSB.asyncJobs.add('clearBrowserCache', null, clearBrowserCacheCallback, 15 * 60 * 1000, true);
 
 /******************************************************************************/
-
-chrome.commands.onCommand.addListener( function( command ) {
-   var extensionURL;
-   switch ( command ) {
-   case 'open-statistics': extensionURL = 'dashboard.html#statistics'; break;
-   case 'open-ubiquitous-rules': extensionURL = 'dashboard.html#ubiquitous-rules'; break;
-   case 'open-scoped-rules': extensionURL = 'dashboard.html#scoped-rules'; break;
-   case 'open-settings': extensionURL = 'dashboard.html#settings'; break;
-   }
-   if ( extensionURL ) {
-      chrome.runtime.sendMessage( { what: 'gotoExtensionURL',
-                                     url: extensionURL } );
-   }
-});

manifest.json

@@ -57,29 +57,29 @@
         "css/noop.css"
     ],
     "commands": {
-       "open-statistics": {
-         "suggested_key": {
-           "default": "Alt+S"
-         },
-         "description": "Open Statistics"
-       },
+        "open-statistics": {
+            "suggested_key": {
+                "default": "Alt+S"
+            },
+            "description": "Open Statistics"
+        },
        "open-scoped-rules": {
-         "suggested_key": {
-           "default": "Alt+F"
-         },
-         "description": "Open Rule Manager"
-       },
+            "suggested_key": {
+                "default": "Alt+F"
+            },
+            "description": "Open Rule manager"
+        },
        "open-settings": {
-         "suggested_key": {
-           "default": "Alt+A"
-         },
-         "description": "Open Settings"
-       },
+            "suggested_key": {
+                "default": "Alt+A"
+            },
+            "description": "Open Settings"
+        },
        "open-ubiquitous-rules": {
-         "suggested_key": {
-           "default": "Ctrl+D"
-         },
-         "description": "Open Ubiquitous Rules"
-       }
+            "suggested_key": {
+                "default": "Ctrl+D"
+            },
+            "description": "Open Ubiquitous rules"
+        }
     }
 }