Add '__init__.py' and fix known bug in benchmark runner.
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 28 Apr 2015 01:06:14 +0000 (01:06 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 28 Apr 2015 01:06:14 +0000 (01:06 +0000)
https://bugs.webkit.org/show_bug.cgi?id=144296

Patch by Dewei Zhu <dewei_zhu@apple.com> on 2015-04-27
Reviewed by Ryosuke Niwa

* Scripts/webkitpy/benchmark_runner/__init__.py: Make python to recognize this module.
* Scripts/webkitpy/benchmark_runner/benchmark_builder/__init__.py: Make python to recognize this module.
* Scripts/webkitpy/benchmark_runner/benchmark_runner.py:
(BenchmarkRunner.__init__):
(BenchmarkRunner.execute): Fix logic bug that '--output-file' option doesn't work, and empty output file bug.
* Scripts/webkitpy/benchmark_runner/browser_driver/__init__.py: Make python to recognize this module.
* Scripts/webkitpy/benchmark_runner/http_server_driver/__init__.py: Make python to recognize this module.

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

Tools/ChangeLog
Tools/Scripts/webkitpy/benchmark_runner/__init__.py [new file with mode: 0644]
Tools/Scripts/webkitpy/benchmark_runner/benchmark_builder/__init__.py [new file with mode: 0644]
Tools/Scripts/webkitpy/benchmark_runner/benchmark_runner.py
Tools/Scripts/webkitpy/benchmark_runner/browser_driver/__init__.py [new file with mode: 0644]
Tools/Scripts/webkitpy/benchmark_runner/http_server_driver/__init__.py [new file with mode: 0644]

index 0ef0c9bf055f8b5c100b8f5c6b4539d03f3ad2d8..bc6e10c97c80a1a2eef80eae3435cbfbd2c432d0 100644 (file)
@@ -1,3 +1,18 @@
+2015-04-27  Dewei Zhu  <dewei_zhu@apple.com>
+
+        Add '__init__.py' and fix known bug in benchmark runner.
+        https://bugs.webkit.org/show_bug.cgi?id=144296
+
+        Reviewed by Ryosuke Niwa
+
+        * Scripts/webkitpy/benchmark_runner/__init__.py: Make python to recognize this module.
+        * Scripts/webkitpy/benchmark_runner/benchmark_builder/__init__.py: Make python to recognize this module.
+        * Scripts/webkitpy/benchmark_runner/benchmark_runner.py:
+        (BenchmarkRunner.__init__):
+        (BenchmarkRunner.execute): Fix logic bug that '--output-file' option doesn't work, and empty output file bug.
+        * Scripts/webkitpy/benchmark_runner/browser_driver/__init__.py: Make python to recognize this module.
+        * Scripts/webkitpy/benchmark_runner/http_server_driver/__init__.py: Make python to recognize this module.
+
 2015-04-27  Alexey Proskuryakov  <ap@apple.com>
 
         More build fix.
diff --git a/Tools/Scripts/webkitpy/benchmark_runner/__init__.py b/Tools/Scripts/webkitpy/benchmark_runner/__init__.py
new file mode 100644 (file)
index 0000000..b376bf2
--- /dev/null
@@ -0,0 +1,13 @@
+# Required for Python to search this directory for module files
+
+# Keep this file free of any code or import statements that could
+# cause either an error to occur or a log message to be logged.
+# This ensures that calling code can import initialization code from
+# webkitpy before any errors or log messages due to code in this file.
+# Initialization code can include things like version-checking code and
+# logging configuration code.
+#
+# We do not execute any version-checking code or logging configuration
+# code in this file so that callers can opt-in as they want.  This also
+# allows different callers to choose different initialization code,
+# as necessary.
diff --git a/Tools/Scripts/webkitpy/benchmark_runner/benchmark_builder/__init__.py b/Tools/Scripts/webkitpy/benchmark_runner/benchmark_builder/__init__.py
new file mode 100644 (file)
index 0000000..b376bf2
--- /dev/null
@@ -0,0 +1,13 @@
+# Required for Python to search this directory for module files
+
+# Keep this file free of any code or import statements that could
+# cause either an error to occur or a log message to be logged.
+# This ensures that calling code can import initialization code from
+# webkitpy before any errors or log messages due to code in this file.
+# Initialization code can include things like version-checking code and
+# logging configuration code.
+#
+# We do not execute any version-checking code or logging configuration
+# code in this file so that callers can opt-in as they want.  This also
+# allows different callers to choose different initialization code,
+# as necessary.
index 8f75d45f91c24abc76b805a816491b7368337bd3..b4b9d19af0ab012beac0267d555dfccf239cdc47 100644 (file)
@@ -32,7 +32,7 @@ class BenchmarkRunner(object):
                 self.httpServerDriver = HTTPServerDriverFactory.create([self.plan['http_server_driver']])
                 self.benchmarks = self.plan['benchmarks']
                 self.buildDir = os.path.abspath(buildDir)
-                self.outputFile = outputFile if outputFile else 'benchmark.result'
+                self.outputFile = outputFile
         except IOError:
             _log.error('Can not open plan file: %s' % planFile)
         except ValueError:
@@ -56,6 +56,7 @@ class BenchmarkRunner(object):
                     with timeout(benchmark['timeout']):
                         result = json.loads(self.httpServerDriver.fetchResult())
                         assert(result)
+                        results.append(result)
                 except:
                     _log.error('No result. Something went wrong. Will skip current benchmark.')
                     self.browserDriver.closeBrowsers()
@@ -64,7 +65,7 @@ class BenchmarkRunner(object):
                     self.browserDriver.closeBrowsers()
                     _log.info('End of %d iteration of current benchmark' % (x + 1))
             results = self.wrap(results)
-            self.dump(results, benchmark['output_file'] if benchmark['output_file'] else self.outputFile)
+            self.dump(results, self.outputFile if self.outputFile else benchmark['output_file'])
             benchmarkBuilder.clean()
 
     @classmethod
diff --git a/Tools/Scripts/webkitpy/benchmark_runner/browser_driver/__init__.py b/Tools/Scripts/webkitpy/benchmark_runner/browser_driver/__init__.py
new file mode 100644 (file)
index 0000000..b376bf2
--- /dev/null
@@ -0,0 +1,13 @@
+# Required for Python to search this directory for module files
+
+# Keep this file free of any code or import statements that could
+# cause either an error to occur or a log message to be logged.
+# This ensures that calling code can import initialization code from
+# webkitpy before any errors or log messages due to code in this file.
+# Initialization code can include things like version-checking code and
+# logging configuration code.
+#
+# We do not execute any version-checking code or logging configuration
+# code in this file so that callers can opt-in as they want.  This also
+# allows different callers to choose different initialization code,
+# as necessary.
diff --git a/Tools/Scripts/webkitpy/benchmark_runner/http_server_driver/__init__.py b/Tools/Scripts/webkitpy/benchmark_runner/http_server_driver/__init__.py
new file mode 100644 (file)
index 0000000..b376bf2
--- /dev/null
@@ -0,0 +1,13 @@
+# Required for Python to search this directory for module files
+
+# Keep this file free of any code or import statements that could
+# cause either an error to occur or a log message to be logged.
+# This ensures that calling code can import initialization code from
+# webkitpy before any errors or log messages due to code in this file.
+# Initialization code can include things like version-checking code and
+# logging configuration code.
+#
+# We do not execute any version-checking code or logging configuration
+# code in this file so that callers can opt-in as they want.  This also
+# allows different callers to choose different initialization code,
+# as necessary.