Changeset 120823 in webkit

Timestamp:

Jun 20, 2012 8:09:19 AM (12 years ago)

Author:

tomz@codeaurora.org

Message:

Import themaninblue.com/experiment/AnimationBenchmark/ as performance tests
​https://bugs.webkit.org/show_bug.cgi?id=78789

Reviewed by Ryosuke Niwa.

PerformanceTests:

Added PerfTestRunner fixture around it.

• Animation/balls.html: Added.

Tools:

Updated parser to include fps as a valid unit.

• Scripts/webkitpy/performance_tests/perftest.py:

(PerfTest):
(PerfTest.parse_output):

Location:

trunk

Files:

• PerformanceTests/Animation (added)
• PerformanceTests/Animation/balls.html (added)
• PerformanceTests/ChangeLog (modified) (1 diff)
• Tools/ChangeLog (modified) (1 diff)
• Tools/Scripts/webkitpy/performance_tests/perftest.py (modified) (2 diffs)

trunk/PerformanceTests/ChangeLog

@@ +1 @@
+2012-06-20  Tom Zakrajsek  <tomz@codeaurora.org>
+
+        Import themaninblue.com/experiment/AnimationBenchmark/ as performance tests
+        https://bugs.webkit.org/show_bug.cgi?id=78789
+
+        Reviewed by Ryosuke Niwa.
+
+        Added PerfTestRunner fixture around it.
+
+        * Animation/balls.html: Added.
+
 2012-06-18  Ilya Tikhonovsky  <loislo@chromium.org>
 

trunk/Tools/ChangeLog

@@ +1 @@
+2012-06-20  Tom Zakrajsek  <tomz@codeaurora.org>
+
+        Import themaninblue.com/experiment/AnimationBenchmark/ as performance tests
+        https://bugs.webkit.org/show_bug.cgi?id=78789
+
+        Reviewed by Ryosuke Niwa.
+
+        Updated parser to include fps as a valid unit.
+
+        * Scripts/webkitpy/performance_tests/perftest.py:
+        (PerfTest):
+        (PerfTest.parse_output):
+
 2012-06-20  Balazs Ankes  <bank@inf.u-szeged.hu>
 

trunk/Tools/Scripts/webkitpy/performance_tests/perftest.py

@@ -96 +96 @@
         re.compile(r'^Ignoring warm-up '),
         re.compile(r'^Info:'),
-        re.compile(r'^\d+(.\d+)?(\s*(runs\/s|ms))?$'),
+        re.compile(r'^\d+(.\d+)?(\s*(runs\/s|ms|fps))?$'),
         # Following are for handle existing test like Dromaeo
         re.compile(re.escape("""main frame - has 1 onunload handler(s)""")),
@@ -136 +136 @@
             if not self._should_ignore_line_in_parser_test_result(line):
                 test_failed = True
-                _log.error(line)
+                _log.error("PerfTest can't parse line [" + line + "]")
 
         if test_failed or set(self._statistics_keys) != set(results.keys()):