Title: [186141] trunk/Source/WebInspectorUI
Revision
186141
Author
[email protected]
Date
2015-06-30 15:58:05 -0700 (Tue, 30 Jun 2015)

Log Message

Web Inspector: ProfileNode cleanup
https://bugs.webkit.org/show_bug.cgi?id=146472

Patch by Joseph Pecoraro <[email protected]> on 2015-06-30
Reviewed by Timothy Hatcher.

* UserInterface/Models/ProfileNode.js:
(WebInspector.ProfileNode):
Use Array.prototype.every instead of reduce.
Correct the name of a function to be what callers expect!

Modified Paths

Diff

Modified: trunk/Source/WebInspectorUI/ChangeLog (186140 => 186141)


--- trunk/Source/WebInspectorUI/ChangeLog	2015-06-30 22:51:16 UTC (rev 186140)
+++ trunk/Source/WebInspectorUI/ChangeLog	2015-06-30 22:58:05 UTC (rev 186141)
@@ -1,5 +1,17 @@
 2015-06-30  Joseph Pecoraro  <[email protected]>
 
+        Web Inspector: ProfileNode cleanup
+        https://bugs.webkit.org/show_bug.cgi?id=146472
+
+        Reviewed by Timothy Hatcher.
+
+        * UserInterface/Models/ProfileNode.js:
+        (WebInspector.ProfileNode):
+        Use Array.prototype.every instead of reduce.
+        Correct the name of a function to be what callers expect!
+
+2015-06-30  Joseph Pecoraro  <[email protected]>
+
         Web Inspector: iOS 8: Profile Nodes not showing in _javascript_ Timeline
         https://bugs.webkit.org/show_bug.cgi?id=146471
 

Modified: trunk/Source/WebInspectorUI/UserInterface/Models/ProfileNode.js (186140 => 186141)


--- trunk/Source/WebInspectorUI/UserInterface/Models/ProfileNode.js	2015-06-30 22:51:16 UTC (rev 186140)
+++ trunk/Source/WebInspectorUI/UserInterface/Models/ProfileNode.js	2015-06-30 22:58:05 UTC (rev 186141)
@@ -34,9 +34,9 @@
         console.assert(id);
         console.assert(calls instanceof Array);
         console.assert(calls.length >= 1);
-        console.assert(calls.reduce(function(previousValue, call) { return previousValue && call instanceof WebInspector.ProfileNodeCall; }, true));
+        console.assert(calls.every(function(call) { return call instanceof WebInspector.ProfileNodeCall; }));
         console.assert(childNodes instanceof Array);
-        console.assert(childNodes.reduce(function(previousValue, node) { return previousValue && node instanceof WebInspector.ProfileNode; }, true));
+        console.assert(childNodes.every(function(node) { return node instanceof WebInspector.ProfileNode; }));
 
         this._id = id;
         this._type = type || WebInspector.ProfileNode.Type.Function;
@@ -207,7 +207,7 @@
 
     // Private
 
-    _computeTotalTimes()
+    _computeTotalTimesIfNeeded()
     {
         if (this._computedTotalTimes)
             return;
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to