Changeset 114368 in webkit

Timestamp:

Apr 17, 2012, 6:03:55 AM (13 years ago)

Author:

pfeldman@chromium.org

Message:

Web Inspector: [regression r112413]content scripts tree outline is not added to the tab.
​https://bugs.webkit.org/show_bug.cgi?id=84146

Reviewed by Vsevolod Vlasov.

Adding the tree to the container.

• inspector/front-end/ScriptsNavigator.js:

(WebInspector.ScriptsNavigator):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• inspector/front-end/ScriptsNavigator.js (modified) (2 diffs)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2012-04-17  Pavel Feldman  <pfeldman@chromium.org>
+
+        Web Inspector: [regression r112413]content scripts tree outline is not added to the tab.
+        https://bugs.webkit.org/show_bug.cgi?id=84146
+
+        Reviewed by Vsevolod Vlasov.
+
+        Adding the tree to the container.
+
+        * inspector/front-end/ScriptsNavigator.js:
+        (WebInspector.ScriptsNavigator):
+
 2012-04-17  Mariusz Grzegorczyk  <mariusz.g@samsung.com>
 

trunk/Source/WebCore/inspector/front-end/ScriptsNavigator.js

@@ -48 +48 @@
 
     var scriptsTreeElement = document.createElement("ol");
-    var scriptsView = new WebInspector.View();
-    scriptsView.element.addStyleClass("fill");
-    scriptsView.element.addStyleClass("navigator-container");
+    this._scriptsTree = new WebInspector.NavigatorTreeOutline(this, scriptsTreeElement);
+
     var scriptsOutlineElement = document.createElement("div");
     scriptsOutlineElement.addStyleClass("outline-disclosure");
     scriptsOutlineElement.addStyleClass("navigator");
     scriptsOutlineElement.appendChild(scriptsTreeElement);
+
+    var scriptsView = new WebInspector.View();
+    scriptsView.element.addStyleClass("fill");
+    scriptsView.element.addStyleClass("navigator-container");
     scriptsView.element.appendChild(scriptsOutlineElement);
-    this._scriptsTree = new WebInspector.NavigatorTreeOutline(this, scriptsTreeElement);
+
     this._tabbedPane.appendTab(WebInspector.ScriptsNavigator.ScriptsTab, WebInspector.UIString("Scripts"), scriptsView);
     this._tabbedPane.selectTab(WebInspector.ScriptsNavigator.ScriptsTab);
 
     var contentScriptsTreeElement = document.createElement("ol");
-    var contentScriptsView = new WebInspector.View();
-    contentScriptsView.element.addStyleClass("fill");
-    contentScriptsView.element.addStyleClass("navigator-container");
+    this._contentScriptsTree = new WebInspector.NavigatorTreeOutline(this, contentScriptsTreeElement);
+
     var contentScriptsOutlineElement = document.createElement("div");
     contentScriptsOutlineElement.addStyleClass("outline-disclosure");
     contentScriptsOutlineElement.addStyleClass("navigator");
     contentScriptsOutlineElement.appendChild(contentScriptsTreeElement);
-    this._contentScriptsTree = new WebInspector.NavigatorTreeOutline(this, contentScriptsTreeElement);
+
+    var contentScriptsView = new WebInspector.View();
+    contentScriptsView.element.addStyleClass("fill");
+    contentScriptsView.element.addStyleClass("navigator-container");
+    contentScriptsView.element.appendChild(contentScriptsOutlineElement);
+
     this._tabbedPane.appendTab(WebInspector.ScriptsNavigator.ContentScriptsTab, WebInspector.UIString("Content scripts"), contentScriptsView);
 
@@ -473 +480 @@
         if ((domain === "" && folderName === "") || !showScriptFolders)
             return isContentScript ? this._contentScriptsTree : this._scriptsTree;
-        
+
         var parentFolderElement;
         if (folderName === "")