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

Side by Side Diff: tools/perf/page_sets/key_mobile_sites.json

Issue 23957002: Telemetry: Fix facebook.com on key_mobile_sites. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 7 years, 3 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 unified diff | Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 { 1 {
2 "description": "Key mobile sites", 2 "description": "Key mobile sites",
3 "archive_data_file": "../data/key_mobile_sites.json", 3 "archive_data_file": "../data/key_mobile_sites.json",
4 "credentials_path": "../data/credentials.json", 4 "credentials_path": "../data/credentials.json",
5 "user_agent_type": "mobile", 5 "user_agent_type": "mobile",
6 "smoothness": { "action": "scroll" }, 6 "smoothness": { "action": "scroll" },
7 "pages": [ 7 "pages": [
8 { 8 {
9 "url": "http://www.androidpolice.com/2012/10/03/rumor-evidence-mounts-that -an-lg-optimus-g-nexus-is-coming-along-with-a-nexus-phone-certification-program/ ", 9 "url": "http://www.androidpolice.com/2012/10/03/rumor-evidence-mounts-that -an-lg-optimus-g-nexus-is-coming-along-with-a-nexus-phone-certification-program/ ",
10 "why": "crbug.com/242544" 10 "why": "crbug.com/242544"
(...skipping 52 matching lines...) Expand 10 before | Expand all | Expand 10 after
63 "smoothness": { 63 "smoothness": {
64 "action": "scroll", 64 "action": "scroll",
65 "scroll_is_infinite": true 65 "scroll_is_infinite": true
66 } 66 }
67 }, 67 },
68 { 68 {
69 "url": "https://facebook.com/barackobama", 69 "url": "https://facebook.com/barackobama",
70 "why": "#1 (Alexa global)", 70 "why": "#1 (Alexa global)",
71 "navigate_steps": [ 71 "navigate_steps": [
72 { "action": "navigate" }, 72 { "action": "navigate" },
73 { "action": "wait", "javascript": "document.getElementById('u_0_c') !== null" } 73 { "action": "wait", "javascript": "document.getElementById('u_0_c') !== null && document.body.scrollHeight > window.innerHeight" }
tonyg 2013/09/04 17:27:19 Does this work for all screen sizes? I'm worried t
Dominik Grewe 2013/09/04 20:05:52 It should work if there's anything to scroll :) An
tonyg 2013/09/04 20:29:43 You are right. The test will fail anyway if there
74 ] 74 ]
75 }, 75 },
76 { 76 {
77 "url": "http://m.youtube.com/watch?v=9hBpF_Zj4OA", 77 "url": "http://m.youtube.com/watch?v=9hBpF_Zj4OA",
78 "why": "#3 (Alexa global)", 78 "why": "#3 (Alexa global)",
79 "navigate_steps": [ 79 "navigate_steps": [
80 { "action": "navigate"}, 80 { "action": "navigate"},
81 { "action": "wait", "javascript": "document.getElementById('paginatortar get') !== null"} 81 { "action": "wait", "javascript": "document.getElementById('paginatortar get') !== null"}
82 ] 82 ]
83 }, 83 },
(...skipping 141 matching lines...) Expand 10 before | Expand all | Expand 10 after
225 "remaining_scroll_distance_function": "function() { return Math.max(0, 1 250 + document.getElementById('element-19').contentDocument.getElementById('elem ent-22').getBoundingClientRect().top); }" 225 "remaining_scroll_distance_function": "function() { return Math.max(0, 1 250 + document.getElementById('element-19').contentDocument.getElementById('elem ent-22').getBoundingClientRect().top); }"
226 } 226 }
227 }, 227 },
228 { 228 {
229 "disabled": "Doesn't scroll; crbug.com/249736", 229 "disabled": "Doesn't scroll; crbug.com/249736",
230 "url": "http://forecast.io", 230 "url": "http://forecast.io",
231 "why": "crbug.com/231413" 231 "why": "crbug.com/231413"
232 } 232 }
233 ] 233 ]
234 } 234 }
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698