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

Unified Diff: tools/perf/page_sets/top_10.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: Putting default navigate_steps inside page_set rather than page 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
« no previous file with comments | « tools/perf/page_sets/pica.json ('k') | tools/perf/page_sets/top_25.json » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/perf/page_sets/top_10.json
diff --git a/tools/perf/page_sets/top_10.json b/tools/perf/page_sets/top_10.json
index b14ae24b49594fa0756625325c0ae9a076c8d9a7..31f5db566607b45fffa74850d2ce5b04329d915b 100644
--- a/tools/perf/page_sets/top_10.json
+++ b/tools/perf/page_sets/top_10.json
@@ -8,26 +8,38 @@
{
"url": "https://www.google.com/#hl=en&q=barack+obama",
"why": "top google property; a google tab is often open",
- "wait_for_element_with_text": "Next"
+ "navigate_steps" : [
+ { "action": "navigate" },
+ { "action": "wait", "condition": "element", "text": "Next" }
+ ]
},
{
"url": "https://mail.google.com/mail/",
"why": "productivity, top google properties",
"credentials": "google",
- "wait_for_javascript_expression": "window.gmonkey !== undefined && document.getElementById('gb') !== null"
+ "navigate_steps" : [
+ { "action": "navigate" },
+ { "action": "wait", "javascript": "window.gmonkey !== undefined && document.getElementById('gb') !== null" }
+ ]
},
{
"url": "https://www.google.com/calendar/",
"why": "productivity, top google properties",
"credentials": "google",
- "wait_for_element_with_selector": "div[class~=\"navForward\"]",
- "post_navigate_javascript_to_execute": "(function() { var elem = document.createElement('meta');elem.name='viewport';elem.content='initial-scale=1';document.body.appendChild(elem); })();",
- "wait_seconds": 2
+ "navigate_steps" : [
+ { "action": "navigate" },
+ { "action": "javascript", "expression": "(function() { var elem = document.createElement('meta');elem.name='viewport';elem.content='initial-scale=1';document.body.appendChild(elem); })();" },
+ { "action": "wait", "seconds": 2 },
+ { "action": "wait", "condition": "element", "selector": "div[class~=\"navForward\"]" }
+ ]
},
{
"url": "http://www.youtube.com",
"why": "#3 (Alexa global)",
- "wait_seconds": 2,
+ "navigate_steps" : [
+ { "action": "navigate" },
+ { "action": "wait", "seconds": 2 }
+ ],
"credentials": "google"
},
{
@@ -35,7 +47,10 @@
"url": "http://www.facebook.com/barackobama",
"why": "top social,Public profile",
"credentials": "facebook",
- "wait_for_element_with_text": "About"
+ "navigate_steps" : [
+ { "action": "navigate" },
+ { "action": "wait", "condition": "element", "text": "About" }
+ ]
},
{
"name": "Wikipedia (1 tab)",
« no previous file with comments | « tools/perf/page_sets/pica.json ('k') | tools/perf/page_sets/top_25.json » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698