Change for preparation for reference-build on perfbot (media)

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
@@ skipping 175 matching lines @@
 
   def PreEachRunProcess(self, run_counter):
     """A method to execute before each run.
 
     The default behavior is to record start time.
 
     Args:
       run_counter: counter for each run.
     """
     self.start = time.time()
+    if eval(os.getenv(MediaTestEnvNames.REFERENCE_BUILD_ENV_NAME)):

[dennis_jeffrey, 2011/05/19 01:20:13]

(optional) I recommend adding a comment here that this environment variable is
assumed to have value "True" or "False" (until I realized that, I wasn't sure
why you were calling eval() on it).

[Nirnimesh, 2011/05/19 06:40:20]

Why do you need to eval it anyway?
Why not assume False if the env is missing, and True otherwise.

[imasaki1, 2011/05/20 05:12:01]

Changed based on Nirnimesh comments.

[imasaki1, 2011/05/20 05:12:01]

Done.

Also change other boolean environment valuables as well.

+      self.current_trace_type = 't_ref'
 
   def PostEachRunProcess(self, run_counter):
     """A method to execute after each run.
 
     The default behavior is to do nothing.
 
     Args:
       run_counter: counter for each run.
     """
     pass
 
   def GetPlayerHTMLFileName(self):
     """A method to get the player HTML file name."""
     return 'media_playback.html'
 
 
 if __name__ == '__main__':
   pyauto_media.Main()