Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1657)

Unified Diff: tools/perf/page_sets/2012Q3.json

Issue 22883011: Removing Page.WaitToLoad and update all pagesets to use explicit wait actions (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: . Created 7 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: tools/perf/page_sets/2012Q3.json
diff --git a/tools/perf/page_sets/2012Q3.json b/tools/perf/page_sets/2012Q3.json
index 151a1d650d9141b164dc518ecae381ac5a520223..47a938dffdce15526206c278c9a04a678310545e 100644
--- a/tools/perf/page_sets/2012Q3.json
+++ b/tools/perf/page_sets/2012Q3.json
@@ -3,7 +3,11 @@
"archive_data_file": "../data/2012Q3.json",
"credentials_path": "../data/credentials.json",
"smoothness": { "action": "scroll" },
- "reload_and_gc": [{"action": "reload", "wait_seconds": 1}, {"action": "js_collect_garbage"}],
+ "reload_and_gc": [
+ {"action": "reload"},
+ {"action": "wait", "condition": "seconds", "seconds": 1},
tonyg 2013/08/21 00:04:33 Or should this be "condition": "duration"?
+ {"action": "js_collect_garbage"}
+ ],
"stress_memory": { "action": "reload_and_gc", "repeat": 3 },
"pages": [
{ "url": "http://www.facebook.com/barackobama" },

Powered by Google App Engine
This is Rietveld 408576698