Fix cr SKPs naming

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 370 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
-    webpage = webpage.lstrip('http___').lstrip('https___')
+    if webpage.startswith('http___'):
+      webpage = webpage[7:]

[borenet, 2015/03/31 15:29:53]

I prefer something like this:

prefix = 'http___'
if webpage.startswith(prefix):
  webpage = webpage[len(prefix):]

Or even:

for prefix in ('http___', 'https___'):
  if webpage.startswith(prefix):
    webpage = webpage[len(prefix):]

+    elif webpage.startswith('https___'):
+      webpage = webpage[8:]
     # /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())