Changeset 140534 in webkit

Timestamp:

Jan 23, 2013 6:58:44 AM (11 years ago)

Author:

yurys@chromium.org

Message:

Web Inspector: some measurements are missing in PerformanceTests/inspector/heap-*.html
​https://bugs.webkit.org/show_bug.cgi?id=107687

Reviewed by Pavel Feldman.

• inspector/heap-snapshot-performance-test.js: override methods on JSHeapSnapshot instead

of HeapSnapshot, otherwise overrides from JSHeapSnapshot will rule.

Location:

trunk/PerformanceTests

Files:

• ChangeLog (modified) (1 diff)
• inspector/heap-snapshot-performance-test.js (modified) (1 diff)

trunk/PerformanceTests/ChangeLog

@@ +1 @@
+2013-01-23  Yury Semikhatsky  <yurys@chromium.org>
+
+        Web Inspector: some measurements are missing in PerformanceTests/inspector/heap-*.html
+        https://bugs.webkit.org/show_bug.cgi?id=107687
+
+        Reviewed by Pavel Feldman.
+
+        * inspector/heap-snapshot-performance-test.js: override methods on JSHeapSnapshot instead
+        of HeapSnapshot, otherwise overrides from JSHeapSnapshot will rule.
+
 2013-01-18  Ryosuke Niwa  <rniwa@webkit.org>
 

trunk/PerformanceTests/inspector/heap-snapshot-performance-test.js

@@ -2 +2 @@
 {
     WebInspector.showPanel("profiles");
-    InspectorTest.measureFunction(WebInspector.HeapSnapshot.prototype, "_buildEdgeIndexes");
-    InspectorTest.measureFunction(WebInspector.HeapSnapshot.prototype, "_buildRetainers");
-    InspectorTest.measureFunction(WebInspector.HeapSnapshot.prototype, "_buildDominatedNodes");
-    InspectorTest.measureFunction(WebInspector.HeapSnapshot.prototype, "_calculateFlags");
-    InspectorTest.measureFunction(WebInspector.HeapSnapshot.prototype, "_buildAggregates");
-    InspectorTest.measureFunction(WebInspector.HeapSnapshot.prototype, "_calculateClassesRetainedSize");
-    InspectorTest.measureFunction(WebInspector.HeapSnapshot.prototype, "_calculateDistances");
-    InspectorTest.measureFunction(WebInspector.HeapSnapshot.prototype, "_calculateRetainedSizes");
-    InspectorTest.measureFunction(WebInspector.HeapSnapshot.prototype, "_markDetachedDOMTreeNodes");
-    InspectorTest.measureFunction(WebInspector.HeapSnapshot.prototype, "_markQueriableHeapObjects");
-    InspectorTest.measureFunction(WebInspector.HeapSnapshot.prototype, "_markPageOwnedNodes");
-    InspectorTest.measureFunction(WebInspector.HeapSnapshot.prototype, "_splitNodesAndContainmentEdges");
-    InspectorTest.measureFunction(WebInspector.HeapSnapshot.prototype, "_buildPostOrderIndex");
-    InspectorTest.measureFunction(WebInspector.HeapSnapshot.prototype, "_buildDominatorTree");
+    InspectorTest.measureFunction(WebInspector.JSHeapSnapshot.prototype, "_buildEdgeIndexes");
+    InspectorTest.measureFunction(WebInspector.JSHeapSnapshot.prototype, "_buildRetainers");
+    InspectorTest.measureFunction(WebInspector.JSHeapSnapshot.prototype, "_buildDominatedNodes");
+    InspectorTest.measureFunction(WebInspector.JSHeapSnapshot.prototype, "_calculateFlags");
+    InspectorTest.measureFunction(WebInspector.JSHeapSnapshot.prototype, "_buildAggregates");
+    InspectorTest.measureFunction(WebInspector.JSHeapSnapshot.prototype, "_calculateClassesRetainedSize");
+    InspectorTest.measureFunction(WebInspector.JSHeapSnapshot.prototype, "_calculateDistances");
+    InspectorTest.measureFunction(WebInspector.JSHeapSnapshot.prototype, "_calculateRetainedSizes");
+    InspectorTest.measureFunction(WebInspector.JSHeapSnapshot.prototype, "_markDetachedDOMTreeNodes");
+    InspectorTest.measureFunction(WebInspector.JSHeapSnapshot.prototype, "_markQueriableHeapObjects");
+    InspectorTest.measureFunction(WebInspector.JSHeapSnapshot.prototype, "_markPageOwnedNodes");
+    InspectorTest.measureFunction(WebInspector.JSHeapSnapshot.prototype, "_splitNodesAndContainmentEdges");
+    InspectorTest.measureFunction(WebInspector.JSHeapSnapshot.prototype, "_buildPostOrderIndex");
+    InspectorTest.measureFunction(WebInspector.JSHeapSnapshot.prototype, "_buildDominatorTree");
     InspectorTest.measureFunction(WebInspector.HeapSnapshotConstructorsDataGrid.prototype, "_aggregatesReceived");