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

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

Issue 126453006: aw: Clean up android webview lint suppressions (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Remove .class from suppression Created 6 years, 11 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 | Annotate | Revision Log
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 11 matching lines...) Expand all
22 directory path (e.g. out/Debug) 22 directory path (e.g. out/Debug)
23 --> 23 -->
24 <issue id="AllowBackup"> 24 <issue id="AllowBackup">
25 <ignore path="AndroidManifest.xml"/> 25 <ignore path="AndroidManifest.xml"/>
26 </issue> 26 </issue>
27 <issue id="CommitPrefEdits"> 27 <issue id="CommitPrefEdits">
28 <ignore path="third_party/cacheinvalidation/src/java/com/google/ipc/invalida tion/ticl/android2/channel/AndroidChannelPreferences.java"/> 28 <ignore path="third_party/cacheinvalidation/src/java/com/google/ipc/invalida tion/ticl/android2/channel/AndroidChannelPreferences.java"/>
29 </issue> 29 </issue>
30 <issue id="DefaultLocale"> 30 <issue id="DefaultLocale">
31 <ignore path="PRODUCT_DIR/content_shell_test_apk/classes/org/chromium/conten t/browser/ViewportTest.class"/> 31 <ignore path="PRODUCT_DIR/content_shell_test_apk/classes/org/chromium/conten t/browser/ViewportTest.class"/>
32 <ignore path="android_webview/javatests/src/org/chromium/android_webview/tes t/AwViewportTest.java"/>
33 <ignore path="android_webview/javatests/src/org/chromium/android_webview/tes t/LoadUrlTest.java"/>
34 <ignore path="chrome/android/java/src/org/chromium/chrome/browser/net/spdypr oxy/DataReductionProxySettings.java"/> 32 <ignore path="chrome/android/java/src/org/chromium/chrome/browser/net/spdypr oxy/DataReductionProxySettings.java"/>
35 <ignore path="chrome/android/testshell/javatests/src/org/chromium/chrome/tes tshell/ChromiumTestShellUrlTest.java"/> 33 <ignore path="chrome/android/testshell/javatests/src/org/chromium/chrome/tes tshell/ChromiumTestShellUrlTest.java"/>
36 <ignore path="chrome/test/android/javatests/src/org/chromium/chrome/test/uti l/browser/sync/SyncTestUtil.java"/> 34 <ignore path="chrome/test/android/javatests/src/org/chromium/chrome/test/uti l/browser/sync/SyncTestUtil.java"/>
37 <ignore path="content/public/android/java/src/org/chromium/content/app/Linke r.java"/> 35 <ignore path="content/public/android/java/src/org/chromium/content/app/Linke r.java"/>
38 <ignore path="content/public/android/java/src/org/chromium/content/browser/L oadUrlParams.java"/> 36 <ignore path="content/public/android/java/src/org/chromium/content/browser/L oadUrlParams.java"/>
39 <ignore path="content/public/android/java/src/org/chromium/content/browser/a ccessibility/JellyBeanAccessibilityInjector.java"/> 37 <ignore path="content/public/android/java/src/org/chromium/content/browser/a ccessibility/JellyBeanAccessibilityInjector.java"/>
40 <ignore path="content/public/android/javatests/src/org/chromium/content/brow ser/ViewportTest.java"/> 38 <ignore path="content/public/android/javatests/src/org/chromium/content/brow ser/ViewportTest.java"/>
41 <ignore path="third_party/cacheinvalidation/src/java/com/google/ipc/invalida tion/external/client/contrib/AndroidListenerState.java"/> 39 <ignore path="third_party/cacheinvalidation/src/java/com/google/ipc/invalida tion/external/client/contrib/AndroidListenerState.java"/>
42 </issue> 40 </issue>
43 <issue id="DrawAllocation"> 41 <issue id="DrawAllocation">
(...skipping 25 matching lines...) Expand all
69 </issue> 67 </issue>
70 <issue id="MissingRegistered" severity="ignore"/> 68 <issue id="MissingRegistered" severity="ignore"/>
71 <issue id="MissingSuperCall"> 69 <issue id="MissingSuperCall">
72 <ignore path="android_webview/java/src/org/chromium/android_webview/AwConten ts.java"/> 70 <ignore path="android_webview/java/src/org/chromium/android_webview/AwConten ts.java"/>
73 <ignore path="content/public/android/java/src/org/chromium/content/browser/C ontentViewCore.java"/> 71 <ignore path="content/public/android/java/src/org/chromium/content/browser/C ontentViewCore.java"/>
74 </issue> 72 </issue>
75 <issue id="MissingVersion"> 73 <issue id="MissingVersion">
76 <ignore path="AndroidManifest.xml"/> 74 <ignore path="AndroidManifest.xml"/>
77 </issue> 75 </issue>
78 <issue id="NewApi"> 76 <issue id="NewApi">
79 <ignore path="PRODUCT_DIR/android_webview_apk/classes/org/chromium/android_w ebview/test/AwTestContainerView.class"/>
80 <ignore path="PRODUCT_DIR/chromium_testshell_test_apk/classes/org/chromium/p rinting/PrintingControllerTest$6.class"/> 77 <ignore path="PRODUCT_DIR/chromium_testshell_test_apk/classes/org/chromium/p rinting/PrintingControllerTest$6.class"/>
81 <ignore path="PRODUCT_DIR/chromium_testshell_test_apk/classes/org/chromium/p rinting/PrintingControllerTest.class"/> 78 <ignore path="PRODUCT_DIR/chromium_testshell_test_apk/classes/org/chromium/p rinting/PrintingControllerTest.class"/>
82 <ignore path="PRODUCT_DIR/content_shell_test_apk/classes/org/chromium/conten t/browser/ClipboardTest.class"/> 79 <ignore path="PRODUCT_DIR/content_shell_test_apk/classes/org/chromium/conten t/browser/ClipboardTest.class"/>
83 <ignore path="android_webview/java/src/org/chromium/android_webview/AwPdfExp orter.java"/> 80 <ignore path="android_webview/java/src/org/chromium/android_webview/AwPdfExp orter.java"/>
84 <ignore path="android_webview/java/src/org/chromium/android_webview/AwPrintD ocumentAdapter.java"/> 81 <ignore path="android_webview/java/src/org/chromium/android_webview/AwPrintD ocumentAdapter.java"/>
85 <ignore path="android_webview/java/src/org/chromium/android_webview/AwWebCon tentsDelegateAdapter.java"/> 82 <ignore path="android_webview/java/src/org/chromium/android_webview/AwWebCon tentsDelegateAdapter.java"/>
86 <ignore path="android_webview/test/shell/src/org/chromium/android_webview/te st/AwTestContainerView.java"/> 83 <ignore path="android_webview/test/shell/src/org/chromium/android_webview/te st/AwTestContainerView.java"/>
87 <ignore path="base/android/java/src/org/chromium/base/ApiCompatibilityUtils. java"/> 84 <ignore path="base/android/java/src/org/chromium/base/ApiCompatibilityUtils. java"/>
88 <ignore path="chrome/android/java/src/org/chromium/chrome/browser/BookmarkUt ils.java"/> 85 <ignore path="chrome/android/java/src/org/chromium/chrome/browser/BookmarkUt ils.java"/>
89 <ignore path="chrome/android/java/src/org/chromium/chrome/browser/TtsPlatfor mImpl.java"/> 86 <ignore path="chrome/android/java/src/org/chromium/chrome/browser/TtsPlatfor mImpl.java"/>
(...skipping 34 matching lines...) Expand 10 before | Expand all | Expand 10 after
124 <ignore path="chrome/android/javatests/src/org/chromium/chrome/browser/test/ ProviderBookmarkNodeTest.java"/> 121 <ignore path="chrome/android/javatests/src/org/chromium/chrome/browser/test/ ProviderBookmarkNodeTest.java"/>
125 <ignore path="content/public/test/android/javatests/src/org/chromium/content /browser/test/util/TestTouchUtils.java"/> 122 <ignore path="content/public/test/android/javatests/src/org/chromium/content /browser/test/util/TestTouchUtils.java"/>
126 <ignore path="content/public/test/android/javatests/src/org/chromium/content /browser/test/util/TouchCommon.java"/> 123 <ignore path="content/public/test/android/javatests/src/org/chromium/content /browser/test/util/TouchCommon.java"/>
127 </issue> 124 </issue>
128 <issue id="Registered" severity="ignore"/> 125 <issue id="Registered" severity="ignore"/>
129 <issue id="SdCardPath"> 126 <issue id="SdCardPath">
130 <ignore path="content/public/android/java/src/org/chromium/content/browser/M ediaResourceGetter.java"/> 127 <ignore path="content/public/android/java/src/org/chromium/content/browser/M ediaResourceGetter.java"/>
131 </issue> 128 </issue>
132 <issue id="SetJavaScriptEnabled" severity="ignore"/> 129 <issue id="SetJavaScriptEnabled" severity="ignore"/>
133 <issue id="UseSparseArrays"> 130 <issue id="UseSparseArrays">
134 <ignore path="android_webview/java/src/org/chromium/android_webview/AwQuotaM anagerBridge.java"/>
135 <ignore path="android_webview/java/src/org/chromium/android_webview/AwResour ce.java"/> 131 <ignore path="android_webview/java/src/org/chromium/android_webview/AwResour ce.java"/>
136 <ignore path="remoting/android/java/src/org/chromium/chromoting/TapGestureDe tector.java"/> 132 <ignore path="remoting/android/java/src/org/chromium/chromoting/TapGestureDe tector.java"/>
137 <ignore path="ui/android/java/src/org/chromium/ui/base/WindowAndroid.java"/> 133 <ignore path="ui/android/java/src/org/chromium/ui/base/WindowAndroid.java"/>
138 </issue> 134 </issue>
139 <issue id="ViewConstructor" severity="ignore"/> 135 <issue id="ViewConstructor" severity="ignore"/>
140 <issue id="WrongCall" severity="ignore"/> 136 <issue id="WrongCall" severity="ignore"/>
141 </lint> 137 </lint>
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698