[Pkg-mozext-commits] [firebug] 09/18: Notification popup: auto enable the DOM panel together with Firebug theme

[David Prévot]

This is an automated email from the git hooks/post-receive script.

taffit pushed a commit to branch master
in repository firebug.

commit 38a171ec4fba569c0930f5000e15ad2dbbf1e8b2
Author: Jan Odvarko <odvarko at gmail.com>
Date:   Mon May 30 15:36:40 2016 +0200

    Notification popup: auto enable the DOM panel together with Firebug theme
---
 .../content/firebug/firefox/browserOverlay.js      | 24 ++++++++++++++++++++++
 1 file changed, 24 insertions(+)

diff --git a/extension/content/firebug/firefox/browserOverlay.js b/extension/content/firebug/firefox/browserOverlay.js
index 0eb1d62..67e6617 100644
--- a/extension/content/firebug/firefox/browserOverlay.js
+++ b/extension/content/firebug/firefox/browserOverlay.js
@@ -766,6 +766,7 @@ BrowserOverlay.prototype =
 
         this.oldTheme = Options.getPref("devtools", "theme");
         this.oldTheme = (this.oldTheme == "firebug") ? "light" : this.oldTheme;
+        this.domPanelWasEnabled = Options.getPref("devtools", "dom.enabled");
 
         var popupSet = $(this.doc, "mainPopupSet");
         var panel = this.doc.querySelector("fbNewMultiprocessNotificationPanel");
@@ -787,6 +788,7 @@ BrowserOverlay.prototype =
         if (firebugTheme)
         {
             this.activateFirebugTheme("firebug");
+            this.enableDomPanel(true);
         }
 
         panel.internationalize(Locale);
@@ -826,9 +828,14 @@ BrowserOverlay.prototype =
       var panel = this.doc.querySelector("fbNewMultiprocessNotificationPanel");
       var newTheme = panel.firebugTheme.checked ? "firebug" : this.oldTheme;
 
+      // Remember state of the checkbox.
       Options.set("activateFirebugTheme", panel.firebugTheme.checked);
 
+      // Activate/deactivate Firebug theme.
       this.activateFirebugTheme(newTheme);
+
+      // When Firebug theme is activated show also the DOM panel.
+      this.enableDomPanel(panel.firebugTheme.checked);
     },
 
     activateFirebugTheme: function(newTheme) {
@@ -845,6 +852,23 @@ BrowserOverlay.prototype =
       DevTools.gDevTools.emit("pref-changed", data);
     },
 
+    enableDomPanel: function(enable) {
+      var enableDomPanel = enable ? true : this.domPanelWasEnabled;
+      var currValue = Options.getPref("devtools", "dom.enabled");
+      if (currValue == enableDomPanel) {
+        return;
+      }
+
+      Options.setPref("devtools", "dom.enabled", enableDomPanel);
+
+      if (enableDomPanel) {
+        DevTools.gDevTools.emit("tool-registered", "dom");
+      } else {
+        let toolDefinition = DevTools.gDevTools._tools.get("dom");
+        DevTools.gDevTools.emit("tool-unregistered", toolDefinition);
+      }
+    },
+
     toggleDevTools: function(forceOpen) {
       var toolbox = getToolbox(this.win);
       if (toolbox && forceOpen) {

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/pkg-mozext/firebug.git