Turn on Chromium's top 25 page set

tools/skp/webpages_playback.py

 #!/usr/bin/env python
 # Copyright (c) 2012 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.
 
 """Archives or replays webpages and creates SKPs in a Google Storage location.
 
 To archive webpages and store SKP files (archives should be rarely updated):
 
 cd skia
@@ skipping 103 matching lines @@
   (gs_utils.GSUtils.IdType.GROUP_BY_DOMAIN, 'google.com',
    gs_utils.GSUtils.Permission.READ),
 ]
 
 # Path to Chromium's page sets.
 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',
 }
 
 
 def remove_prefix(s, prefix):
   if s.startswith(prefix):
     return s[len(prefix):]
   return s
 
 
 class SkPicturePlayback(object):
@@ skipping 244 matching lines @@
     # skia_yahooanswers_desktop -> skia, yahooanswers, desktop
     _, page_name, device = ps_basename.split('_')
     basename = '%s_%s' % (DEVICE_TO_PLATFORM_PREFIX[device], page_name)
     return basename[:MAX_SKP_BASE_NAME_LEN] + '.skp'
 
   def _GetChromiumSkpFileName(self, page_set, site):
     """Returns the SKP file name for Chromium page sets."""
     # /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
     basename = '%s_%s' % (CHROMIUM_PAGE_SETS_TO_PREFIX[ps_filename], webpage)
     return basename[:MAX_SKP_BASE_NAME_LEN] + '.skp'
 
   def _RenameSkpFiles(self, page_set):
     """Rename generated SKP files into more descriptive names.
 
     Look into the subdirectory of TMP_SKP_DIR and find the most interesting
@@ skipping 148 matching lines @@
       default=None)
   option_parser.add_option(
       '', '--non-interactive', action='store_true',
       help='Runs the script without any prompts. If this flag is specified and '
            '--skia_tools is specified then the debugger is not run.',
       default=False)
   options, unused_args = option_parser.parse_args()
 
   playback = SkPicturePlayback(options)
   sys.exit(playback.Run())