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

Unified Diff: tools/perf/page_sets/mobile_memory.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/mobile_memory.json
diff --git a/tools/perf/page_sets/mobile_memory.json b/tools/perf/page_sets/mobile_memory.json
index 57884749e7f53388e41063676d3a4915e01d7ca6..8b89415e6f0a8dedbb7520c9416d40cad8a3cf36 100644
--- a/tools/perf/page_sets/mobile_memory.json
+++ b/tools/perf/page_sets/mobile_memory.json
@@ -3,7 +3,11 @@
"archive_data_file": "../data/mobile_memory.json",
"credentials_path": "../data/credentials.json",
"user_agent_type": "mobile",
- "reload_and_gc": [{"action": "reload", "wait_seconds": 15}, {"action": "js_collect_garbage"}],
+ "reload_and_gc": [
+ {"action": "reload"},
+ {"action": "wait", "condition": "seconds", "seconds": 15},
tonyg 2013/08/21 00:04:33 Another seconds vs duration question
+ {"action": "js_collect_garbage"}
+ ],
"stress_memory": { "action": "reload_and_gc", "repeat": 3 },
"pages": [
{

Powered by Google App Engine
This is Rietveld 408576698