Enable downloading Google Play services via deps hook

build/android/play_services/utils.py

 # Copyright 2015 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.
 
 '''
 Utility functions for all things related to manipulating google play services
 related files.
 '''
 
 import argparse
 import filecmp
 import logging
 import os
 import re
 import sys
 import yaml
 import zipfile
 
 sys.path.append(os.path.join(os.path.dirname(__file__), os.pardir))
 from devil.utils import cmd_helper
+from pylib import constants
+
+sys.path.append(os.path.join(constants.DIR_SOURCE_ROOT,
+                             'third_party', 'colorama', 'src'))
+import colorama
 
 
 _CONFIG_VERSION_NUMBER_KEY = 'version_number'
 _YAML_VERSION_NUMBER_PATTERN = re.compile(
     r'(^\s*%s\s*:\s*)(\d+)(.*$)' % _CONFIG_VERSION_NUMBER_KEY, re.MULTILINE)
 _XML_VERSION_NUMBER_PATTERN = re.compile(
     r'<integer name="google_play_services_version">(\d+)<\/integer>')
 
 
 class DefaultsRawHelpFormatter(argparse.ArgumentDefaultsHelpFormatter,
                                argparse.RawDescriptionHelpFormatter):
   '''
   Combines the features of RawDescriptionHelpFormatter and
   ArgumentDefaultsHelpFormatter, providing defaults for the arguments and raw
   text for the description.
   '''
   pass
 
 
+def EmphasizeMessage(message):

[dgn, 2015/10/28 13:54:33]

Should I make a proper stream handler for the logger instead? I'll move it to a
separate CL then.

[jbudorick, 2015/10/28 14:08:25]

Have warnings print yellow and errors print red? I might actually be interested
in that elsewhere.

[dgn, 2015/10/28 14:27:13]

Should I put that in pylib?

[jbudorick, 2015/10/28 14:28:53]

Yeah, that'd be great.

You may see it in the test runner in the not-too-distant future :)

[jbudorick, 2015/10/28 14:29:31]

(specifically, pylib/utils/)

[dgn, 2015/10/28 16:32:58]

Addressed in https://codereview.chromium.org/1418573010/

+  # pylint warning: does not see members added dynamically in the constructor.
+  # pylint: disable=no-member
+  return (colorama.Fore.YELLOW + colorama.Style.DIM + message +
+      colorama.Style.RESET_ALL)
+
+
 def FileEquals(expected_file, actual_file):
   '''
   Returns whether the two files are equal. Returns False if any of the files
   doesn't exist.
   '''
 
   if not os.path.isfile(actual_file) or not os.path.isfile(expected_file):
     return False
   return filecmp.cmp(expected_file, actual_file)
 
@@ skipping 67 matching lines @@
 
 
 def ZipDir(output, base_dir):
   '''Creates a zip file from a directory.'''
 
   base = os.path.join(base_dir, os.pardir)
   with zipfile.ZipFile(output, 'w') as out_file:
     for root, _, files in os.walk(base_dir):
       for in_file in files:
         out_file.write(os.path.join(root, in_file), base)