Index: tools/telemetry/telemetry/page/page_runner.py |
diff --git a/tools/telemetry/telemetry/page/page_runner.py b/tools/telemetry/telemetry/page/page_runner.py |
index 4d86f62b4ef7d8b059f13463294634c3641d1848..b9091c11ae98bba495a9083ec38334ee7aef7cd2 100644 |
--- a/tools/telemetry/telemetry/page/page_runner.py |
+++ b/tools/telemetry/telemetry/page/page_runner.py |
@@ -211,6 +211,11 @@ def _PrepareAndRunPage(test, page_set, expectations, finder_options, |
tries = 3 |
while tries: |
try: |
+ if test.AlwaysRestartBrowserForEachRun(): |
+ # If we are restarting the browser for each page customize the per page |
+ # options just before starting the browser |
jeremy
2013/12/15 13:03:36
nit: indent comment 2 spaces.
aberent
2013/12/16 17:07:43
Done.
|
+ test.CustomizeBrowserOptionsForPage(page, |
+ possible_browser.finder_options) |
state.StartBrowser(test, page_set, page, possible_browser, |
credentials_path, page.archive_path) |
@@ -298,8 +303,12 @@ def Run(test, page_set, expectations, finder_options): |
if page_set.user_agent_type: |
browser_options.browser_user_agent_type = page_set.user_agent_type |
- for page in pages: |
- test.CustomizeBrowserOptionsForPage(page, possible_browser.finder_options) |
+ if not test.AlwaysRestartBrowserForEachRun(): |
jeremy
2013/12/15 13:03:36
Does the above if mean that we restart the browser
aberent
2013/12/16 17:07:43
Yes. Renamed to make this clearer.
|
+ # If we are restarting the browser for each page then we want to |
+ # customize the browser options for one page at a time. |
+ for page in pages: |
+ test.CustomizeBrowserOptionsForPage(page, |
+ possible_browser.finder_options) |
if finder_options.profiler: |
profiler_class = profiler_finder.FindProfiler(finder_options.profiler) |
profiler_class.CustomizeBrowserOptions(possible_browser.browser_type, |