Commit 3b06c181 authored by tonyg@chromium.org's avatar tonyg@chromium.org

[Telemetry] Don't restart the browser when the WPR archive path changes.

Simply restart WPR using the same port it was on before. The browser will still
be set up to point to that port.

I think we originally restarted the browser because we used to not explicitly
pass the ports to use to WPR. Now that those are stored in browser_backend,
they can stay stable across WPR restarts.

BUG=None
TEST=tools/perf/run_measurement --browser=android-content-shell page_cycler tools/perf/page_sets/top_25.json --page-filter="youtube|plus" --pageset-repeat=1 -v
NOTRY=True

Review URL: https://chromiumcodereview.appspot.com/15917016

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@202877 0039d316-1c4b-4281-b951-d872f2087c98
parent d0bd2d41
......@@ -105,9 +105,8 @@ class PageRunner(object):
possible_browser.options.wpr_mode = wpr_modes.WPR_REPLAY
else:
possible_browser.options.wpr_mode = wpr_modes.WPR_OFF
if last_archive_path != page.archive_path:
state.Close()
state = _RunState()
if last_archive_path != page.archive_path and state.browser:
state.browser.SetReplayArchivePath(page.archive_path)
last_archive_path = page.archive_path
if (test.discard_first_result and
not self.has_called_will_run_page_set):
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment