[telemetry] Cleanups for benchmarks importing page sets.

tools/telemetry/telemetry/page/page_set.py

 # Copyright 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
 from telemetry.page import page_set_archive_info
 
 class PageSetError(Exception):
   pass
 
 
 class PageSet(object):
-  def __init__(self, file_path=None, description='', archive_data_file='',
+  def __init__(self, file_path=None, archive_data_file='',
                credentials_path=None, user_agent_type=None,
                make_javascript_deterministic=True, startup_url='',
                serving_dirs=None):
     # The default value of file_path is location of the file that define this
     # page set instance's class.
     if file_path is None:
       file_path = inspect.getfile(self.__class__)
       # Turn pyc file into py files if we can
       if file_path.endswith('.pyc') and os.path.exists(file_path[:-1]):
         file_path = file_path[:-1]
 
     self.file_path = file_path
     # These attributes can be set dynamically by the page set.
-    self.description = description
     self.archive_data_file = archive_data_file
     self.credentials_path = credentials_path
     self.user_agent_type = user_agent_type
     self.make_javascript_deterministic = make_javascript_deterministic
     self._wpr_archive_info = None
     self.startup_url = startup_url
     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):
+    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