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

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

Issue 880083004: Change which manifest and resources are passed to lint.py. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Added resource_dir to chrome_java target. Created 5 years, 10 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
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 23 matching lines...) Expand all
34 <issue id="DrawAllocation"> 34 <issue id="DrawAllocation">
35 <ignore path="content/public/android/java/src/org/chromium/content/browser/C ontentViewRenderView.java"/> 35 <ignore path="content/public/android/java/src/org/chromium/content/browser/C ontentViewRenderView.java"/>
36 <ignore path="content/public/android/java/src/org/chromium/content/browser/P opupZoomer.java"/> 36 <ignore path="content/public/android/java/src/org/chromium/content/browser/P opupZoomer.java"/>
37 </issue> 37 </issue>
38 <issue id="ExportedContentProvider"> 38 <issue id="ExportedContentProvider">
39 <ignore path="AndroidManifest.xml"/> 39 <ignore path="AndroidManifest.xml"/>
40 </issue> 40 </issue>
41 <issue id="HandlerLeak"> 41 <issue id="HandlerLeak">
42 <ignore path="remoting/android/java/src/org/chromium/chromoting/TapGestureDe tector.java"/> 42 <ignore path="remoting/android/java/src/org/chromium/chromoting/TapGestureDe tector.java"/>
43 </issue> 43 </issue>
44 <issue id="InconsistentLayout" severity="ignore"/>
45 <issue id="InflateParams" severity="ignore"/>
44 <issue id="MissingApplicationIcon" severity="ignore"/> 46 <issue id="MissingApplicationIcon" severity="ignore"/>
45 <issue id="MissingRegistered" severity="ignore"/> 47 <issue id="MissingRegistered" severity="ignore"/>
46 <issue id="MissingVersion"> 48 <issue id="MissingVersion">
47 <ignore path="AndroidManifest.xml"/> 49 <ignore path="AndroidManifest.xml"/>
48 </issue> 50 </issue>
49 <!-- Disabling is InlinedApi and NewApi is bad but we have too many of these e rrors and nobody is fixing it. crbug.com/411461 --> 51 <!-- Disabling is InlinedApi and NewApi is bad but we have too many of these e rrors and nobody is fixing it. crbug.com/411461 -->
50 <issue id="InlinedApi" severity="ignore"/> 52 <issue id="InlinedApi" severity="ignore"/>
51 <issue id="NewApi" severity="ignore"/> 53 <issue id="NewApi" severity="ignore"/>
52 <issue id="OldTargetApi"> 54 <issue id="OldTargetApi">
53 <ignore path="AndroidManifest.xml"/> 55 <ignore path="AndroidManifest.xml"/>
54 </issue> 56 </issue>
55 <issue id="Recycle" severity="ignore"/> 57 <issue id="Recycle" severity="ignore"/>
56 <issue id="Registered" severity="ignore"/> 58 <issue id="Registered" severity="ignore"/>
59 <issue id="RtlCompat" severity="ignore"/>
60 <issue id="RtlSymmetry" severity="ignore"/>
57 <issue id="SdCardPath"> 61 <issue id="SdCardPath">
58 <ignore path="content/public/android/java/src/org/chromium/content/browser/M ediaResourceGetter.java"/> 62 <ignore path="content/public/android/java/src/org/chromium/content/browser/M ediaResourceGetter.java"/>
59 </issue> 63 </issue>
60 <issue id="SetJavaScriptEnabled" severity="ignore"/> 64 <issue id="SetJavaScriptEnabled" severity="ignore"/>
65 <issue id="SignatureOrSystemPermissions" severity="ignore"/>
66 <issue id="UnusedAttribute" severity="ignore"/>
61 <issue id="ViewConstructor" severity="ignore"/> 67 <issue id="ViewConstructor" severity="ignore"/>
62 <issue id="WrongCall" severity="ignore"/> 68 <issue id="WrongCall" severity="ignore"/>
63 </lint> 69 </lint>
OLDNEW
« no previous file with comments | « build/android/gyp/lint.py ('k') | build/android/lint_action.gypi » ('j') | chrome/chrome.gyp » ('J')

Powered by Google App Engine
This is Rietveld 408576698