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

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

Issue 1306083005: [tools/Perf] Disable pylint's long line check for urls (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 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 | tools/perf/page_sets/key_mobile_sites_repaint.py » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Copyright 2015 The Chromium Authors. All rights reserved. 1 # Copyright 2015 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 from telemetry.page import page as page_module 4 from telemetry.page import page as page_module
5 from telemetry.page import shared_page_state 5 from telemetry.page import shared_page_state
6 from telemetry import story 6 from telemetry import story
7 7
8 from page_sets import key_mobile_sites_pages 8 from page_sets import key_mobile_sites_pages
9 9
10 10
(...skipping 55 matching lines...) Expand 10 before | Expand all | Expand 10 after
66 url='http://cuteoverload.com', page_set=self, labels=['fastpath'])) 66 url='http://cuteoverload.com', page_set=self, labels=['fastpath']))
67 67
68 # Why: #11 (Alexa global), google property; some blogger layouts 68 # Why: #11 (Alexa global), google property; some blogger layouts
69 # have infinite scroll but more interesting. 69 # have infinite scroll but more interesting.
70 self.AddStory(KeyMobileSitesPage( 70 self.AddStory(KeyMobileSitesPage(
71 url='http://googlewebmastercentral.blogspot.com/', 71 url='http://googlewebmastercentral.blogspot.com/',
72 page_set=self, name='Blogger')) 72 page_set=self, name='Blogger'))
73 73
74 # Why: #18 (Alexa global), Picked an interesting post """ 74 # Why: #18 (Alexa global), Picked an interesting post """
75 self.AddStory(KeyMobileSitesPage( 75 self.AddStory(KeyMobileSitesPage(
76 # pylint: disable=line-too-long
77 url='http://en.blog.wordpress.com/2012/09/04/freshly-pressed-editors-picks -for-august-2012/', 76 url='http://en.blog.wordpress.com/2012/09/04/freshly-pressed-editors-picks -for-august-2012/',
78 page_set=self, 77 page_set=self,
79 name='Wordpress')) 78 name='Wordpress'))
80 79
81 # Why: #6 (Alexa) most visited worldwide, picked an interesting page 80 # Why: #6 (Alexa) most visited worldwide, picked an interesting page
82 self.AddStory(KeyMobileSitesPage( 81 self.AddStory(KeyMobileSitesPage(
83 url='http://en.wikipedia.org/wiki/Wikipedia', 82 url='http://en.wikipedia.org/wiki/Wikipedia',
84 page_set=self, 83 page_set=self,
85 name='Wikipedia (1 tab)')) 84 name='Wikipedia (1 tab)'))
86 85
(...skipping 33 matching lines...) Expand 10 before | Expand all | Expand 10 after
120 url='http://www.reddit.com/r/programming/comments/1g96ve', 119 url='http://www.reddit.com/r/programming/comments/1g96ve',
121 page_set=self, labels=['fastpath'])) 120 page_set=self, labels=['fastpath']))
122 121
123 # Why: Problematic use of fixed position elements """ 122 # Why: Problematic use of fixed position elements """
124 self.AddStory(KeyMobileSitesPage( 123 self.AddStory(KeyMobileSitesPage(
125 url='http://www.boingboing.net', page_set=self, labels=['fastpath'])) 124 url='http://www.boingboing.net', page_set=self, labels=['fastpath']))
126 125
127 # Add simple pages with no custom navigation logic or labels. 126 # Add simple pages with no custom navigation logic or labels.
128 urls_list = [ 127 urls_list = [
129 # Why: Social; top Google property; Public profile; infinite scrolls. 128 # Why: Social; top Google property; Public profile; infinite scrolls.
130 # pylint: disable=line-too-long
131 'https://plus.google.com/app/basic/110031535020051778989/posts?source=appp romo', 129 'https://plus.google.com/app/basic/110031535020051778989/posts?source=appp romo',
132 # Why: crbug.com/242544 130 # Why: crbug.com/242544
133 ('http://www.androidpolice.com/2012/10/03/rumor-evidence-mounts-that-an-' 131 ('http://www.androidpolice.com/2012/10/03/rumor-evidence-mounts-that-an-'
134 'lg-optimus-g-nexus-is-coming-along-with-a-nexus-phone-certification-' 132 'lg-optimus-g-nexus-is-coming-along-with-a-nexus-phone-certification-'
135 'program/'), 133 'program/'),
136 # Why: crbug.com/149958 134 # Why: crbug.com/149958
137 'http://gsp.ro', 135 'http://gsp.ro',
138 # Why: Top tech blog 136 # Why: Top tech blog
139 'http://theverge.com', 137 'http://theverge.com',
140 # Why: Top tech site 138 # Why: Top tech site
141 'http://digg.com', 139 'http://digg.com',
142 # Why: Top Google property; a Google tab is often open 140 # Why: Top Google property; a Google tab is often open
143 'https://www.google.com/#hl=en&q=barack+obama', 141 'https://www.google.com/#hl=en&q=barack+obama',
144 # Why: #1 news worldwide (Alexa global) 142 # Why: #1 news worldwide (Alexa global)
145 'http://news.yahoo.com', 143 'http://news.yahoo.com',
146 # Why: #2 news worldwide 144 # Why: #2 news worldwide
147 'http://www.cnn.com', 145 'http://www.cnn.com',
148 # Why: #1 commerce website by time spent by users in US 146 # Why: #1 commerce website by time spent by users in US
149 'http://shop.mobileweb.ebay.com/searchresults?kw=viking+helmet', 147 'http://shop.mobileweb.ebay.com/searchresults?kw=viking+helmet',
150 # Why: #1 Alexa recreation 148 # Why: #1 Alexa recreation
151 # pylint: disable=line-too-long
152 'http://www.booking.com/searchresults.html?src=searchresults&latitude=65.0 500&longitude=25.4667', 149 'http://www.booking.com/searchresults.html?src=searchresults&latitude=65.0 500&longitude=25.4667',
153 # Why: #1 Alexa sports 150 # Why: #1 Alexa sports
154 'http://sports.yahoo.com/', 151 'http://sports.yahoo.com/',
155 # Why: Top tech blog 152 # Why: Top tech blog
156 'http://techcrunch.com', 153 'http://techcrunch.com',
157 # Why: #6 Alexa sports 154 # Why: #6 Alexa sports
158 'http://mlb.com/', 155 'http://mlb.com/',
159 # Why: #14 Alexa California 156 # Why: #14 Alexa California
160 'http://www.sfgate.com/', 157 'http://www.sfgate.com/',
161 # Why: Non-latin character set 158 # Why: Non-latin character set
162 'http://worldjournal.com/', 159 'http://worldjournal.com/',
163 # Why: Mobile wiki 160 # Why: Mobile wiki
164 'http://www.wowwiki.com/World_of_Warcraft:_Mists_of_Pandaria', 161 'http://www.wowwiki.com/World_of_Warcraft:_Mists_of_Pandaria',
165 # Why: #15 Alexa news 162 # Why: #15 Alexa news
166 'http://online.wsj.com/home-page', 163 'http://online.wsj.com/home-page',
167 # Why: Image-heavy mobile site 164 # Why: Image-heavy mobile site
168 'http://www.deviantart.com/', 165 'http://www.deviantart.com/',
169 # Why: Top search engine 166 # Why: Top search engine
170 ('http://www.baidu.com/s?wd=barack+obama&rsv_bp=0&rsv_spt=3&rsv_sug3=9&' 167 ('http://www.baidu.com/s?wd=barack+obama&rsv_bp=0&rsv_spt=3&rsv_sug3=9&'
171 'rsv_sug=0&rsv_sug4=3824&rsv_sug1=3&inputT=4920'), 168 'rsv_sug=0&rsv_sug4=3824&rsv_sug1=3&inputT=4920'),
172 # Why: Top search engine 169 # Why: Top search engine
173 'http://www.bing.com/search?q=sloths', 170 'http://www.bing.com/search?q=sloths',
174 # Why: Good example of poor initial scrolling 171 # Why: Good example of poor initial scrolling
175 'http://ftw.usatoday.com/2014/05/spelling-bee-rules-shenanigans' 172 'http://ftw.usatoday.com/2014/05/spelling-bee-rules-shenanigans'
176 ] 173 ]
177 174
178 for url in urls_list: 175 for url in urls_list:
179 self.AddStory(KeyMobileSitesPage(url, self)) 176 self.AddStory(KeyMobileSitesPage(url, self))
OLDNEW
« no previous file with comments | « no previous file | tools/perf/page_sets/key_mobile_sites_repaint.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698