diff options
| author | dominikg@chromium.org <dominikg@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-02-07 17:41:07 +0000 |
|---|---|---|
| committer | dominikg@chromium.org <dominikg@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-02-07 17:41:07 +0000 |
| commit | e8fbdc66fff2a058b67e210e8ba6af5cc3a1bef1 (patch) | |
| tree | 4f56107cc25d50407591f0e8a15dcac996149aac /tools | |
| parent | 95de0e19e9cef0bc13106967af9d536b8d26fdaf (diff) | |
| download | chromium_src-e8fbdc66fff2a058b67e210e8ba6af5cc3a1bef1.zip chromium_src-e8fbdc66fff2a058b67e210e8ba6af5cc3a1bef1.tar.gz chromium_src-e8fbdc66fff2a058b67e210e8ba6af5cc3a1bef1.tar.bz2 | |
Re-record mobile-news page for key_mobile_sites page set.
We had some issues with Telemetry scrolling the page before it is ready to
scroll. The page now has a variable (NEWS_telemetryReady) to indicate that it's
ready to be scrolled.
This patch contains the recording of the new version of the page and updates the
navigate steps accordingly.
BUG=248928
Review URL: https://codereview.chromium.org/157483002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@249716 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'tools')
| -rw-r--r-- | tools/perf/page_sets/data/key_mobile_sites.json | 4 | ||||
| -rw-r--r-- | tools/perf/page_sets/data/key_mobile_sites_006.wpr.sha1 | 1 | ||||
| -rw-r--r-- | tools/perf/page_sets/key_mobile_sites.json | 6 |
3 files changed, 9 insertions, 2 deletions
diff --git a/tools/perf/page_sets/data/key_mobile_sites.json b/tools/perf/page_sets/data/key_mobile_sites.json index 0a5a933..b83a651 100644 --- a/tools/perf/page_sets/data/key_mobile_sites.json +++ b/tools/perf/page_sets/data/key_mobile_sites.json @@ -2,7 +2,6 @@ "description": "Describes the Web Page Replay archives for a page set. Don't edit by hand! Use record_wpr for updating.", "archives": { "key_mobile_sites_005.wpr": [ - "http://mobile-news.sandbox.google.com/news/pt1", "http://mobile-news.sandbox.google.com/news/pt0" ], "key_mobile_sites_001.wpr": [ @@ -14,6 +13,9 @@ "key_mobile_sites_003.wpr": [ "https://facebook.com/barackobama" ], + "key_mobile_sites_006.wpr": [ + "http://mobile-news.sandbox.google.com/news/pt1" + ], "key_mobile_sites_000.wpr": [ "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/", "http://nytimes.com/", diff --git a/tools/perf/page_sets/data/key_mobile_sites_006.wpr.sha1 b/tools/perf/page_sets/data/key_mobile_sites_006.wpr.sha1 new file mode 100644 index 0000000..b725428 --- /dev/null +++ b/tools/perf/page_sets/data/key_mobile_sites_006.wpr.sha1 @@ -0,0 +1 @@ +8ea95eb86c54231730fb4871c116893bda602d8a
\ No newline at end of file diff --git a/tools/perf/page_sets/key_mobile_sites.json b/tools/perf/page_sets/key_mobile_sites.json index b89bc82..6435e8a 100644 --- a/tools/perf/page_sets/key_mobile_sites.json +++ b/tools/perf/page_sets/key_mobile_sites.json @@ -265,7 +265,11 @@ }, { "url": "http://mobile-news.sandbox.google.com/news/pt1", - "why": "Google News: accelerated scrolling version" + "why": "Google News: accelerated scrolling version", + "navigate_steps": [ + { "action": "navigate" }, + { "action": "wait", "javascript": "typeof NEWS_telemetryReady !== 'undefined' && NEWS_telemetryReady == true" } + ] }, { "url": "http://mobile-news.sandbox.google.com/news/pt0", |