DevTools: split platformFontsWidget.css out of elementsPanel.css

third_party/WebKit/Source/devtools/front_end/elements/PlatformFontsWidget.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
@@ skipping 17 matching lines @@
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
 /**
  * @constructor
  * @extends {WebInspector.ThrottledWidget}
  * @param {!WebInspector.SharedSidebarModel} sharedModel
  */
 WebInspector.PlatformFontsWidget = function(sharedModel)
 {
     WebInspector.ThrottledWidget.call(this);

[dgozman, 2016/03/16 18:59:07]

Can we make it a web component as well (pass true here)?

[lushnikov, 2016/03/16 20:18:19]

Done.

     this.element.classList.add("platform-fonts");

[dgozman, 2016/03/16 18:59:07]

Perhaps, you don't need this class anymore and could use :host instead?

[lushnikov, 2016/03/16 20:18:19]

Done.

 
     this._sharedModel = sharedModel;
     this._sharedModel.addEventListener(WebInspector.SharedSidebarModel.Events.ComputedStyleChanged, this.update, this);
 
     this._sectionTitle = createElementWithClass("div", "sidebar-separator");
     this.element.appendChild(this._sectionTitle);
     this._sectionTitle.textContent = WebInspector.UIString("Rendered Fonts");
     this._fontStatsSection = this.element.createChild("div", "stats-section");
+    this.registerRequiredCSS("elements/platformFontsWidget.css");

[dgozman, 2016/03/16 18:59:07]

nit: usually do that in a second line of constructor.

[lushnikov, 2016/03/16 20:18:19]

Done.

 }
 
 /**
  * @param {!WebInspector.SharedSidebarModel} sharedModel
  * @return {!WebInspector.ElementsSidebarViewWrapperPane}
  */
 WebInspector.PlatformFontsWidget.createSidebarWrapper = function(sharedModel)
 {
     var widget = new WebInspector.PlatformFontsWidget(sharedModel);
     return new WebInspector.ElementsSidebarViewWrapperPane(WebInspector.UIString("Fonts"), widget)
@@ skipping 48 matching lines @@
             fontOrigin.textContent = platformFonts[i].isCustomFont? WebInspector.UIString("Network resource") : WebInspector.UIString("Local file");
 
             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);
         }
     },
 
     __proto__: WebInspector.ThrottledWidget.prototype
 }