commit 2a3c8493af6efe2256025b6cfbec882845925bd0
author rmistry <rmistry@google.com> Tue Mar 31 10:59:15 2015 -0700
committer Commit bot <commit-bot@chromium.org> Tue Mar 31 10:59:15 2015 -0700
tree 5671404cac5d89d58cf29fa961d65a357b791149
parent f5f8518fe0bbd2703e4ffc1b11ad7b4312ff7641

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

diff --git a/tools/skp/webpages_playback.py b/tools/skp/webpages_playback.py
index 7fbb372..7f79c46 100644
--- a/tools/skp/webpages_playback.py
+++ b/tools/skp/webpages_playback.py
@@ -121,9 +121,7 @@
 # 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 @@
     # /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