diff options
author | agrieve <agrieve@chromium.org> | 2016-03-23 17:20:00 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2016-03-24 00:21:24 +0000 |
commit | 26befa044b2ce9ed1b5dfd83eb5a67d1a964bb8c (patch) | |
tree | 5c490176b7e49cfd77fb06237f1f972022c4e3d6 | |
parent | 5c1bca05189d7e8024b848e25ecb2813b53b6056 (diff) | |
download | chromium_src-26befa044b2ce9ed1b5dfd83eb5a67d1a964bb8c.zip chromium_src-26befa044b2ce9ed1b5dfd83eb5a67d1a964bb8c.tar.gz chromium_src-26befa044b2ce9ed1b5dfd83eb5a67d1a964bb8c.tar.bz2 |
Stop lint.py pulling in implicit dependencies via AndroidManifest.xml
This will mean even fewer UnusedResource checks, but that is covered by:
https://bugs.chromium.org/p/chromium/issues/detail?id=595810
BUG=597007
Review URL: https://codereview.chromium.org/1826563002
Cr-Commit-Position: refs/heads/master@{#382986}
-rwxr-xr-x | build/android/gyp/lint.py | 10 | ||||
-rw-r--r-- | chrome/test/chromedriver/test/webview_shell/java/AndroidManifest.xml | 2 | ||||
-rw-r--r-- | chrome/test/chromedriver/test/webview_shell/java/res/mipmap/icon.png (renamed from chrome/test/chromedriver/test/webview_shell/java/res/drawable/icon.png) | bin | 718 -> 718 bytes |
3 files changed, 9 insertions, 3 deletions
diff --git a/build/android/gyp/lint.py b/build/android/gyp/lint.py index 20484f7..71c1a6f 100755 --- a/build/android/gyp/lint.py +++ b/build/android/gyp/lint.py @@ -107,7 +107,6 @@ def _OnStaleMd5(changes, lint_path, config_path, processed_config_path, new_dir = os.path.join(temp_dir, str(len(src_dirs))) os.mkdir(new_dir) src_dirs.append(new_dir) - cmd.extend(['--sources', _RelativizePath(new_dir)]) return new_dir def PathInDir(d, src): @@ -121,10 +120,17 @@ def _OnStaleMd5(changes, lint_path, config_path, processed_config_path, break if not src_dir: src_dir = NewSourceDir() + cmd.extend(['--sources', _RelativizePath(src_dir)]) os.symlink(os.path.abspath(src), PathInDir(src_dir, src)) + # Put the manifest in a temporary directory in order to avoid lint detecting + # sibling res/ and src/ directories (which should be pass explicitly if they + # are to be included). if manifest_path: - cmd.append(_RelativizePath(os.path.join(manifest_path, os.pardir))) + src_dir = NewSourceDir() + os.symlink(os.path.abspath(manifest_path), + PathInDir(src_dir, manifest_path)) + cmd.append(src_dir) if os.path.exists(result_path): os.remove(result_path) diff --git a/chrome/test/chromedriver/test/webview_shell/java/AndroidManifest.xml b/chrome/test/chromedriver/test/webview_shell/java/AndroidManifest.xml index 3703b70..32070d0c 100644 --- a/chrome/test/chromedriver/test/webview_shell/java/AndroidManifest.xml +++ b/chrome/test/chromedriver/test/webview_shell/java/AndroidManifest.xml @@ -14,7 +14,7 @@ <uses-permission android:name="android.permission.READ_EXTERNAL_STORAGE" /> <application android:label="ChromeDriverWebviewShell" - android:icon="@drawable/icon" + android:icon="@mipmap/icon" android:allowBackup="false"> <activity android:name=".Main" android:launchMode="singleTask" diff --git a/chrome/test/chromedriver/test/webview_shell/java/res/drawable/icon.png b/chrome/test/chromedriver/test/webview_shell/java/res/mipmap/icon.png Binary files differindex d2b0293..d2b0293 100644 --- a/chrome/test/chromedriver/test/webview_shell/java/res/drawable/icon.png +++ b/chrome/test/chromedriver/test/webview_shell/java/res/mipmap/icon.png |