Bug 1118179 - Assume light theme colors for the font inspector as the default. r=bgrins
authorJan Odvarko <odvarko@gmail.com>
Wed, 07 Jan 2015 17:38:33 +0100
changeset 248739 20663b394b2c1c433d08f42a53874215c4d3f604
parent 248738 d511112704cfcc989a6ceb0f536d5e9ae4a7fcc5
child 248740 72ee3f94b5f8d471ccb48db5c9699dbe628434ce
push id4489
push userraliiev@mozilla.com
push dateMon, 23 Feb 2015 15:17:55 +0000
treeherdermozilla-beta@fd7c3dc24146 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersbgrins
bugs1118179
milestone37.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1118179 - Assume light theme colors for the font inspector as the default. r=bgrins
browser/devtools/fontinspector/font-inspector.js
--- a/browser/devtools/fontinspector/font-inspector.js
+++ b/browser/devtools/fontinspector/font-inspector.js
@@ -93,18 +93,19 @@ FontInspector.prototype = {
         !this.inspector.selection.isConnected() ||
         !this.inspector.selection.isElementNode() ||
         this.chromeDoc.body.classList.contains("dim")) {
       return;
     }
 
     this.chromeDoc.querySelector("#all-fonts").innerHTML = "";
 
-    let fillStyle = (Services.prefs.getCharPref("devtools.theme") == "light") ?
-        "black" : "white";
+    // Assume light theme colors as the default (see also bug 1118179).
+    let fillStyle = (Services.prefs.getCharPref("devtools.theme") == "dark") ?
+        "white" : "black";
     let options = {
       includePreviews: true,
       previewFillStyle: fillStyle
     }
     let fonts = [];
     if (showAllFonts){
       fonts = yield this.pageStyle.getAllUsedFontFaces(options)
                       .then(null, console.error);