Have repeats understand "time"

tools/telemetry/telemetry/page/page_test.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 logging
 
 from telemetry.core import util
 from telemetry.page import gtest_test_results
 from telemetry.page import page_test_results
 from telemetry.page.actions import all_page_actions
 from telemetry.page.actions import page_action
@@ skipping 89 matching lines @@
       action.CustomizeBrowserOptions(options)
 
   def SetUpBrowser(self, browser):
     """Override to customize the browser right after it has launched."""
     pass
 
   def CanRunForPage(self, page): #pylint: disable=W0613
     """Override to customize if the test can be ran for the given page."""
     return True
 
-  def WillRunPageSet(self, tab):
-    """Override to do operations before the page set is navigated."""
+  def WillRunTest(self, tab):
+    """Override to do operations before the page set(s) are navigated."""
     pass
 
-  def DidRunPageSet(self, tab, results):
-    """Override to do operations after page set is completed, but before browser
-    is torn down."""
+  def DidRunTest(self, tab, results):
+    """Override to do operations after all page set(s) are completed, but before
+
+    browser is torn down."""

[dtu, 2013/07/11 17:28:38]

Sorry, this isn't quite what Dennis meant. The first line should be a complete
one-line summary. A longer description or any clarifying statements can follow
after the blank line. You need to break this sentence up into two.

[edmundyan, 2013/07/11 18:55:01]

Got it now, thanks.

     pass
 
   def DidStartHTTPServer(self, tab):
     """Override to do operations after the HTTP server is started."""
     pass
 
   def WillNavigateToPage(self, page, tab):
     """Override to do operations before the page is navigated."""
     pass
 
@@ skipping 68 matching lines @@
         finally:
           self.DidRunAction(page, tab, action)
 
       # Closing the connections periodically is needed; otherwise we won't be
       # able to open enough sockets, and the pages will time out.
       util.CloseConnections(tab)
 
   @property
   def action_name_to_run(self):
     return self._action_name_to_run