run-perf-tests should ignore whitespace lines when snarfing test output
authorggaren@apple.com <ggaren@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 23 May 2014 22:37:19 +0000 (22:37 +0000)
committerggaren@apple.com <ggaren@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 23 May 2014 22:37:19 +0000 (22:37 +0000)
https://bugs.webkit.org/show_bug.cgi?id=133238

Reviewed by Ryosuke Niwa.

Required for PerformanceTests/Canvas/terrain.html.

* Scripts/webkitpy/performance_tests/perftest.py:
(PerfTest._run_with_driver):
(PerfTest):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@169286 268f45cc-cd09-0410-ab3c-d52691b4dbfc

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

index 62776ab..4930ad3 100644 (file)
@@ -1,3 +1,16 @@
+2014-05-23  Geoffrey Garen  <ggaren@apple.com>
+
+        run-perf-tests should ignore whitespace lines when snarfing test output
+        https://bugs.webkit.org/show_bug.cgi?id=133238
+
+        Reviewed by Ryosuke Niwa.
+
+        Required for PerformanceTests/Canvas/terrain.html.
+
+        * Scripts/webkitpy/performance_tests/perftest.py:
+        (PerfTest._run_with_driver):
+        (PerfTest):
+
 2014-05-23  Michael Saboff  <msaboff@apple.com>
 
         REGRESSION (r169241) Crash running run-jsc-stress-tests on EFL ARM
 2014-05-23  Michael Saboff  <msaboff@apple.com>
 
         REGRESSION (r169241) Crash running run-jsc-stress-tests on EFL ARM
index 5a2f043..3b2b90c 100644 (file)
@@ -189,7 +189,7 @@ class PerfTest(object):
 
             metric_match = self._metrics_regex.match(line)
             if not metric_match:
 
             metric_match = self._metrics_regex.match(line)
             if not metric_match:
-                _log.error('ERROR: ' + line)
+                _log.error('ERROR: [' + line + ']')
                 return False
 
             metric = self._ensure_metrics(metric_match.group('metric'), metric_match.group('subtest'), metric_match.group('unit'), metric_match.group('aggregator'))
                 return False
 
             metric = self._ensure_metrics(metric_match.group('metric'), metric_match.group('subtest'), metric_match.group('unit'), metric_match.group('aggregator'))
@@ -242,6 +242,7 @@ class PerfTest(object):
         return False
 
     _lines_to_ignore_in_parser_result = [
         return False
 
     _lines_to_ignore_in_parser_result = [
+        re.compile("^\s+$"),
         # Following are for handle existing test like Dromaeo
         re.compile(re.escape("""main frame - has 1 onunload handler(s)""")),
         re.compile(re.escape("""frame "<!--framePath //<!--frame0-->-->" - has 1 onunload handler(s)""")),
         # Following are for handle existing test like Dromaeo
         re.compile(re.escape("""main frame - has 1 onunload handler(s)""")),
         re.compile(re.escape("""frame "<!--framePath //<!--frame0-->-->" - has 1 onunload handler(s)""")),