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

chrome/test/functional/media/pyauto_media.py

Index: chrome/test/functional/media/pyauto_media.py
diff --git a/chrome/test/functional/media/pyauto_media.py b/chrome/test/functional/media/pyauto_media.py
index c40598df88fdbe0c72d37bc7b1eb6cc275843261..c6431724c323707b2c85d69f92d6edec24c56d91 100644
--- a/chrome/test/functional/media/pyauto_media.py
+++ b/chrome/test/functional/media/pyauto_media.py
@@ -25,6 +25,12 @@ 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."""
@@ -34,6 +40,9 @@ def _SetupPaths():
   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()