Bug 1297239 - Enable eslint for browser/components/shell. r=gijs draft
authorJared Wein <jwein@mozilla.com>
Tue, 23 Aug 2016 15:39:18 -0400
changeset 404572 228035ea8f70b5c8fd5a1348494ab42f6bb3bdb7
parent 404571 ce7d88fceb2220eaad70d3e108d0ef1b9b7f57a3
child 529217 357908b8b20ce36dc0d0c031ba3e0a5d25b44e98
push id27244
push userjwein@mozilla.com
push dateTue, 23 Aug 2016 19:40:07 +0000
reviewersgijs
bugs1297239
milestone51.0a1
Bug 1297239 - Enable eslint for browser/components/shell. r=gijs MozReview-Commit-ID: 9mCR495fwEn
.eslintignore
browser/components/shell/ShellService.jsm
browser/components/shell/content/setDesktopBackground.js
browser/components/shell/jar.mn
browser/components/shell/test/browser_420786.js
browser/components/shell/test/unit/test_421977.js
--- a/.eslintignore
+++ b/.eslintignore
@@ -61,17 +61,16 @@ browser/base/content/browser-social.js
 browser/base/content/nsContextMenu.js
 browser/base/content/sanitizeDialog.js
 browser/base/content/test/**
 browser/base/content/newtab/**
 browser/components/downloads/**
 browser/components/feeds/**
 browser/components/privatebrowsing/**
 browser/components/sessionstore/**
-browser/components/shell/**
 browser/components/tabview/**
 browser/components/translation/**
 browser/extensions/pdfjs/**
 browser/extensions/pocket/content/panels/js/vendor/**
 browser/locales/**
 
 # devtools/ exclusions
 devtools/client/canvasdebugger/**
--- a/browser/components/shell/ShellService.jsm
+++ b/browser/components/shell/ShellService.jsm
@@ -1,113 +1,114 @@
-/* This Source Code Form is subject to the terms of the Mozilla Public
-* License, v. 2.0. If a copy of the MPL was not distributed with this file,
-* You can obtain one at http://mozilla.org/MPL/2.0/. */
-
-"use strict";
-
-this.EXPORTED_SYMBOLS = ["ShellService"];
-
-const { classes: Cc, interfaces: Ci, utils: Cu, results: Cr } = Components;
-
-Cu.import("resource://gre/modules/AppConstants.jsm");
-Cu.import("resource://gre/modules/Services.jsm");
-Cu.import("resource://gre/modules/XPCOMUtils.jsm");
-XPCOMUtils.defineLazyModuleGetter(this, "WindowsRegistry",
-                                  "resource://gre/modules/WindowsRegistry.jsm");
-
-/**
- * Internal functionality to save and restore the docShell.allow* properties.
- */
-let ShellServiceInternal = {
-  /**
-   * Used to determine whether or not to offer "Set as desktop background"
-   * functionality. Even if shell service is available it is not
-   * guaranteed that it is able to set the background for every desktop
-   * which is especially true for Linux with its many different desktop
-   * environments.
-   */
-  get canSetDesktopBackground() {
-    if (AppConstants.platform == "win" ||
-        AppConstants.platform == "macosx") {
-      return true;
-    }
-
-    if (AppConstants.platform == "linux") {
-      if (this.shellService) {
-        let linuxShellService = this.shellService
-                                    .QueryInterface(Ci.nsIGNOMEShellService);
-        return linuxShellService.canSetDesktopBackground;
-      }
-    }
-
-    return false;
-  },
-
-  /**
-   * Used to determine whether or not to show a "Set Default Browser"
-   * query dialog. This attribute is true if the application is starting
-   * up and "browser.shell.checkDefaultBrowser" is true, otherwise it
-   * is false.
-   */
-  _checkedThisSession: false,
-  get shouldCheckDefaultBrowser() {
-    // If we've already checked, the browser has been started and this is a
-    // new window open, and we don't want to check again.
-    if (this._checkedThisSession) {
-      return false;
-    }
-
-    if (!Services.prefs.getBoolPref("browser.shell.checkDefaultBrowser")) {
-      return false;
-    }
-
-    if (AppConstants.platform == "win") {
-      let optOutValue = WindowsRegistry.readRegKey(Ci.nsIWindowsRegKey.ROOT_KEY_CURRENT_USER,
-                                                   "Software\\Mozilla\\Firefox",
-                                                   "DefaultBrowserOptOut");
-      WindowsRegistry.removeRegKey(Ci.nsIWindowsRegKey.ROOT_KEY_CURRENT_USER,
-                                   "Software\\Mozilla\\Firefox",
-                                   "DefaultBrowserOptOut");
-      if (optOutValue == "True") {
-        Services.prefs.setBoolPref("browser.shell.checkDefaultBrowser", false);
-        return false;
-      }
-    }
-
-    return true;
-  },
-
-  set shouldCheckDefaultBrowser(shouldCheck) {
-    Services.prefs.setBoolPref("browser.shell.checkDefaultBrowser", !!shouldCheck);
-  },
-
-  isDefaultBrowser(startupCheck, forAllTypes) {
-    // If this is the first browser window, maintain internal state that we've
-    // checked this session (so that subsequent window opens don't show the
-    // default browser dialog).
-    if (startupCheck) {
-      this._checkedThisSession = true;
-    }
-    if (this.shellService) {
-      return this.shellService.isDefaultBrowser(startupCheck, forAllTypes);
-    }
-  }
-};
-
-XPCOMUtils.defineLazyServiceGetter(ShellServiceInternal, "shellService",
-  "@mozilla.org/browser/shell-service;1", Ci.nsIShellService);
-
-/**
- * The external API exported by this module.
- */
-this.ShellService = new Proxy(ShellServiceInternal, {
-  get(target, name) {
-    if (name in target) {
-      return target[name];
-    }
-    if (target.shellService) {
-      return target.shellService[name];
-    }
-    Services.console.logStringMessage(`${name} not found in ShellService: ${target.shellService}`);
-    return undefined;
-  }
-});
+/* This Source Code Form is subject to the terms of the Mozilla Public
+* License, v. 2.0. If a copy of the MPL was not distributed with this file,
+* You can obtain one at http://mozilla.org/MPL/2.0/. */
+
+"use strict";
+
+this.EXPORTED_SYMBOLS = ["ShellService"];
+
+const { classes: Cc, interfaces: Ci, utils: Cu, results: Cr } = Components;
+
+Cu.import("resource://gre/modules/AppConstants.jsm");
+Cu.import("resource://gre/modules/Services.jsm");
+Cu.import("resource://gre/modules/XPCOMUtils.jsm");
+XPCOMUtils.defineLazyModuleGetter(this, "WindowsRegistry",
+                                  "resource://gre/modules/WindowsRegistry.jsm");
+
+/**
+ * Internal functionality to save and restore the docShell.allow* properties.
+ */
+let ShellServiceInternal = {
+  /**
+   * Used to determine whether or not to offer "Set as desktop background"
+   * functionality. Even if shell service is available it is not
+   * guaranteed that it is able to set the background for every desktop
+   * which is especially true for Linux with its many different desktop
+   * environments.
+   */
+  get canSetDesktopBackground() {
+    if (AppConstants.platform == "win" ||
+        AppConstants.platform == "macosx") {
+      return true;
+    }
+
+    if (AppConstants.platform == "linux") {
+      if (this.shellService) {
+        let linuxShellService = this.shellService
+                                    .QueryInterface(Ci.nsIGNOMEShellService);
+        return linuxShellService.canSetDesktopBackground;
+      }
+    }
+
+    return false;
+  },
+
+  /**
+   * Used to determine whether or not to show a "Set Default Browser"
+   * query dialog. This attribute is true if the application is starting
+   * up and "browser.shell.checkDefaultBrowser" is true, otherwise it
+   * is false.
+   */
+  _checkedThisSession: false,
+  get shouldCheckDefaultBrowser() {
+    // If we've already checked, the browser has been started and this is a
+    // new window open, and we don't want to check again.
+    if (this._checkedThisSession) {
+      return false;
+    }
+
+    if (!Services.prefs.getBoolPref("browser.shell.checkDefaultBrowser")) {
+      return false;
+    }
+
+    if (AppConstants.platform == "win") {
+      let optOutValue = WindowsRegistry.readRegKey(Ci.nsIWindowsRegKey.ROOT_KEY_CURRENT_USER,
+                                                   "Software\\Mozilla\\Firefox",
+                                                   "DefaultBrowserOptOut");
+      WindowsRegistry.removeRegKey(Ci.nsIWindowsRegKey.ROOT_KEY_CURRENT_USER,
+                                   "Software\\Mozilla\\Firefox",
+                                   "DefaultBrowserOptOut");
+      if (optOutValue == "True") {
+        Services.prefs.setBoolPref("browser.shell.checkDefaultBrowser", false);
+        return false;
+      }
+    }
+
+    return true;
+  },
+
+  set shouldCheckDefaultBrowser(shouldCheck) {
+    Services.prefs.setBoolPref("browser.shell.checkDefaultBrowser", !!shouldCheck);
+  },
+
+  isDefaultBrowser(startupCheck, forAllTypes) {
+    // If this is the first browser window, maintain internal state that we've
+    // checked this session (so that subsequent window opens don't show the
+    // default browser dialog).
+    if (startupCheck) {
+      this._checkedThisSession = true;
+    }
+    if (this.shellService) {
+      return this.shellService.isDefaultBrowser(startupCheck, forAllTypes);
+    }
+    return false;
+  }
+};
+
+XPCOMUtils.defineLazyServiceGetter(ShellServiceInternal, "shellService",
+  "@mozilla.org/browser/shell-service;1", Ci.nsIShellService);
+
+/**
+ * The external API exported by this module.
+ */
+this.ShellService = new Proxy(ShellServiceInternal, {
+  get(target, name) {
+    if (name in target) {
+      return target[name];
+    }
+    if (target.shellService) {
+      return target.shellService[name];
+    }
+    Services.console.logStringMessage(`${name} not found in ShellService: ${target.shellService}`);
+    return undefined;
+  }
+});
--- a/browser/components/shell/content/setDesktopBackground.js
+++ b/browser/components/shell/content/setDesktopBackground.js
@@ -1,54 +1,52 @@
-# This Source Code Form is subject to the terms of the Mozilla Public
-# License, v. 2.0. If a copy of the MPL was not distributed with this
-# file, You can obtain one at http://mozilla.org/MPL/2.0/.
+/* This Source Code Form is subject to the terms of the Mozilla Public
+ * License, v. 2.0. If a copy of the MPL was not distributed with this file,
+ * You can obtain one at http://mozilla.org/MPL/2.0/. */
+
+Components.utils.import("resource://gre/modules/AppConstants.jsm");
 
 var Ci = Components.interfaces;
 
 var gSetBackground = {
-#ifndef XP_MACOSX
-  _position        : "",
-  _backgroundColor : 0,
-#else
-  _position        : "STRETCH",
-#endif
+  _position        : AppConstants.platform == "macosx" ? "STRETCH" : "",
+  _backgroundColor : AppConstants.platform != "macosx" ? 0 : undefined,
   _screenWidth     : 0,
   _screenHeight    : 0,
   _image           : null,
   _canvas          : null,
 
   get _shell()
   {
     return Components.classes["@mozilla.org/browser/shell-service;1"]
                      .getService(Ci.nsIShellService);
   },
 
   load: function ()
   {
     this._canvas = document.getElementById("screen");
     this._screenWidth = screen.width;
     this._screenHeight = screen.height;
-#ifdef XP_MACOSX
-    document.documentElement.getButton("accept").hidden = true;
-#endif
+    if (AppConstants.platform == "macosx") {
+      document.documentElement.getButton("accept").hidden = true;
+    }
     if (this._screenWidth / this._screenHeight >= 1.6)
       document.getElementById("monitor").setAttribute("aspectratio", "16:10");
 
-#ifdef XP_WIN
-    // hide fill + fit options if <win7 since don't work 
-    var version = Components.classes["@mozilla.org/system-info;1"]
-                  .getService(Ci.nsIPropertyBag2)
-                  .getProperty("version");
-    var isWindows7OrHigher = (parseFloat(version) >= 6.1);
-    if (!isWindows7OrHigher) {
-      document.getElementById("fillPosition").hidden = true;
-      document.getElementById("fitPosition").hidden = true;
+    if (AppConstants.platform == "win") {
+      // Hide fill + fit options if < Win7 since they don't work.
+      var version = Components.classes["@mozilla.org/system-info;1"]
+                    .getService(Ci.nsIPropertyBag2)
+                    .getProperty("version");
+      var isWindows7OrHigher = (parseFloat(version) >= 6.1);
+      if (!isWindows7OrHigher) {
+        document.getElementById("fillPosition").hidden = true;
+        document.getElementById("fitPosition").hidden = true;
+      }
     }
-#endif
 
     // make sure that the correct dimensions will be used
     setTimeout(function(self) {
       self.init(window.arguments[0]);
     }, 0, this);
   },
 
   init: function (aImage)
@@ -57,157 +55,160 @@ var gSetBackground = {
 
     // set the size of the coordinate space
     this._canvas.width = this._canvas.clientWidth;
     this._canvas.height = this._canvas.clientHeight;
 
     var ctx = this._canvas.getContext("2d");
     ctx.scale(this._canvas.clientWidth / this._screenWidth, this._canvas.clientHeight / this._screenHeight);
 
-#ifndef XP_MACOSX
-    this._initColor();
-#else
-    // Make sure to reset the button state in case the user has already
-    // set an image as their desktop background.
-    var setDesktopBackground = document.getElementById("setDesktopBackground");
-    setDesktopBackground.hidden = false;
-    var bundle = document.getElementById("backgroundBundle");
-    setDesktopBackground.label = bundle.getString("DesktopBackgroundSet");
-    setDesktopBackground.disabled = false;
+    if (AppConstants.platform != "macosx") {
+      this._initColor();
+    } else {
+      // Make sure to reset the button state in case the user has already
+      // set an image as their desktop background.
+      var setDesktopBackground = document.getElementById("setDesktopBackground");
+      setDesktopBackground.hidden = false;
+      var bundle = document.getElementById("backgroundBundle");
+      setDesktopBackground.label = bundle.getString("DesktopBackgroundSet");
+      setDesktopBackground.disabled = false;
 
-    document.getElementById("showDesktopPreferences").hidden = true;
-#endif
+      document.getElementById("showDesktopPreferences").hidden = true;
+    }
     this.updatePosition();
   },
 
-#ifndef XP_MACOSX
-  _initColor: function ()
+  setDesktopBackground: function ()
+  {
+    if (AppConstants.platform != "macosx") {
+      document.persist("menuPosition", "value");
+      this._shell.desktopBackgroundColor = this._hexStringToLong(this._backgroundColor);
+    } else {
+      Components.classes["@mozilla.org/observer-service;1"]
+                .getService(Ci.nsIObserverService)
+                .addObserver(this, "shell:desktop-background-changed", false);
+
+      var bundle = document.getElementById("backgroundBundle");
+      var setDesktopBackground = document.getElementById("setDesktopBackground");
+      setDesktopBackground.disabled = true;
+      setDesktopBackground.label = bundle.getString("DesktopBackgroundDownloading");
+    }
+    this._shell.setDesktopBackground(this._image,
+                                     Ci.nsIShellService["BACKGROUND_" + this._position]);
+  },
+
+  updatePosition: function ()
+  {
+    var ctx = this._canvas.getContext("2d");
+    ctx.clearRect(0, 0, this._screenWidth, this._screenHeight);
+
+    if (AppConstants.platform != "macosx") {
+      this._position = document.getElementById("menuPosition").value;
+    }
+
+    switch (this._position) {
+      case "TILE":
+        ctx.save();
+        ctx.fillStyle = ctx.createPattern(this._image, "repeat");
+        ctx.fillRect(0, 0, this._screenWidth, this._screenHeight);
+        ctx.restore();
+        break;
+      case "STRETCH":
+        ctx.drawImage(this._image, 0, 0, this._screenWidth, this._screenHeight);
+        break;
+      case "CENTER": {
+        let x = (this._screenWidth - this._image.naturalWidth) / 2;
+        let y = (this._screenHeight - this._image.naturalHeight) / 2;
+        ctx.drawImage(this._image, x, y);
+        break;
+      }
+      case "FILL": {
+        // Try maxing width first, overflow height.
+        let widthRatio = this._screenWidth / this._image.naturalWidth;
+        let width = this._image.naturalWidth * widthRatio;
+        let height = this._image.naturalHeight * widthRatio;
+        if (height < this._screenHeight) {
+          // Height less than screen, max height and overflow width.
+          let heightRatio = this._screenHeight / this._image.naturalHeight;
+          width = this._image.naturalWidth * heightRatio;
+          height = this._image.naturalHeight * heightRatio;
+        }
+        let x = (this._screenWidth - width) / 2;
+        let y = (this._screenHeight - height) / 2;
+        ctx.drawImage(this._image, x, y, width, height);
+        break;
+      }
+      case "FIT": {
+        // Try maxing width first, top and bottom borders.
+        let widthRatio = this._screenWidth / this._image.naturalWidth;
+        let width = this._image.naturalWidth * widthRatio;
+        let height = this._image.naturalHeight * widthRatio;
+        let x = 0;
+        let y = (this._screenHeight - height) / 2;
+        if (height > this._screenHeight) {
+          // Height overflow, maximise height, side borders.
+          let heightRatio = this._screenHeight / this._image.naturalHeight;
+          width = this._image.naturalWidth * heightRatio;
+          height = this._image.naturalHeight * heightRatio;
+          x = (this._screenWidth - width) / 2;
+          y = 0;
+        }
+        ctx.drawImage(this._image, x, y, width, height);
+        break;
+      }
+    }
+  }
+};
+
+if (AppConstants.platform != "macosx") {
+  gSetBackground["_initColor"] = function ()
   {
     var color = this._shell.desktopBackgroundColor;
 
     const rMask = 4294901760;
     const gMask = 65280;
     const bMask = 255;
     var r = (color & rMask) >> 16;
     var g = (color & gMask) >> 8;
     var b = (color & bMask);
     this.updateColor(this._rgbToHex(r, g, b));
 
     var colorpicker = document.getElementById("desktopColor");
     colorpicker.color = this._backgroundColor;
-  },
+  };
 
-  updateColor: function (aColor)
+  gSetBackground["updateColor"] = function (aColor)
   {
     this._backgroundColor = aColor;
     this._canvas.style.backgroundColor = aColor;
-  },
+  };
 
   // Converts a color string in the format "#RRGGBB" to an integer.
