Web Inspector: Heap Snapshot test: split overal time metric into separate metrics.
authorloislo@chromium.org <loislo@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 6 Mar 2012 13:41:21 +0000 (13:41 +0000)
committerloislo@chromium.org <loislo@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 6 Mar 2012 13:41:21 +0000 (13:41 +0000)
https://bugs.webkit.org/show_bug.cgi?id=80409

Reviewed by Yury Semikhatsky.

* inspector/detailed-heapshots-smoke-test.html:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@109913 268f45cc-cd09-0410-ab3c-d52691b4dbfc

PerformanceTests/ChangeLog
PerformanceTests/inspector/detailed-heapshots-smoke-test.html

index 38bc1645f3553711977cfd4e9993ce98f7ca170e..acb85a9d7f98fbe616877ec4ff7a4dc88b44a463 100644 (file)
@@ -1,3 +1,12 @@
+2012-03-06  Ilya Tikhonovsky  <loislo@chromium.org>
+
+        Web Inspector: Heap Snapshot test: split overal time metric into separate metrics.
+        https://bugs.webkit.org/show_bug.cgi?id=80409
+
+        Reviewed by Yury Semikhatsky.
+
+        * inspector/detailed-heapshots-smoke-test.html:
+
 2012-03-05  Ilya Tikhonovsky  <loislo@chromium.org>
 
         Web Inspector: [chromium] introduce HeapSnapshot performance test.
index 79123244f8c3c4d967001a6de7d13088b80dd782..1313742058e720018c6e426d5c3a4906c63b2830 100644 (file)
@@ -10,11 +10,19 @@ function test()
     function performanceTest(timer)
     {
 
-        var showPanelTimerCookie = timer.start("panel-update");
+        var transferTimerCookie;
+        var showTimerCookie;
+        var changeViewTimerCookie;
+        var clearTimerCookie;
+
+        var allTimerCookie = timer.start("summary-snapshot-time");
+        var backendTimerCookie = timer.start("take-snapshot");
         ProfilerAgent.takeHeapSnapshot(step0);
 
         function step0()
         {
+            timer.finish(backendTimerCookie);
+            transferTimerCookie = timer.start("transfer-snapshot");
             var profiles = WebInspector.panels.profiles.getProfiles("HEAP");
             WebInspector.panels.profiles.showProfile(profiles[profiles.length - 1]);
             InspectorTest.addSniffer(WebInspector.panels.profiles, "_finishHeapSnapshot", step1);
@@ -22,6 +30,8 @@ function test()
 
         function step1(uid)
         {
+            timer.finish(transferTimerCookie);
+            showTimerCookie = timer.start("show-snapshot");
             var panel = WebInspector.panels.profiles;
             var profile = panel._profilesIdMap[panel._makeKey(uid, WebInspector.DetailedHeapshotProfileType.TypeId)];
             InspectorTest.addSniffer(profile.proxy, "_callLoadCallbacks", step2);
@@ -29,19 +39,24 @@ function test()
 
         function step2()
         {
+            timer.finish(showTimerCookie);
+            timer.finish(allTimerCookie);
+            changeViewTimerCookie = timer.start("switch-to-containment-view");
             InspectorTest.switchToView("Containment", cleanup);
         }
 
         function cleanup()
         {
-            timer.finish(showPanelTimerCookie);
+            timer.finish(changeViewTimerCookie);
+            clearTimerCookie = timer.start("clear-snapshot");
             ProfilerAgent.clearProfiles(done);
             WebInspector.panels.profiles._reset();
         }
 
         function done()
         {
-            timer.done("take-snapshot");
+            timer.finish(clearTimerCookie);
+            timer.done("panel-update");
         }
     }