From 2a3c8493af6efe2256025b6cfbec882845925bd0 Mon Sep 17 00:00:00 2001 From: rmistry Date: Tue, 31 Mar 2015 10:59:15 -0700 Subject: Turn on Chromium's top 25 page set #crskps BUG=skia: BUG=skia:3574 NOTRY=true TBR=borenet Review URL: https://codereview.chromium.org/1045153004 --- tools/skp/webpages_playback.py | 11 ++++------- 1 file changed, 4 insertions(+), 7 deletions(-) (limited to 'tools/skp') diff --git a/tools/skp/webpages_playback.py b/tools/skp/webpages_playback.py index 7fbb372fd3..7f79c4665d 100644 --- a/tools/skp/webpages_playback.py +++ b/tools/skp/webpages_playback.py @@ -121,9 +121,7 @@ CHROMIUM_PAGE_SETS_PATH = os.path.join('tools', 'perf', 'page_sets') # Dictionary of supported Chromium page sets to their file prefixes. CHROMIUM_PAGE_SETS_TO_PREFIX = { 'key_mobile_sites_smooth.py': 'keymobi', - # TODO(rmistry): Uncomment the below after it is verified that the above - # works. - # 'top_25_smooth.py': 'top25desk', + 'top_25_smooth.py': 'top25desk', } @@ -388,10 +386,9 @@ class SkPicturePlayback(object): # /path/to/http___mobile_news_sandbox_pt0 -> http___mobile_news_sandbox_pt0 _, webpage = os.path.split(site) # http___mobile_news_sandbox_pt0 -> mobile_news_sandbox_pt0 - if webpage.startswith('http___'): - webpage = webpage[7:] - elif webpage.startswith('https___'): - webpage = webpage[8:] + for prefix in ('http___', 'https___'): + if webpage.startswith(prefix): + webpage = webpage[len(prefix):] # /path/to/skia_yahooanswers_desktop.py -> skia_yahooanswers_desktop.py ps_filename = os.path.basename(page_set) # http___mobile_news_sandbox -> pagesetprefix_http___mobile_news_sandbox -- cgit v1.2.3