[Performance test] In perftest.py, do not include 'description' in 'results'
authorharaken@chromium.org <haraken@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 17 May 2012 10:09:59 +0000 (10:09 +0000)
committerharaken@chromium.org <haraken@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 17 May 2012 10:09:59 +0000 (10:09 +0000)
https://bugs.webkit.org/show_bug.cgi?id=86696

Reviewed by Csaba Osztrogon√°.

'results' in perftest.py should not include any data that is not expected
by the perf bot server. We do not need to send 'description' to the perf bot
server since 'description' is always the same.

Thus this patch just removes 'description' from 'results'.

* Scripts/webkitpy/performance_tests/perftest.py:
(PerfTest.parse_output):
(PerfTest.output_statistics):
(ChromiumStylePerfTest.parse_output):
(PageLoadingPerfTest.run):

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

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

index 0c5dc72..2f54e95 100644 (file)
@@ -1,5 +1,24 @@
 2012-05-17  Kentaro Hara  <haraken@chromium.org>
 
+        [Performance test] In perftest.py, do not include 'description' in 'results'
+        https://bugs.webkit.org/show_bug.cgi?id=86696
+
+        Reviewed by Csaba Osztrogon√°.
+
+        'results' in perftest.py should not include any data that is not expected
+        by the perf bot server. We do not need to send 'description' to the perf bot
+        server since 'description' is always the same.
+
+        Thus this patch just removes 'description' from 'results'.
+
+        * Scripts/webkitpy/performance_tests/perftest.py:
+        (PerfTest.parse_output):
+        (PerfTest.output_statistics):
+        (ChromiumStylePerfTest.parse_output):
+        (PageLoadingPerfTest.run):
+
+2012-05-17  Kentaro Hara  <haraken@chromium.org>
+
         [Performance test] Always fill results['description'] in perftest.py
         https://bugs.webkit.org/show_bug.cgi?id=86696
 
index b8c02ed..a81c478 100644 (file)
@@ -119,18 +119,17 @@ class PerfTest(object):
         if test_failed or set(self._statistics_keys) != set(results.keys()):
             return None
 
-        results['description'] = description_string
         results['unit'] = unit
 
         test_name = re.sub(r'\.\w+$', '', self._test_name)
-        self.output_statistics(test_name, results)
+        self.output_statistics(test_name, results, description_string)
 
         return {test_name: results}
 
-    def output_statistics(self, test_name, results):
+    def output_statistics(self, test_name, results, description_string):
         unit = results['unit']
-        if results['description']:
-            _log.info('DESCRIPTION: %s' % results['description'])
+        if description_string:
+            _log.info('DESCRIPTION: %s' % description_string)
         _log.info('RESULT %s= %s %s' % (test_name.replace('/', ': '), results['avg'], unit))
         _log.info(', '.join(['%s= %s %s' % (key, results[key], unit) for key in self._statistics_keys[1:]]))
 
@@ -154,7 +153,6 @@ class ChromiumStylePerfTest(PerfTest):
             elif not len(line) == 0:
                 test_failed = True
                 _log.error(line)
-        results['description'] = ''
         return results if results and not test_failed else None
 
 
@@ -191,9 +189,8 @@ class PageLoadingPerfTest(PerfTest):
             'max': max(test_times),
             'median': test_times[middle] if len(test_times) % 2 else (test_times[middle - 1] + test_times[middle]) / 2,
             'stdev': math.sqrt(squareSum),
-            'description': '',
             'unit': 'ms'}
-        self.output_statistics(self.test_name(), results)
+        self.output_statistics(self.test_name(), results, '')
         return {self.test_name(): results}