-  _hexStringToLong: function (aString)
+  gSetBackground["_hexStringToLong"] = function (aString)
   {
-    return parseInt(aString.substring(1,3), 16) << 16 |
-           parseInt(aString.substring(3,5), 16) << 8 |
-           parseInt(aString.substring(5,7), 16);
-  },
+    return parseInt(aString.substring(1, 3), 16) << 16 |
+           parseInt(aString.substring(3, 5), 16) << 8 |
+           parseInt(aString.substring(5, 7), 16);
+  };
 
-  _rgbToHex: function (aR, aG, aB)
+  gSetBackground["_rgbToHex"] = function (aR, aG, aB)
   {
     return "#" + [aR, aG, aB].map(aInt => aInt.toString(16).replace(/^(.)$/, "0$1"))
                              .join("").toUpperCase();
-  },
-#else
-  observe: function (aSubject, aTopic, aData)
+  };
+} else {
+  gSetBackground["observe"] = function (aSubject, aTopic, aData)
   {
     if (aTopic == "shell:desktop-background-changed") {
       document.getElementById("setDesktopBackground").hidden = true;
       document.getElementById("showDesktopPreferences").hidden = false;
 
       Components.classes["@mozilla.org/observer-service;1"]
                 .getService(Ci.nsIObserverService)
                 .removeObserver(this, "shell:desktop-background-changed");
     }
-  },
+  };
 
