Title: [176853] trunk/Source/WebInspectorUI
Revision
176853
Author
[email protected]
Date
2014-12-05 10:48:05 -0800 (Fri, 05 Dec 2014)

Log Message

Web Inspector: Uncaught Exceptions when attempting to show Content Flow
https://bugs.webkit.org/show_bug.cgi?id=139276

Patch by Joseph Pecoraro <[email protected]> on 2014-12-05
Reviewed by Brian Burg.

* UserInterface/Views/ContentView.js:
(WebInspector.ContentView):
(WebInspector.ContentView.isViewable):
LegacyJavaScriptProfileObject no longer exists, these references should be removed.

* UserInterface/Views/FolderizedTreeElement.js:
(WebInspector.FolderizedTreeElement.prototype._insertChildTreeElement):
This sort function references "this" and needed to be bound.

Modified Paths

Diff

Modified: trunk/Source/WebInspectorUI/ChangeLog (176852 => 176853)


--- trunk/Source/WebInspectorUI/ChangeLog	2014-12-05 18:46:56 UTC (rev 176852)
+++ trunk/Source/WebInspectorUI/ChangeLog	2014-12-05 18:48:05 UTC (rev 176853)
@@ -1,3 +1,19 @@
+2014-12-05  Joseph Pecoraro  <[email protected]>
+
+        Web Inspector: Uncaught Exceptions when attempting to show Content Flow
+        https://bugs.webkit.org/show_bug.cgi?id=139276
+
+        Reviewed by Brian Burg.
+
+        * UserInterface/Views/ContentView.js:
+        (WebInspector.ContentView):
+        (WebInspector.ContentView.isViewable):
+        LegacyJavaScriptProfileObject no longer exists, these references should be removed.
+
+        * UserInterface/Views/FolderizedTreeElement.js:
+        (WebInspector.FolderizedTreeElement.prototype._insertChildTreeElement):
+        This sort function references "this" and needed to be bound.
+
 2014-12-04  Joseph Pecoraro  <[email protected]>
 
         Web Inspector: Network Timeline Filter Bar only has "All", missing Resource Type filters

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/ContentView.js (176852 => 176853)


--- trunk/Source/WebInspectorUI/UserInterface/Views/ContentView.js	2014-12-05 18:46:56 UTC (rev 176852)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/ContentView.js	2014-12-05 18:48:05 UTC (rev 176853)
@@ -69,9 +69,6 @@
         if (representedObject instanceof WebInspector.LogObject)
             return new WebInspector.LogContentView(representedObject);
 
-        if (representedObject instanceof WebInspector.LegacyJavaScriptProfileObject)
-            return new WebInspector.LegacyJavaScriptProfileView(representedObject);
-
         if (representedObject instanceof WebInspector.ContentFlow)
             return new WebInspector.ContentFlowDOMTreeContentView(representedObject);
 
@@ -127,8 +124,6 @@
         return true;
     if (representedObject instanceof WebInspector.LogObject)
         return true;
-    if (representedObject instanceof WebInspector.LegacyJavaScriptProfileObject)
-        return true;
     if (representedObject instanceof WebInspector.ContentFlow)
         return true;
     if (typeof representedObject === "string" || representedObject instanceof String)

Modified: trunk/Source/WebInspectorUI/UserInterface/Views/FolderizedTreeElement.js (176852 => 176853)


--- trunk/Source/WebInspectorUI/UserInterface/Views/FolderizedTreeElement.js	2014-12-05 18:46:56 UTC (rev 176852)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/FolderizedTreeElement.js	2014-12-05 18:48:05 UTC (rev 176853)
@@ -237,7 +237,7 @@
     _insertChildTreeElement: function(parentTreeElement, childTreeElement)
     {
         console.assert(!childTreeElement.parent);
-        parentTreeElement.insertChild(childTreeElement, insertionIndexForObjectInListSortedByFunction(childTreeElement, parentTreeElement.children, this.compareChildTreeElements));
+        parentTreeElement.insertChild(childTreeElement, insertionIndexForObjectInListSortedByFunction(childTreeElement, parentTreeElement.children, this.compareChildTreeElements.bind(this)));
     },
 
     _removeTreeElement: function(childTreeElement, suppressOnDeselect, suppressSelectSibling)
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to