Title: [233445] trunk/Tools
Revision
233445
Author
[email protected]
Date
2018-07-02 16:17:41 -0700 (Mon, 02 Jul 2018)

Log Message

run-benchmark should include the version number of MotionMark's results
https://bugs.webkit.org/show_bug.cgi?id=187274

Reviewed by Jon Lee.

Append the version number as a part of the suite name.

* Scripts/webkitpy/benchmark_runner/data/patches/webserver/MotionMark.patch:

Modified Paths

Diff

Modified: trunk/Tools/ChangeLog (233444 => 233445)


--- trunk/Tools/ChangeLog	2018-07-02 23:15:31 UTC (rev 233444)
+++ trunk/Tools/ChangeLog	2018-07-02 23:17:41 UTC (rev 233445)
@@ -1,3 +1,14 @@
+2018-07-02  Ryosuke Niwa  <[email protected]>
+
+        run-benchmark should include the version number of MotionMark's results
+        https://bugs.webkit.org/show_bug.cgi?id=187274
+
+        Reviewed by Jon Lee.
+
+        Append the version number as a part of the suite name.
+
+        * Scripts/webkitpy/benchmark_runner/data/patches/webserver/MotionMark.patch:
+
 2018-07-02  Wenson Hsieh  <[email protected]>
 
         [WK1] editing/spelling/markers.html is failing on recent builds of macOS Mojave

Modified: trunk/Tools/Scripts/webkitpy/benchmark_runner/data/patches/webserver/MotionMark.patch (233444 => 233445)


--- trunk/Tools/Scripts/webkitpy/benchmark_runner/data/patches/webserver/MotionMark.patch	2018-07-02 23:15:31 UTC (rev 233444)
+++ trunk/Tools/Scripts/webkitpy/benchmark_runner/data/patches/webserver/MotionMark.patch	2018-07-02 23:17:41 UTC (rev 233445)
@@ -2,7 +2,7 @@
 index d45155e..b10635a 100644
 --- a/resources/runner/motionmark.js
 +++ b/resources/runner/motionmark.js
-@@ -411,6 +411,61 @@ window.benchmarkRunnerClient = {
+@@ -411,6 +411,62 @@ window.benchmarkRunnerClient = {
      didFinishLastIteration: function()
      {
          benchmarkController.showResults();
@@ -26,9 +26,10 @@
 +    _computeIterationReports: function(iterationResults)
 +    {
 +        var suitesReports = {};
++        var suffix = Strings.version ? '-' + Strings.version : '';
 +        for (var suiteName in iterationResults[Strings.json.results.tests]) {
 +            var testsReports = this._computeSuiteReports(suiteName, iterationResults[Strings.json.results.tests][suiteName]);
-+            suitesReports[suiteName] = { "metrics": { "Score": ["Geometric"] }, "tests": testsReports };
++            suitesReports[suiteName + suffix] = { "metrics": { "Score": ["Geometric"] }, "tests": testsReports };
 +        }
 +        return suitesReports;
 +    },
@@ -64,7 +65,7 @@
      }
  };
  
-@@ -652,4 +707,7 @@ window.benchmarkController = {
+@@ -652,4 +708,7 @@ window.benchmarkController = {
      }
  };
  
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to