Explicitly setting names for media story sets

tools/perf/page_sets/media_cns_cases.py

 # Copyright 2014 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.
 from telemetry.page import page as page_module
 from telemetry import story
 
 
 class BasicPlayPage(page_module.Page):
 
   def __init__(self, url, page_set, name=''):
@@ skipping 28 matching lines @@
 
   def RunPageInteractions(self, action_runner):
     self.SeekBeforeAndAfterPlayhead(action_runner)
 
 
 class MediaCnsCasesPageSet(story.StorySet):
 
   """ Media benchmark on network constrained conditions. """
 
   def __init__(self):
-    super(MediaCnsCasesPageSet, self).__init__()
+    super(MediaCnsCasesPageSet, self).__init__(verify_names=True)
 
     urls_list = [
       # pylint: disable=line-too-long
       'file://tough_video_cases/video.html?id=no_constraints_webm&src=tulip2.webm&net=none',
       # pylint: disable=line-too-long
       'file://tough_video_cases/video.html?id=cable_webm&src=tulip2.webm&net=cable',
       # pylint: disable=line-too-long
       'file://tough_video_cases/video.html?id=wifi_webm&src=tulip2.webm&net=wifi',
       # pylint: disable=line-too-long
       'file://tough_video_cases/video.html?id=no_constraints_ogv&src=tulip2.ogv&net=none',
@@ skipping 27 matching lines @@
       'file://tough_video_cases/video.html?id=wifi_mp3&src=tulip2.mp3&type=audio&net=wifi',
       # pylint: disable=line-too-long
       'file://tough_video_cases/video.html?id=no_constraints_m4a&src=tulip2.m4a&type=audio&net=none',
       # pylint: disable=line-too-long
       'file://tough_video_cases/video.html?id=cable_m4a&src=tulip2.m4a&type=audio&net=cable',
       # pylint: disable=line-too-long
       'file://tough_video_cases/video.html?id=wifi_m4a&src=tulip2.m4a&type=audio&net=wifi'
     ]
 
     for url in urls_list:
-      self.AddStory(BasicPlayPage(url, self))
+      self.AddStory(BasicPlayPage(url, self, url.split('/')[-1]))
 
     urls_list2 = [
       # pylint: disable=line-too-long
       'file://tough_video_cases/video.html?id=wifi_mp3&src=tulip2.mp3&type=audio&net=wifi',
       # pylint: disable=line-too-long
       'file://tough_video_cases/video.html?id=wifi_m4a&src=tulip2.m4a&type=audio&net=wifi',
       # pylint: disable=line-too-long
       'file://tough_video_cases/video.html?id=wifi_ogg&src=tulip2.ogg&type=audio&net=wifi',
       # pylint: disable=line-too-long
       'file://tough_video_cases/video.html?id=wifi_wav&src=tulip2.wav&type=audio&net=wifi',
       # pylint: disable=line-too-long
       'file://tough_video_cases/video.html?id=wifi_mp4&src=tulip2.mp4&type=audio&net=wifi',
       # pylint: disable=line-too-long
       'file://tough_video_cases/video.html?id=wifi_ogv&src=tulip2.ogv&type=audio&net=wifi',
       # pylint: disable=line-too-long
       'file://tough_video_cases/video.html?id=wifi_webm&src=tulip2.webm&type=audio&net=wifi'
     ]
 
     for url in urls_list2:
       if url in urls_list:
         name = 'seek_' + url
       else:
-        name = ''
+        name = url.split('/')[-1]
+      print name

[johnchen, 2017/06/05 14:11:52]

Is "print name" intended to be permanent, or just for debugging?

[ashleymarie1, 2017/06/05 14:16:15]

Just for debugging; I saw this and uploaded a second patch set right after
sending for review :)

       self.AddStory(SeekBeforeAndAfterPlayheadPage(url, self, name=name))