DevTools: Front-end for platform font inspection

Source/devtools/front_end/PlatformFontsSidebarPane.js

+/*
+ * Copyright (C) 2013 Google Inc. All rights reserved.
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions are
+ * met:
+ *
+ *     * Redistributions of source code must retain the above copyright
+ * notice, this list of conditions and the following disclaimer.
+ *     * Redistributions in binary form must reproduce the above
+ * copyright notice, this list of conditions and the following disclaimer
+ * in the documentation and/or other materials provided with the
+ * distribution.
+ *     * Neither the name of Google Inc. nor the names of its
+ * contributors may be used to endorse or promote products derived from
+ * this software without specific prior written permission.
+ *
+ * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
+ * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
+ * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
+ * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
+ * OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
+ * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
+ * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
+ * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
+ * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
+ * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
+ * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+ */
+
+/**
+ * @constructor
+ * @extends {WebInspector.SidebarPane}
+ */
+WebInspector.PlatformFontsSidebarPane = function()
+{
+    WebInspector.SidebarPane.call(this, WebInspector.UIString("Fonts"));
+    this.element.addStyleClass("platform-fonts");
+    WebInspector.domAgent.addEventListener(WebInspector.DOMAgent.Events.AttrModified, this._onNodeChange.bind(this));
+    WebInspector.domAgent.addEventListener(WebInspector.DOMAgent.Events.AttrRemoved, this._onNodeChange.bind(this));
+    WebInspector.domAgent.addEventListener(WebInspector.DOMAgent.Events.CharacterDataModified, this._onNodeChange.bind(this));
+
+    var cssFontSection = this.element.createChild("div", "stats-section monospace");
+    var cssFontPrefix = cssFontSection.createChild("span", "webkit-css-property");
+    cssFontPrefix.textContent = "font-family";
+    cssFontSection.createTextChild(":");
+    this._cssFontValue = cssFontSection.createChild("span", "css-font-value");
+
+    this._fontStatsSection = this.element.createChild("div", "stats-section");
+}
+
+WebInspector.PlatformFontsSidebarPane.prototype = {
+    _onNodeChange: function()
+    {
+        if (this._innerUpdateTimeout)
+            return;
+        this._innerUpdateTimeout = setTimeout(this._innerUpdate.bind(this), 100);
+    },
+
+    /**
+     * @param {WebInspector.DOMNode=} node
+     */
+    update: function(node)
+    {
+        if (!node) {
+            this.element.removeChildren();

[pfeldman, 2013/08/26 12:10:50]

Is this a bug?

+            delete this._node;
+            return;
+        }
+        this._node = node;
+        this._innerUpdate();
+    },
+
+    _innerUpdate: function()
+    {
+        if (this._innerUpdateTimeout) {
+            clearTimeout(this._innerUpdateTimeout);
+            delete this._innerUpdateTimeout;
+        }
+        if (!this._node)
+            return;
+        WebInspector.cssModel.getPlatformFontsForNode(this._node.id, this._render.bind(this, this._node));
+    },
+
+    /**
+     * @param {String} cssFamilyName
+     * @param {WebInspector.DOMNode} node
+     */
+    _render: function(node, cssFamilyName, platformFonts)

[pfeldman, 2013/08/26 12:10:50]

_refreshUI

+    {
+        if (this._node !== node)
+            return;
+        this._cssFontValue.textContent = cssFamilyName + ";";
+        this._fontStatsSection.removeChildren();
+
+        if (!platformFonts || !platformFonts.length)
+            return;
+        platformFonts.sort(function (a, b) {
+            return b.glyphCount - a.glyphCount;
+        });
+        this._fontStatsSection.remove();

[pfeldman, 2013/08/26 12:10:50]

don't

+        for (var i = 0; i < platformFonts.length; ++i) {
+            var fontStatElement = this._fontStatsSection.createChild("div", "font-stats-item");
+
+            var fontNameElement = fontStatElement.createChild("span", "font-name");
+            fontNameElement.textContent = platformFonts[i].familyName;
+
+            var fontDelimeterElement = fontStatElement.createChild("span", "delimeter");
+            fontDelimeterElement.textContent = "—";

[pfeldman, 2013/08/26 12:10:50]

Please use \u notation

+
+            var fontUsageElement = fontStatElement.createChild("span", "font-usage");
+            var usage = platformFonts[i].glyphCount;
+            fontUsageElement.textContent = usage === 1 ? WebInspector.UIString("%d glyph", usage) : WebInspector.UIString("%d glyphs", usage);
+        }
+        this.element.appendChild(this._fontStatsSection);
+    },
+
+    __proto__: WebInspector.SidebarPane.prototype
+}