Refactor how we fetch Telemetry deps conditionally

tools/perf/conditionally_execute

+#!/usr/bin/env python
+# Copyright 2017 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 argparse
+import os
+import subprocess
+import sys
+
+
+def main(args):
+  parser = argparse.ArgumentParser()
+  parser.add_argument(
+      '--gyp-condition', '-c', type=str, required=True,
+      help=('Name of environment variable that acts as the switch. If the '

[Kai Ninomiya, 2017/06/05 21:58:51]

nit: it's not really an environment variable that acts as a switch, but a string
that must appear in the environment variable GYP_DEFINES

[nednguyen, 2017/06/05 22:24:24]

Done.

+            'variable is set in the environment, this will execute the target '
+            'script'))
+  options, script_args = parser.parse_known_args()
+
+  # Make sure that we always execute target script with python.
+  if 'python' not in script_args[0]:
+    script_args.insert(0, sys.executable)
+
+  gyp_defines = os.environ.get('GYP_DEFINES', '')
+  if options.gyp_condition in gyp_defines:

[Kai Ninomiya, 2017/06/05 21:58:51]

This may not matter since this should get replaced soon, but maybe it would be
good to guard against, e.g., 'FOOfetch_telemetry_dependencies=1BAR'. Most of the
existing code that checks GYP_DEFINES doesn't do this currently, though.

[nednguyen, 2017/06/05 22:24:24]

Done.

+    print 'Found matching condition in GYP_DEFINES. Execute: %s' % (
+        ' '.join(script_args))
+    subprocess.check_call(script_args)
+  else:
+    print ('Not found "%s" condition in GYP_DEFINES="%s". '
+           'Skip script execution.' %
+           (options.gyp_condition, gyp_defines))
+  return 0
+
+
+if __name__ == '__main__':
+  main(sys.argv[1:])