Fix Python imported modules depfile generation

build/android/gyp/util/build_utils.py

Index: build/android/gyp/util/build_utils.py
diff --git a/build/android/gyp/util/build_utils.py b/build/android/gyp/util/build_utils.py
index f69401bbb39f825981157dd125d45b63a0f67ad0..5aaf6f949c810c51c5867e8e35d60d6caaf9d4d7 100644
--- a/build/android/gyp/util/build_utils.py
+++ b/build/android/gyp/util/build_utils.py
@@ -338,9 +338,10 @@ def GetPythonDependencies():
                   if m is not None and hasattr(m, '__file__'))
 
   abs_module_paths = map(os.path.abspath, module_paths)
+  abs_root = os.path.abspath(CHROMIUM_SRC)

[agrieve, 2015/10/30 00:11:45]

I think it'd be even better to make CHROMIUM_SRC an absolute path so that it
doesn't become invalid when the cwd changes

[agrieve, 2015/10/30 00:25:06]

Actually - almost everywhere actually uses DIR_SOURCE_ROOT from:
https://code.google.com/p/chromium/codesearch#chromium/src/build/android/pyli...

Can you delete CHROMIUM_SRC in favour of DIR_SOURCE_ROOT? Seems only this file
and build/android/gyp/jinja_template.py use it.

[tsniatowski, 2015/10/30 00:31:21]

The problem is when someone goes and changes it back to a relative path this
will silently break. I think since this function's correctness relies on
comparing absolute paths, it should explicitly make sure they are absolute.

DIR_SOURCE_ROOT seems okay too, I'll change it (tomorrow) [but personally I'd
keep the explicit abspath here, or an os.path.isabs check]

 
   non_system_module_paths = [
-      p for p in abs_module_paths if p.startswith(CHROMIUM_SRC)]
+      p for p in abs_module_paths if p.startswith(abs_root)]
   def ConvertPycToPy(s):
     if s.endswith('.pyc'):
       return s[:-1]