[Telemetry] Add a page set option to use all cold page loads.

tools/telemetry/telemetry/page/page_runner.py

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 8f5aec0924f4edc3d07eb61f7d60ac47ae603188..b39571c70a6d66dd7755326a5adeb3f791456663 100644
--- a/tools/telemetry/telemetry/page/page_runner.py
+++ b/tools/telemetry/telemetry/page/page_runner.py
@@ -234,6 +234,8 @@ class PageRunner(object):
       self._PreparePage(page, tab, page_state, test, results)
       test.Run(options, page, tab, results)
       util.CloseConnections(tab)

[nduca, 2013/06/20 19:54:15]

and this for that matter shoudl move into page state or clean up page

[tonyg, 2013/06/21 22:36:18]

Agreed. I decided not to touch CloseConnections in this patch though. I want to
keep it self-contained.

+      if options.cold_page_loads:

[nduca, 2013/06/20 19:54:15]

lets make this part of clean up page

also, to ask a question.... what semantic of cold_page_loads is implied if we
load a site in two tabs? Should thiss be part of prepare, rather than after? In
the world of multi tab, you wouldn't clean up page until after the page was
completely gone -- meaning going to google.com in two tabs would hit the cache
on the second tab, in the case that you clear cache after the test runs....

[tonyg, 2013/06/21 22:36:18]

Really good point. I put this into PreparePage.

+        tab.ClearCache()
     except page_test.Failure:
       logging.warning('%s:\n%s', page.url, traceback.format_exc())
       results.AddFailure(page, sys.exc_info())