Explicitly setting story names for all smoothness benchmarks

tools/perf/page_sets/image_decoding_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 ImageDecodingCasesPage(page_module.Page):
 
-  def __init__(self, url, page_set):
-    super(ImageDecodingCasesPage, self).__init__(url=url, page_set=page_set)
+  def __init__(self, url, page_set, name):
+    super(ImageDecodingCasesPage, self).__init__(
+        url=url, page_set=page_set, name=name)
 
   def RunPageInteractions(self, action_runner):
     with action_runner.CreateInteraction('DecodeImage'):
       action_runner.Wait(5)
 
 class ImageDecodingCasesPageSet(story.StorySet):
 
   """ A directed benchmark of accelerated jpeg image decoding performance """
 
   def __init__(self):
-    super(ImageDecodingCasesPageSet, self).__init__()
+    super(ImageDecodingCasesPageSet, self).__init__(verify_names=True)
 
     urls_list = [
       'file://image_decoding_cases/yuv_decoding.html'
     ]
 
     for url in urls_list:
-      self.AddStory(ImageDecodingCasesPage(url, self))
+      self.AddStory(ImageDecodingCasesPage(url, self, name='yuv_decoding.html'))

[cblume, 2017/06/06 20:10:58]

The only url in the urls_list is yuv_decoding.
This line hard-codes that name in.

But if we added another url to the urls_list, would this hard-coded value then
be incorrect?

[ashleymarie1, 2017/06/06 20:35:00]

Good point! I update the urls_list to be a list of tuples url, name
PTAL at patchset 6+