-  showDesktopPrefs: function()
+  gSetBackground["showDesktopPrefs"] = function()
   {
     this._shell.openApplication(Ci.nsIMacShellService.APPLICATION_DESKTOP);
-  },
-#endif
-
-  setDesktopBackground: function ()
-  {
-#ifndef XP_MACOSX
-    document.persist("menuPosition", "value");
-    this._shell.desktopBackgroundColor = this._hexStringToLong(this._backgroundColor);
-#else
-    Components.classes["@mozilla.org/observer-service;1"]
-              .getService(Ci.nsIObserverService)
-              .addObserver(this, "shell:desktop-background-changed", false);
-
-    var bundle = document.getElementById("backgroundBundle");
-    var setDesktopBackground = document.getElementById("setDesktopBackground");
-    setDesktopBackground.disabled = true;
-    setDesktopBackground.label = bundle.getString("DesktopBackgroundDownloading");
-#endif
-    this._shell.setDesktopBackground(this._image,
-                                     Ci.nsIShellService["BACKGROUND_" + this._position]);
-  },
-
-  updatePosition: function ()
-  {
-    var ctx = this._canvas.getContext("2d");
-    ctx.clearRect(0, 0, this._screenWidth, this._screenHeight);
-
-#ifndef XP_MACOSX
-    this._position = document.getElementById("menuPosition").value;
-#endif
-
-    switch (this._position) {
-      case "TILE":
-        ctx.save();
-        ctx.fillStyle = ctx.createPattern(this._image, "repeat");
-        ctx.fillRect(0, 0, this._screenWidth, this._screenHeight);
-        ctx.restore();
-        break;
-      case "STRETCH":
-        ctx.drawImage(this._image, 0, 0, this._screenWidth, this._screenHeight);
-        break;
-      case "CENTER":
-        var x = (this._screenWidth - this._image.naturalWidth) / 2;
-        var y = (this._screenHeight - this._image.naturalHeight) / 2;
-        ctx.drawImage(this._image, x, y);
-        break;
-      case "FILL":
-        //Try maxing width first, overflow height
-        var widthRatio = this._screenWidth / this._image.naturalWidth;
-        var width = this._image.naturalWidth * widthRatio;
-        var height = this._image.naturalHeight * widthRatio;
-        if (height < this._screenHeight) {
-          //height less than screen, max height and overflow width
-          var heightRatio = this._screenHeight / this._image.naturalHeight;
-          width = this._image.naturalWidth * heightRatio;
-          height = this._image.naturalHeight * heightRatio;
-        }
-        var x = (this._screenWidth - width) / 2;
-        var y = (this._screenHeight - height) / 2;
-        ctx.drawImage(this._image, x, y, width, height);
-        break;
-      case "FIT":
-        //Try maxing width first, top and bottom borders
-        var widthRatio = this._screenWidth / this._image.naturalWidth;
-        var width = this._image.naturalWidth * widthRatio;
-        var height = this._image.naturalHeight * widthRatio;
-        var x = 0;
-        var y = (this._screenHeight - height) / 2;
-        if (height > this._screenHeight) {
-          //height overflow, maximise height, side borders
-          var heightRatio = this._screenHeight / this._image.naturalHeight;
-          width = this._image.naturalWidth * heightRatio;
-          height = this._image.naturalHeight * heightRatio;
-          x = (this._screenWidth - width) / 2;
-          y = 0;
-        }
-        ctx.drawImage(this._image, x, y, width, height);
-        break;      
-    }
-  }
-};
+  };
+}
--- a/browser/components/shell/jar.mn
+++ b/browser/components/shell/jar.mn
@@ -1,7 +1,7 @@
 # This Source Code Form is subject to the terms of the Mozilla Public
 # License, v. 2.0. If a copy of the MPL was not distributed with this
 # file, You can obtain one at http://mozilla.org/MPL/2.0/.
 
 browser.jar:
 *       content/browser/setDesktopBackground.xul              (content/setDesktopBackground.xul)
