Unreviewed, rolling out r128562.
authorossy@webkit.org <ossy@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 14 Sep 2012 10:40:40 +0000 (10:40 +0000)
committerossy@webkit.org <ossy@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 14 Sep 2012 10:40:40 +0000 (10:40 +0000)
http://trac.webkit.org/changeset/128562
https://bugs.webkit.org/show_bug.cgi?id=96747

It broke perf tests on Qt and Chromium (Requested by Ossy on
#webkit).

Patch by Sheriff Bot <webkit.review.bot@gmail.com> on 2012-09-14

* resources/runner.js:
(PerfTestRunner._runner):
(PerfTestRunner._perSecondRunnerIterator):

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

PerformanceTests/ChangeLog
PerformanceTests/resources/runner.js

index 32cd265..6d2446d 100644 (file)
@@ -1,3 +1,16 @@
+2012-09-14  Sheriff Bot  <webkit.review.bot@gmail.com>
+
+        Unreviewed, rolling out r128562.
+        http://trac.webkit.org/changeset/128562
+        https://bugs.webkit.org/show_bug.cgi?id=96747
+
+        It broke perf tests on Qt and Chromium (Requested by Ossy on
+        #webkit).
+
+        * resources/runner.js:
+        (PerfTestRunner._runner):
+        (PerfTestRunner._perSecondRunnerIterator):
+
 2012-09-14  Ryosuke Niwa  <rniwa@webkit.org>
 
         Use performance.webkitNow in PerfTestRunner
index 850c46d..6bf1f29 100755 (executable)
@@ -27,8 +27,6 @@ PerfTestRunner.random = Math.random = function() {
     return (randomSeed & 0xfffffff) / 0x10000000;
 };
 
-PerfTestRunner.now = window.performance && window.performance.webkitNow ? window.performance.webkitNow : Date.now;
-
 PerfTestRunner.log = function (text) {
     if (this._logLines) {
         this._logLines.push(text);
@@ -147,7 +145,7 @@ PerfTestRunner._runLoop = function () {
 }
 
 PerfTestRunner._runner = function () {
-    var start = this.now();
+    var start = Date.now();
     var totalTime = 0;
 
     for (var i = 0; i < this._loopsPerRun; ++i) {
@@ -160,7 +158,7 @@ PerfTestRunner._runner = function () {
     }
 
     // Assume totalTime can never be zero when _runFunction returns a number.
-    var time = totalTime ? totalTime : this.now() - start;
+    var time = totalTime ? totalTime : Date.now() - start;
 
     this.ignoreWarmUpAndLog(time);
     this._runLoop();
@@ -260,10 +258,10 @@ PerfTestRunner._perSecondRunner = function () {
 }
 
 PerfTestRunner._perSecondRunnerIterator = function (callsPerIteration) {
-    var startTime = this.now();
+    var startTime = Date.now();
     for (var i = 0; i < callsPerIteration; i++)
         this._test.run();
-    return this.now() - startTime;
+    return Date.now() - startTime;
 }
 
 if (window.testRunner) {