[telemetry] Cleanups for benchmarks importing page sets.

tools/telemetry/telemetry/page/page_set.py

 # 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.
 
 import csv
 import inspect
 import os
 
 from telemetry.core import util
 from telemetry.page import page as page_module
@@ skipping 28 matching lines @@
     self.pages = []
     self.serving_dirs = set()
     serving_dirs = [] if serving_dirs is None else serving_dirs
     # Makes sure that page_set's serving_dirs are absolute paths
     for sd in serving_dirs:
       if os.path.isabs(sd):
         self.serving_dirs.add(os.path.realpath(sd))
       else:
         self.serving_dirs.add(os.path.realpath(os.path.join(self.base_dir, sd)))
 
+  @classmethod
+  def Name(cls):
+    return cls.__module__.split('.')[-1]
+
+  @classmethod
+  def Description(cls):

[nednguyen, 2014/06/04 22:15:45]

Can you add a TODO to remove the member description from page_set?

[dtu, 2014/06/05 00:07:42]

I'm going to make Name/Description a MixIn because it's used by a lot of
classes.
It'll be useful for page sets in the future when listing them from the command
line.

+    if cls.__doc__:
+      return cls.__doc__.splitlines()[0]
+    else:
+      return ''
+
   def AddPage(self, page):
     assert page.page_set is self
     self.pages.append(page)
 
   def AddPageWithDefaultRunNavigate(self, page_url):
     """ Add a simple page with url equals to page_url that contains only default
     RunNavigateSteps.
     """
     self.AddPage(page_module.Page(
       page_url, self, self.base_dir))
@@ skipping 89 matching lines @@
     return self.pages.__iter__()
 
   def __len__(self):
     return len(self.pages)
 
   def __getitem__(self, key):
     return self.pages[key]
 
   def __setitem__(self, key, value):
     self.pages[key] = value