Bug 1471914 - Check that the font editor pref is on before rendering it. r=pbro draft
authorRazvan Caliman <rcaliman@mozilla.com>
Thu, 28 Jun 2018 17:16:54 +0200
changeset 812141 4f3c8b44de0b483a09a8f79b7318be6a16de18a8
parent 811977 6041c030780420b6205cf2d6640513606609884c
push id114454
push userbmo:rcaliman@mozilla.com
push dateThu, 28 Jun 2018 15:17:16 +0000
reviewerspbro
bugs1471914
milestone63.0a1
Bug 1471914 - Check that the font editor pref is on before rendering it. r=pbro MozReview-Commit-ID: C0uQkZCJ2Ck
devtools/client/inspector/fonts/components/FontsApp.js
devtools/client/inspector/fonts/fonts.js
--- a/devtools/client/inspector/fonts/components/FontsApp.js
+++ b/devtools/client/inspector/fonts/components/FontsApp.js
@@ -14,41 +14,43 @@ const FontOverview = createFactory(requi
 
 const Types = require("../types");
 
 class FontsApp extends PureComponent {
   static get propTypes() {
     return {
       fontData: PropTypes.shape(Types.fontData).isRequired,
       fontEditor: PropTypes.shape(Types.fontEditor).isRequired,
+      fontEditorEnabled: PropTypes.bool.isRequired,
       fontOptions: PropTypes.shape(Types.fontOptions).isRequired,
       onInstanceChange: PropTypes.func.isRequired,
       onPreviewFonts: PropTypes.func.isRequired,
       onPropertyChange: PropTypes.func.isRequired,
       onToggleFontHighlight: PropTypes.func.isRequired,
     };
   }
 
   render() {
     const {
       fontData,
       fontEditor,
+      fontEditorEnabled,
       fontOptions,
       onInstanceChange,
       onPreviewFonts,
       onPropertyChange,
       onToggleFontHighlight,
     } = this.props;
 
     return dom.div(
       {
         className: "theme-sidebar inspector-tabpanel",
         id: "sidebar-panel-fontinspector"
       },
-      FontEditor({
+      fontEditorEnabled && FontEditor({
         fontEditor,
         onInstanceChange,
         onPropertyChange,
         onToggleFontHighlight,
       }),
       FontOverview({
         fontData,
         fontOptions,
--- a/devtools/client/inspector/fonts/fonts.js
+++ b/devtools/client/inspector/fonts/fonts.js
@@ -88,16 +88,17 @@ class FontInspector {
   }
 
   init() {
     if (!this.inspector) {
       return;
     }
 
     const fontsApp = FontsApp({
+      fontEditorEnabled: Services.prefs.getBoolPref(PREF_FONT_EDITOR),
       onInstanceChange: this.onInstanceChange,
       onToggleFontHighlight: this.onToggleFontHighlight,
       onPreviewFonts: this.onPreviewFonts,
       onPropertyChange: this.onPropertyChange,
     });
 
     const provider = createElement(Provider, {
       id: "fontinspector",