Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(39)

Side by Side Diff: build/android/lint/suppressions.xml

Issue 1206323004: Remove INJECT_EVENTS permissions from test APKs. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: don't touch src/mojo Created 5 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | chrome/android/javatests/AndroidManifest.xml » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 <?xml version="1.0" encoding="utf-8"?> 1 <?xml version="1.0" encoding="utf-8"?>
2 <lint> 2 <lint>
3 <!-- 3 <!--
4 STOP! It looks like you want to suppress some lint errors: 4 STOP! It looks like you want to suppress some lint errors:
5 - Have you tried identifing the offending patch? 5 - Have you tried identifing the offending patch?
6 Ask the author for a fix and/or revert the patch. 6 Ask the author for a fix and/or revert the patch.
7 - It is preferred to add suppressions in the code instead of 7 - It is preferred to add suppressions in the code instead of
8 sweeping it under the rug here. See: 8 sweeping it under the rug here. See:
9 9
10 http://developer.android.com/tools/debugging/improving-w-lint.html 10 http://developer.android.com/tools/debugging/improving-w-lint.html
(...skipping 65 matching lines...) Expand 10 before | Expand all | Expand 10 after
76 <ignore path="org/chromium/base/JavaHandlerThread.class"/> 76 <ignore path="org/chromium/base/JavaHandlerThread.class"/>
77 <ignore path="org/chromium/base/SysUtils.class"/> 77 <ignore path="org/chromium/base/SysUtils.class"/>
78 <ignore path="org/chromium/chrome/browser/TtsPlatformImpl.class"/> 78 <ignore path="org/chromium/chrome/browser/TtsPlatformImpl.class"/>
79 <ignore path="org/chromium/chrome/browser/TtsPlatformImpl$*.class"/> 79 <ignore path="org/chromium/chrome/browser/TtsPlatformImpl$*.class"/>
80 <ignore path="chrome/android/java/res/values-v17/styles.xml"/> 80 <ignore path="chrome/android/java/res/values-v17/styles.xml"/>
81 </issue> 81 </issue>
82 <issue id="OldTargetApi"> 82 <issue id="OldTargetApi">
83 <ignore path="AndroidManifest.xml"/> 83 <ignore path="AndroidManifest.xml"/>
84 </issue> 84 </issue>
85 <issue id="Overdraw" severity="ignore"/> 85 <issue id="Overdraw" severity="ignore"/>
86 <!-- TODO(newt): remove if INJECT_EVENTS is not needed crbug.com/502028 -->
87 <issue id="ProtectedPermissions" severity="ignore"/>
88 <issue id="Recycle" severity="ignore"/> 86 <issue id="Recycle" severity="ignore"/>
89 <issue id="Registered" severity="ignore"/> 87 <issue id="Registered" severity="ignore"/>
90 <issue id="RtlCompat" severity="ignore"/> 88 <issue id="RtlCompat" severity="ignore"/>
91 <issue id="RtlEnabled" severity="ignore"/> 89 <issue id="RtlEnabled" severity="ignore"/>
92 <issue id="RtlSymmetry" severity="ignore"/> 90 <issue id="RtlSymmetry" severity="ignore"/>
93 <issue id="SdCardPath"> 91 <issue id="SdCardPath">
94 <ignore path="content/public/android/java/src/org/chromium/content/browser/M ediaResourceGetter.java"/> 92 <ignore path="content/public/android/java/src/org/chromium/content/browser/M ediaResourceGetter.java"/>
95 </issue> 93 </issue>
96 <issue id="SetJavaScriptEnabled" severity="ignore"/> 94 <issue id="SetJavaScriptEnabled" severity="ignore"/>
97 <issue id="UnusedResources"> 95 <issue id="UnusedResources">
98 <!-- 96 <!--
99 TODO(pkotwicz): Re-enable after the chrome_java and chrome_staging_java 97 TODO(pkotwicz): Re-enable after the chrome_java and chrome_staging_java
100 targets have been merged. crbug.com/484934 98 targets have been merged. crbug.com/484934
101 --> 99 -->
102 <ignore path="chrome/android/java/res" /> 100 <ignore path="chrome/android/java/res" />
103 <ignore path="content/shell/android/shell_apk/res/layout/content_shell_activ ity.xml" /> 101 <ignore path="content/shell/android/shell_apk/res/layout/content_shell_activ ity.xml" />
104 <ignore path="content/shell/android/shell_apk/res/values/strings.xml" /> 102 <ignore path="content/shell/android/shell_apk/res/values/strings.xml" />
105 </issue> 103 </issue>
106 <issue id="SignatureOrSystemPermissions" severity="ignore"/> 104 <issue id="SignatureOrSystemPermissions" severity="ignore"/>
107 <issue id="UnusedAttribute" severity="ignore"/> 105 <issue id="UnusedAttribute" severity="ignore"/>
108 <issue id="ViewConstructor" severity="ignore"/> 106 <issue id="ViewConstructor" severity="ignore"/>
109 <issue id="WrongCall" severity="ignore"/> 107 <issue id="WrongCall" severity="ignore"/>
110 </lint> 108 </lint>
OLDNEW
« no previous file with comments | « no previous file | chrome/android/javatests/AndroidManifest.xml » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698