Changeset 96209 in webkit

Timestamp:

Sep 28, 2011 2:30:27 AM (13 years ago)

Author:

loislo@chromium.org

Message:

Web Inspector: UI performance test for Inspector start-up time is required.
​https://bugs.webkit.org/show_bug.cgi?id=68986

Reviewed by Yury Semikhatsky.

• inspector/performance/resources/inspector-startup-time.html: Added.
• inspector/performance/resources/performance-test.js:

(initialize_TimeTracker.InspectorTest.runPerformanceTest.Timer.prototype._dump):
(initialize_TimeTracker.InspectorTest.runPerformanceTest):

Location:

trunk/LayoutTests

Files:

• ChangeLog (modified) (1 diff)
• inspector/performance/resources/inspector-startup-time.html (added)
• inspector/performance/resources/performance-test.js (modified) (1 diff)

trunk/LayoutTests/ChangeLog

@@ +1 @@
+2011-09-28  Ilya Tikhonovsky  <loislo@chromium.org>
+
+        Web Inspector: UI performance test for Inspector start-up time is required.
+        https://bugs.webkit.org/show_bug.cgi?id=68986
+
+        Reviewed by Yury Semikhatsky.
+
+        * inspector/performance/resources/inspector-startup-time.html: Added.
+        * inspector/performance/resources/performance-test.js:
+        (initialize_TimeTracker.InspectorTest.runPerformanceTest.Timer.prototype._dump):
+        (initialize_TimeTracker.InspectorTest.runPerformanceTest):
+
 2011-09-28  Kentaro Hara  <haraken@chromium.org>
 

trunk/LayoutTests/inspector/performance/resources/performance-test.js

@@ -68 +68 @@
         {
             for (var testName in this._times)
-                this._dumpTestStats(testName, this._times[testName]);
+                InspectorTest.dumpTestStats(testName, this._times[testName]);
 
             var url = WebInspector.mainResource._documentURL;
             var regExp = /([^\/]+)\.html/;
             var matches = regExp.exec(url);
-            this._dumpTestStats("heap-delta-kb-" + matches[1], this._heapSizeDeltas, 1024);
+            InspectorTest.dumpTestStats("heap-delta-kb-" + matches[1], this._heapSizeDeltas, 1024);
         },
 
-        _dumpTestStats: function(testName, samples, divider)
-        {
-            divider = divider || 1;
-            var stripNResults = Math.floor(samples.length / 10);
-            samples.sort(function(a, b) { return a - b; });
-            var sum = 0;
-            for (var i = stripNResults; i < samples.length - stripNResults; ++i)
-                sum += samples[i];
-            InspectorTest.addResult("* " + testName + ": " + Math.floor(sum / (samples.length - stripNResults * 2) / divider));
-            InspectorTest.addResult(testName + " min/max/count: " + Math.floor(samples[0] / divider) + "/" + Math.floor(samples[samples.length-1] / divider) + "/" + samples.length);
-        }
     }
 
     var timer = new Timer(perfTest, callback);
     timer._runTest();
+}
+
+InspectorTest.dumpTestStats = function(testName, samples, divider)
+{
+    divider = divider || 1;
+    var stripNResults = Math.floor(samples.length / 10);
+    samples.sort(function(a, b) { return a - b; });
+    var sum = 0;
+    for (var i = stripNResults; i < samples.length - stripNResults; ++i)
+        sum += samples[i];
+    InspectorTest.addResult("* " + testName + ": " + Math.floor(sum / (samples.length - stripNResults * 2) / divider));
+    InspectorTest.addResult(testName + " min/max/count: " + Math.floor(samples[0] / divider) + "/" + Math.floor(samples[samples.length-1] / divider) + "/" + samples.length);
 }