Title: [211202] trunk/Tools
Revision
211202
Author
[email protected]
Date
2017-01-25 20:55:32 -0800 (Wed, 25 Jan 2017)

Log Message

run-benchmark nests MotionMark results by the suite name twice
https://bugs.webkit.org/show_bug.cgi?id=164948

Reviewed by Said Abou-Hallawa.

Use a single level MotionMark now that this is the name of the benchmark,
and we no longer have tests that aren't part of the MotionMark suite.

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

Modified Paths

Diff

Modified: trunk/Tools/ChangeLog (211201 => 211202)


--- trunk/Tools/ChangeLog	2017-01-26 04:50:01 UTC (rev 211201)
+++ trunk/Tools/ChangeLog	2017-01-26 04:55:32 UTC (rev 211202)
@@ -1,3 +1,15 @@
+2017-01-25  Ryosuke Niwa  <[email protected]>
+
+        run-benchmark nests MotionMark results by the suite name twice
+        https://bugs.webkit.org/show_bug.cgi?id=164948
+
+        Reviewed by Said Abou-Hallawa.
+
+        Use a single level MotionMark now that this is the name of the benchmark,
+        and we no longer have tests that aren't part of the MotionMark suite.
+
+        * Scripts/webkitpy/benchmark_runner/data/patches/MotionMark.patch:
+
 2017-01-25  Ryan Haddad  <[email protected]>
 
         Unreviewed, rolling out r211193.

Modified: trunk/Tools/Scripts/webkitpy/benchmark_runner/data/patches/MotionMark.patch (211201 => 211202)


--- trunk/Tools/Scripts/webkitpy/benchmark_runner/data/patches/MotionMark.patch	2017-01-26 04:50:01 UTC (rev 211201)
+++ trunk/Tools/Scripts/webkitpy/benchmark_runner/data/patches/MotionMark.patch	2017-01-26 04:55:32 UTC (rev 211202)
@@ -37,11 +37,11 @@
 +    {
 +        var results = this.results.results[0];
 +        var iterationReports = this._computeIterationReports(results);
-+        var debugOutput = {
++        iterationReports['debugOutput'] = {
 +            options: benchmarkRunnerClient.results.options,
 +            data: benchmarkRunnerClient.results.data,
 +        }
-+        return {"MotionMark": {"metrics" : {"Score" : ["Geometric"]}, "tests" : iterationReports }, "debugOutput": debugOutput};
++        return iterationReports;
 +    },
 +
 +    didFinishLastIteration: function()
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to