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

chrome/test/functional/media/pyauto_media.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.
 
 """Setup for PyAuto media tests.
 
 Use the following in your scripts to run them standalone:
 
 # This should be at the top
 import pyauto_media
 
 if __name__ == '__main__':
   pyauto_media.Main()
 
 This script looks similar to pyauto_functional.py. However, unlike
 pyauto_functional, this script can NOT be used as an executable to
 fire off other media scripts since media tests require additional
 parameters to be set in the form of environment variables (unless you
 want these variables to be defaults, which is generally not a good
 idea).
 """
 
 import os
 import sys
 
+from media_test_env_names import MediaTestEnvNames
+
+# Define the location of the binary used for reference build

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

Add period at the end of this sentence.

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

Done.

+# TODO(imasaki): change this directory once reference build
+#                directory is set.
+REFERENCE_BUILD_DIR = os.path.join('/tmp','chrome-linux')
 
 def _SetupPaths():
   """Setting path to find pyauto_functional.py."""
   media_dir = os.path.abspath(os.path.dirname(__file__))
   sys.path.append(media_dir)
   sys.path.append(os.path.normpath(os.path.join(media_dir, os.pardir)))
   sys.path.append(os.path.normpath(os.path.join(
       media_dir, os.pardir, os.pardir, os.pardir, os.pardir,
       'third_party', 'psutil')))
+  # Setting PYTHONPATH for reference build

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

Add period at the end of this sentence.

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

Done.

+  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".

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

Don't eval.

Just:

if os.getenv(...):
  sys.path.insert(0, os.getenv(...))

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

Done.

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

Changed based on Nirnimesh's comments.

+    sys.path.insert(0, REFERENCE_BUILD_DIR)
 
 _SetupPaths()
 
 import pyauto_functional
 import pyauto
 
 
 class Main(pyauto_functional.Main):
   """Main program for running PyAuto media tests."""
 
   def __init__(self):
     pyauto_functional.Main.__init__(self)
 
 
 if __name__ == '__main__':
   Main()