Add another test terminate condition in media perf test.

chrome/test/functional/media/media_test_base.py

 #!/usr/bin/python
 # Copyright (c) 2011 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.
 
 """A framework to run PyAuto HTML media tests.
 
 This PyAuto powered script plays media (video or audio) files (using the HTML5
 tag embedded in an HTML file). The parameters needed to run this test are
 passed in the form of environment variables (such as the number of runs).
@@ skipping 108 matching lines @@
     """
     test_scenarios = []
     rows = csv.reader(open(test_scenario_filename))
     for row in rows:
       test_scenarios.append('|'.join(row))
     return test_scenarios
 
   def ExecuteTest(self):
     """Test HTML5 Media Tag."""
 
-    def _VideoEnded():
+    def _VideoEndedOrErrorOut():
       """Determine if the video ended.

[dennis_jeffrey, 2011/07/07 19:45:21]

add something like this: '...or there was an error when playing.'

[imasaki1, 2011/07/07 20:42:05]

Done.

 
-      When the video has finished playing, its title is updated by player.html.
+      When the video has finished playing or there is error in playing the
+      video (e.g, the video cannot be found), its title is updated by
+      player.html.
 
       Returns:
-        True if the video has ended.
+        True if the video has ended or error out.

[dennis_jeffrey, 2011/07/07 19:45:21]

'error out' --> 'an error occurred'

[imasaki1, 2011/07/07 20:42:05]

Done.

       """
-      return self.GetDOMValue('document.title').strip() == 'END'
+      return self.GetDOMValue('document.title').strip() == 'END' or (
+                 'ERROR' in self.GetDOMValue('document.title'))

[dennis_jeffrey, 2011/07/07 19:45:21]

Recommend using parens like this:

return (self.GetDOMValue('document.title').strip() == 'END' or
        'ERROR' in self.GetDOMValue('document.title'))

[imasaki1, 2011/07/07 20:42:05]

Done.

 
     self.PreAllRunsProcess()
     for run_counter in range(self.number_of_runs):
       self.run_counter = run_counter
       self.PreEachRunProcess(run_counter)
       url = self.url
       if self.whole_test_scenarios:
         url += '&actions=' + self.whole_test_scenarios[run_counter]
       logging.debug('Navigate to %s', url)
       self.NavigateToURL(url)
-      self.WaitUntil(lambda: _VideoEnded(),
+      self.WaitUntil(lambda: _VideoEndedOrErrorOut(),
                      self.TIMEOUT)
       self.PostEachRunProcess(run_counter)
 
     self.PostAllRunsProcess()
 
   # A list of methods that should be overridden in the subclass.
   # It is a good practice to call these methods even if these are
   # overridden.
 
   def PreAllRunsProcess(self):
@@ skipping 56 matching lines @@
     """
     pass
 
   def GetPlayerHTMLFileName(self):
     """A method to get the player HTML file name."""
     return 'media_playback.html'
 
 
 if __name__ == '__main__':
   pyauto_media.Main()