[Telemetry] Remove session_restore's use of PageTest.CanRunForPage.

tools/telemetry/telemetry/user_story/user_story_set.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.
 
 import inspect
 import os
 
 from telemetry import user_story as user_story_module
 from telemetry.wpr import archive_info
 
 
 class UserStorySet(object):
   """A collection of user story.
 
   A typical usage of UserStorySet would be to subclass it and then calling
   AddUserStory for each UserStory..

[qyearsley, 2015/01/28 19:39:08]

Extra ".".

[slamm, 2015/02/23 23:46:35]

Done.

   """
 
   def __init__(self, archive_data_file='', cloud_storage_bucket=None,
                serving_dirs=None):
     """Creates a new UserStorySet.
 
     Args:
       archive_data_file: The path to Web Page Replay's archive data, relative
           to self.base_dir.
       cloud_storage_bucket: The cloud storage bucket used to download
@@ skipping 36 matching lines @@
     """Lazily constructs wpr_archive_info if it's not set and returns it."""
     if self.archive_data_file and not self._wpr_archive_info:
       self._wpr_archive_info = archive_info.WprArchiveInfo.FromFile(
           os.path.join(self.base_dir, self.archive_data_file), self.bucket)
     return self._wpr_archive_info
 
   def AddUserStory(self, user_story):
     assert isinstance(user_story, user_story_module.UserStory)
     self.user_stories.append(user_story)
 
+  def RemoveUserStory(self, user_story):
+    """Remove a user story.

[qyearsley, 2015/01/28 19:39:08]

1. The verb in the docstring should be third-person singular, i.e. "Removes...".

2. Could say UserStory instead of "user story", since we're expecting the
argument to be a UserStory instance.

[slamm, 2015/02/23 23:46:35]

Done.

+
+    Allows the user stories to be filtered.
+    """
+    self.user_stories.remove(user_story)
+
   @classmethod
   def Name(cls):
     """ Returns the string name of this UserStorySet.

[qyearsley, 2015/01/28 19:39:08]

Outside the scope of this CL:
There's an extra space here, and a blank line should be added after the first
line of the docstring.

[slamm, 2015/02/23 23:46:35]

Done.

These extra spaces are all over the place.

     Note that this should be a classmethod so benchmark_runner script can match
     user story class with its name specified in the run command:
     'Run <User story test name> <User story class name>'
     """
     return cls.__module__.split('.')[-1]
 
   @classmethod
   def Description(cls):
     """ Return a string explaining in human-understandable terms what this
     user story represents.
@@ skipping 26 matching lines @@
     return self.user_stories.__iter__()
 
   def __len__(self):
     return len(self.user_stories)
 
   def __getitem__(self, key):
     return self.user_stories[key]
 
   def __setitem__(self, key, value):
     self.user_stories[key] = value