-*       content/browser/setDesktopBackground.js               (content/setDesktopBackground.js)
+        content/browser/setDesktopBackground.js               (content/setDesktopBackground.js)
--- a/browser/components/shell/test/browser_420786.js
+++ b/browser/components/shell/test/browser_420786.js
@@ -76,16 +76,16 @@ function test() {
   try {
     // If GSettings is available, then the GConf tests
     // will fail
     var gsettings = Cc["@mozilla.org/gsettings-service;1"].
                     getService(Ci.nsIGSettingsService).
                     getCollectionForSchema("org.gnome.desktop.background");
     todo(false, "This test doesn't work when GSettings is available");
     return;
-  } catch(e) { }
+  } catch (e) { }
 
   gBrowser.selectedTab = gBrowser.addTab();
   gBrowser.selectedBrowser.addEventListener("load", onPageLoad, true);
   content.location = "about:logo";
 
   waitForExplicitFinish();
 }
--- a/browser/components/shell/test/unit/test_421977.js
+++ b/browser/components/shell/test/unit/test_421977.js
@@ -25,19 +25,19 @@ function colorToHex(aColor) {
                              .join("").toUpperCase();
 }
 
 /**
  * Converts a color string in #RRGGBB format to a rgb numerical color value
  *  (r << 16 | g << 8 | b).
  */
 function hexToColor(aString) {
-  return parseInt(aString.substring(1,3), 16) << 16 |
-         parseInt(aString.substring(3,5), 16) << 8 |
-         parseInt(aString.substring(5,7), 16);
+  return parseInt(aString.substring(1, 3), 16) << 16 |
+         parseInt(aString.substring(3, 5), 16) << 8 |
+         parseInt(aString.substring(5, 7), 16);
 }
 
 /**
  * Checks that setting the GConf background key to aGConfColor will
  * result in the Shell component returning a background color equals
  * to aExpectedShellColor in #RRGGBB format.
  */
 function checkGConfToShellColor(aGConfColor, aExpectedShellColor) {
@@ -69,17 +69,17 @@ function run_test() {
 
   try {
     // If GSettings is available, then the GConf tests
     // will fail
     var gsettings = Cc["@mozilla.org/gsettings-service;1"].
                     getService(Ci.nsIGSettingsService).
                     getCollectionForSchema("org.gnome.desktop.background");
     return;
-  } catch(e) { }
+  } catch (e) { }
 
   gGConf = Cc["@mozilla.org/gnome-gconf-service;1"].
            getService(Ci.nsIGConfService);
 
   gShell = Cc["@mozilla.org/browser/shell-service;1"].
            getService(Ci.nsIShellService);
 
   // Save the original background color so that we can restore it