[telemetry] Simplify the way PageRunner is called.

Before:
results = page_test_results.PageTestResults()
with page_runner.PageRunner(ps):
  possible_browser = browser_finder.FindBrowser(options)
  runner.Run(options, possible_browser, test, results)

After:
results = page_runner.Run(test, ps, options)

Turns out, PageRunner keeps very little state on its own, and stores most of its state in RunState. Using a with statement does nothing because PageRunner.Close() just passes. Therefore, we can get rid of the PageRunner object and just have page_runner.Run() directly. The only exception was has_called_will_run_page_set, which I solved by getting rid of WillRunPageSet() :(

We can also move a lot of its setup methods to RunState, since those methods delve pretty deeply into RunState anyway. There's more work that could be done here in the future.

Finally, move the results object and possible_browser creation from page_test_runner to page_runner/page_test. This will make it easier for the future test_runner to call page_runner directly. The separation of concerns is that test_runner/page_test_runner only deal with parsing the command line, and page_runner does everything else. In the future, page_test_runner can be obsoleted.

BUG=237412
TEST=./run_tests, smoothness_measurement, and page_cycler with no failures.
R=nduca@chromium.org

Review URL: https://codereview.chromium.org/16373012

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@206529 0039d316-1c4b-4281-b951-d872f2087c98

0 files changed, 0 insertions, 0 deletions