Bug 1198830 - Disable toolboxfullscreen if sidebars are enabled. r=jryans

browser/devtools/webide/content/webide.js

@@ -1056,6 +1056,10 @@ let UI = {
   },
 
   updateToolboxFullscreenState: function() {
+    if (projectList.sidebarsEnabled) {
+      return;
+    }
+
     let panel = document.querySelector("#deck").selectedPanel;
     let nbox = document.querySelector("#notificationbox");
     if (panel && panel.id == "deck-panel-details" &&

browser/devtools/webide/test/sidebars/test_fullscreenToolbox.html

@@ -51,7 +51,7 @@
           ok(win.UI.toolboxIframe, "Toolbox iframe exists");
 
           let nbox = win.document.querySelector("#notificationbox");
-          ok(nbox.hasAttribute("toolboxfullscreen"), "Toolbox is fullsreen");
+          ok(!nbox.hasAttribute("toolboxfullscreen"), "Toolbox is not fullscreen");
 
           win.Cmds.showRuntimeDetails();