Title: [139728] trunk/PerformanceTests
Revision
139728
Author
domin...@chromium.org
Date
2013-01-15 01:42:25 -0800 (Tue, 15 Jan 2013)

Log Message

Allow zero values as results from the runFunction.
https://bugs.webkit.org/show_bug.cgi?id=102500

Reviewed by Kentaro Hara.

DOM/DOMWalk.js fails on the Chromium Win perf bot because one or
more parts of the test usually return zero at some point.

* resources/runner.js: Accept merely non-negative results.

Modified Paths

Diff

Modified: trunk/PerformanceTests/ChangeLog (139727 => 139728)


--- trunk/PerformanceTests/ChangeLog	2013-01-15 09:33:54 UTC (rev 139727)
+++ trunk/PerformanceTests/ChangeLog	2013-01-15 09:42:25 UTC (rev 139728)
@@ -1,3 +1,15 @@
+2013-01-15  Dominic Cooney  <domin...@chromium.org>
+
+        Allow zero values as results from the runFunction.
+        https://bugs.webkit.org/show_bug.cgi?id=102500
+
+        Reviewed by Kentaro Hara.
+
+        DOM/DOMWalk.js fails on the Chromium Win perf bot because one or
+        more parts of the test usually return zero at some point.
+
+        * resources/runner.js: Accept merely non-negative results.
+
 2012-12-28  Yury Semikhatsky  <yu...@chromium.org>
 
         Web Inspector: rename distanceToWindow to distance

Modified: trunk/PerformanceTests/resources/runner.js (139727 => 139728)


--- trunk/PerformanceTests/resources/runner.js	2013-01-15 09:33:54 UTC (rev 139727)
+++ trunk/PerformanceTests/resources/runner.js	2013-01-15 09:42:25 UTC (rev 139728)
@@ -257,8 +257,8 @@
         var end = PerfTestRunner.now();
 
         if (returnValue - 0 === returnValue) {
-            if (returnValue <= 0)
-                PerfTestRunner.log("runFunction returned a non-positive value: " + returnValue);
+            if (returnValue < 0)
+                PerfTestRunner.log("runFunction returned a negative value: " + returnValue);
             return returnValue;
         }
 
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to