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

Unified Diff: tools/perf/page_sets/popular_urls.py

Issue 1912443002: Reimplement basic PopularUrls page cycler in telemetry. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 8 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/popular_urls.py
diff --git a/tools/perf/page_sets/popular_urls.py b/tools/perf/page_sets/popular_urls.py
new file mode 100644
index 0000000000000000000000000000000000000000..68d501e1b97fcb42b040699ab12a3463cca8da80
--- /dev/null
+++ b/tools/perf/page_sets/popular_urls.py
@@ -0,0 +1,138 @@
+# Copyright 2016 The Chromium Authors. All rights reserved.
+# Use of this source code is governed by a BSD-style license that can be
+# found in the LICENSE file.
+
+from telemetry.page import page as page_module
+from telemetry.page import shared_page_state
+from telemetry import story
+
+
+class PopularUrlsPage(page_module.Page):
+
+ def __init__(self, url, page_set):
+ super(PopularUrlsPage, self).__init__(
+ url=url, page_set=page_set,
+ shared_page_state_class=shared_page_state.SharedMobilePageState,
+ credentials_path='data/credentials.json')
+
+urls_list = [
jbudorick 2016/05/02 17:46:12 nit: I_AM_A_MODULE_SCOPE_CONSTANT also, I'd prefe
+ "http://www.google.com/",
+ "http://www.facebook.com/",
+ "http://www.youtube.com/",
+ "http://www.yahoo.com/",
+ "http://www.baidu.com/",
+ "http://www.wikipedia.org/",
+ "http://www.live.com/",
+ "http://www.twitter.com/",
+ "http://www.qq.com/",
+ "http://www.amazon.com/",
+ "http://www.blogspot.com/",
+ "http://www.linkedin.com/",
+ "http://www.google.co.in/",
+ "http://www.taobao.com/",
+ "http://www.sina.com.cn/",
+ "http://www.yahoo.co.jp/",
+ "http://www.msn.com/",
+ "http://www.wordpress.com/",
+ "http://www.t.co/",
+ "http://www.google.de/",
+ "http://www.google.com.hk/",
+ "http://www.ebay.com/",
+ "http://www.googleusercontent.com/",
+ "http://www.google.co.uk/",
+ "http://www.google.co.jp/",
+ "http://www.yandex.ru/",
+ "http://www.163.com/",
+ "http://www.google.fr/",
+ "http://www.weibo.com/",
+ "http://www.bing.com/",
+ "http://www.microsoft.com/",
+ "http://www.google.com.br/",
+ "http://www.mail.ru/",
+ "http://www.tumblr.com/",
+ "http://www.pinterest.com/",
+ "http://www.paypal.com/",
+ "http://www.apple.com/",
+ "http://www.google.it/",
+ "http://www.blogger.com/",
+ "http://www.babylon.com/",
+ "http://www.google.es/",
+ "http://www.google.ru/",
+ "http://www.craigslist.org/",
+ "http://www.sohu.com/",
+ "http://www.imdb.com/",
+ "http://www.flickr.com/",
+ "http://www.bbc.co.uk/",
+ "http://www.xvideos.com/",
+ "http://www.fc2.com/",
+ "http://www.go.com/",
+ "http://www.xhamster.com/",
+ "http://www.ask.com/",
+ "http://www.ifeng.com/",
+ "http://www.google.com.mx/",
+ "http://www.youku.com/",
+ "http://www.google.ca/",
+ "http://www.livejasmin.com/",
+ "http://www.tmall.com/",
+ "http://www.zedo.com/",
+ "http://www.imgur.com/",
+ "http://www.conduit.com/",
+ "http://www.odnoklassniki.ru/",
+ "http://www.cnn.com/",
+ "http://www.adobe.com/",
+ "http://www.google.co.id/",
+ "http://www.mediafire.com/",
+ "http://www.thepiratebay.se/",
+ "http://www.pornhub.com/",
+ "http://www.aol.com/",
+ "http://www.hao123.com/",
+ "http://www.espn.go.com/",
+ "http://www.alibaba.com/",
+ "http://www.ebay.de/",
+ "http://www.google.com.tr/",
+ "http://www.rakuten.co.jp/",
+ "http://www.about.com/",
+ "http://www.avg.com/",
+ "http://www.google.com.au/",
+ "http://www.blogspot.in/",
+ "http://www.wordpress.org/",
+ "http://www.chinaz.com/",
+ "http://www.ameblo.jp/",
+ "http://www.ebay.co.uk/",
+ "http://www.godaddy.com/",
+ "http://www.amazon.de/",
+ "http://www.google.pl/",
+ "http://www.uol.com.br/",
+ "http://www.360buy.com/",
+ "http://www.stackoverflow.com/",
+ "http://www.amazon.co.jp/",
+ "http://www.bp.blogspot.com/",
+ "http://www.dailymotion.com/",
+ "http://www.huffingtonpost.com/",
+ "http://www.4shared.com/",
+ "http://www.mercadolivre.com.br/",
+ "http://www.vagalume.com.br/",
+ "http://www.letras.mus.br/",
+ "http://www.abril.com.br/",
+ "http://www.detik.com/",
+ "http://www.stafaband.info/",
+ "http://www.kaskus.co.id/",
+ "http://www.tribunnews.com/",
+ "http://www.kompas.com/",
+ "http://www.indianrail.gov.in/",
+ "http://www.flipkart.com/",
+ "http://www.snapdeal.com/",
+ "http://www.espncricinfo.com/",
+ "http://www.amazon.in/",
+]
+
rnephew (Reviews Here) 2016/05/02 18:02:52 nit: 2 lines.
+class PopularUrlsPageSet(story.StorySet):
+ """Visit list of popular urls from webpage replay."""
+
+ def __init__(self):
+ super(PopularUrlsPageSet, self).__init__(
+ archive_data_file='data/popular_urls.json',
+ cloud_storage_bucket=story.PARTNER_BUCKET)
+
+ for url in urls_list:
+ self.AddStory(PopularUrlsPage(url, self))
« tools/perf/page_sets/key_mobile_sites.py ('K') | « tools/perf/page_sets/key_mobile_sites.py ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698