Changeset 243597 in webkit

Timestamp:

Mar 27, 2019, 6:47:29 PM (7 years ago)

Author:

sbarati@apple.com

Message:

JetStream 2 should not report time values as scores
​https://bugs.webkit.org/show_bug.cgi?id=196334

Reviewed by Yusuke Suzuki.

PerformanceTests:

• JetStream2/JetStreamDriver.js:

(toTimeValue):
(Driver.prototype.async.reportScoreToRunBenchmarkRunner):

Websites/browserbench.org:

• JetStream2.0/JetStreamDriver.js:

(toTimeValue):
(Driver.prototype.async.reportScoreToRunBenchmarkRunner):

Location:

trunk

Files:

• PerformanceTests/ChangeLog (modified) (1 diff)
• PerformanceTests/JetStream2/JetStreamDriver.js (modified) (2 diffs)
• Websites/browserbench.org/ChangeLog (modified) (1 diff)
• Websites/browserbench.org/JetStream2.0/JetStreamDriver.js (modified) (2 diffs)

trunk/PerformanceTests/ChangeLog

@@ +1 @@
+2019-03-27  Saam Barati  <sbarati@apple.com>
+
+        JetStream 2 should not report time values as scores
+        https://bugs.webkit.org/show_bug.cgi?id=196334
+
+        Reviewed by Yusuke Suzuki.
+
+        * JetStream2/JetStreamDriver.js:
+        (toTimeValue):
+        (Driver.prototype.async.reportScoreToRunBenchmarkRunner):
+
 2019-03-27  Saam barati  <sbarati@apple.com>
 

trunk/PerformanceTests/JetStream2/JetStreamDriver.js

@@ -83 +83 @@
 }
 
+function toTimeValue(score) {
+    return 5000 / score;
+}
+
 function updateUI() {
     return new Promise((resolve) => {
@@ -320 +324 @@
             const subTimes = benchmark.subTimes();
             for (const name in subTimes) {
-                subResults[name] = {"metrics": {"Time": {"current": [subTimes[name]]}}};
+                subResults[name] = {"metrics": {"Time": {"current": [toTimeValue(subTimes[name])]}}};
             }
             results[benchmark.name] = {

trunk/Websites/browserbench.org/ChangeLog

@@ +1 @@
+2019-03-27  Saam Barati  <sbarati@apple.com>
+
+        JetStream 2 should not report time values as scores
+        https://bugs.webkit.org/show_bug.cgi?id=196334
+
+        Reviewed by Yusuke Suzuki.
+
+        * JetStream2.0/JetStreamDriver.js:
+        (toTimeValue):
+        (Driver.prototype.async.reportScoreToRunBenchmarkRunner):
+
 2019-03-27  Saam barati  <sbarati@apple.com>
 

trunk/Websites/browserbench.org/JetStream2.0/JetStreamDriver.js

@@ -83 +83 @@
 }
 
+function toTimeValue(score) {
+    return 5000 / score;
+}
+
 function updateUI() {
     return new Promise((resolve) => {
@@ -320 +324 @@
             const subTimes = benchmark.subTimes();
             for (const name in subTimes) {
-                subResults[name] = {"metrics": {"Time": {"current": [subTimes[name]]}}};
+                subResults[name] = {"metrics": {"Time": {"current": [toTimeValue(subTimes[name])]}}};
             }
             results[benchmark.name] = {