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

Issues Search
    My Issues | Starred     Open | Closed | All

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

Issue 199733002: Cleanup AppMenu code. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Move lint suppression to AppMenuDragHelper Created 6 years, 9 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
« no previous file with comments | « no previous file | chrome/android/java/res/layout/menu_item.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 55 matching lines...) Expand 10 before | Expand all | Expand 10 after
66 <ignore path="PRODUCT_DIR/chrome_shell_test_apk/classes/org/chromium/printin g/PrintingControllerTest$7.class"/> 66 <ignore path="PRODUCT_DIR/chrome_shell_test_apk/classes/org/chromium/printin g/PrintingControllerTest$7.class"/>
67 <ignore path="PRODUCT_DIR/chrome_shell_test_apk/classes/org/chromium/printin g/PrintingControllerTest.class"/> 67 <ignore path="PRODUCT_DIR/chrome_shell_test_apk/classes/org/chromium/printin g/PrintingControllerTest.class"/>
68 <ignore path="PRODUCT_DIR/content_shell_test_apk/classes/org/chromium/conten t/browser/ClipboardTest.class"/> 68 <ignore path="PRODUCT_DIR/content_shell_test_apk/classes/org/chromium/conten t/browser/ClipboardTest.class"/>
69 <ignore path="android_webview/java/src/org/chromium/android_webview/AwPdfExp orter.java"/> 69 <ignore path="android_webview/java/src/org/chromium/android_webview/AwPdfExp orter.java"/>
70 <ignore path="android_webview/java/src/org/chromium/android_webview/AwPrintD ocumentAdapter.java"/> 70 <ignore path="android_webview/java/src/org/chromium/android_webview/AwPrintD ocumentAdapter.java"/>
71 <ignore path="android_webview/java/src/org/chromium/android_webview/AwWebCon tentsDelegateAdapter.java"/> 71 <ignore path="android_webview/java/src/org/chromium/android_webview/AwWebCon tentsDelegateAdapter.java"/>
72 <ignore path="android_webview/test/shell/src/org/chromium/android_webview/te st/AwTestContainerView.java"/> 72 <ignore path="android_webview/test/shell/src/org/chromium/android_webview/te st/AwTestContainerView.java"/>
73 <ignore path="base/android/java/src/org/chromium/base/ApiCompatibilityUtils. java"/> 73 <ignore path="base/android/java/src/org/chromium/base/ApiCompatibilityUtils. java"/>
74 <ignore path="chrome/android/java/src/org/chromium/chrome/browser/BookmarkUt ils.java"/> 74 <ignore path="chrome/android/java/src/org/chromium/chrome/browser/BookmarkUt ils.java"/>
75 <ignore path="chrome/android/java/src/org/chromium/chrome/browser/TtsPlatfor mImpl.java"/> 75 <ignore path="chrome/android/java/src/org/chromium/chrome/browser/TtsPlatfor mImpl.java"/>
76 <ignore path="chrome/android/java/src/org/chromium/chrome/browser/appmenu/Ap pMenu.java"/>
77 <ignore path="chrome/android/java/src/org/chromium/chrome/browser/infobar/An imationHelper.java"/> 76 <ignore path="chrome/android/java/src/org/chromium/chrome/browser/infobar/An imationHelper.java"/>
78 <ignore path="chrome/android/java/src/org/chromium/chrome/browser/signin/Sig ninManager.java"/> 77 <ignore path="chrome/android/java/src/org/chromium/chrome/browser/signin/Sig ninManager.java"/>
79 <ignore path="chrome/test/chromedriver/test/webview_shell/java/src/org/chrom ium/chromedriver_webview_shell/Main.java"/> 78 <ignore path="chrome/test/chromedriver/test/webview_shell/java/src/org/chrom ium/chromedriver_webview_shell/Main.java"/>
80 <ignore path="content/public/android/java/src/org/chromium/content/browser/C ontentViewCore.java"/> 79 <ignore path="content/public/android/java/src/org/chromium/content/browser/C ontentViewCore.java"/>
81 <ignore path="content/public/android/java/src/org/chromium/content/browser/G enericTouchGesture.java"/> 80 <ignore path="content/public/android/java/src/org/chromium/content/browser/G enericTouchGesture.java"/>
82 <ignore path="content/public/android/java/src/org/chromium/content/browser/V SyncMonitor.java"/> 81 <ignore path="content/public/android/java/src/org/chromium/content/browser/V SyncMonitor.java"/>
83 <ignore path="content/public/android/java/src/org/chromium/content/browser/a ccessibility/BrowserAccessibilityManager.java"/> 82 <ignore path="content/public/android/java/src/org/chromium/content/browser/a ccessibility/BrowserAccessibilityManager.java"/>
84 <ignore path="content/public/android/java/src/org/chromium/content/browser/a ccessibility/JellyBeanAccessibilityInjector.java"/> 83 <ignore path="content/public/android/java/src/org/chromium/content/browser/a ccessibility/JellyBeanAccessibilityInjector.java"/>
85 <ignore path="content/public/android/java/src/org/chromium/content/browser/a ccessibility/JellyBeanBrowserAccessibilityManager.java"/> 84 <ignore path="content/public/android/java/src/org/chromium/content/browser/a ccessibility/JellyBeanBrowserAccessibilityManager.java"/>
86 <ignore path="content/public/android/java/src/org/chromium/content/browser/a ccessibility/KitKatBrowserAccessibilityManager.java"/> 85 <ignore path="content/public/android/java/src/org/chromium/content/browser/a ccessibility/KitKatBrowserAccessibilityManager.java"/>
(...skipping 19 matching lines...) Expand all
106 </issue> 105 </issue>
107 <issue id="Recycle" severity="ignore"/> 106 <issue id="Recycle" severity="ignore"/>
108 <issue id="Registered" severity="ignore"/> 107 <issue id="Registered" severity="ignore"/>
109 <issue id="SdCardPath"> 108 <issue id="SdCardPath">
110 <ignore path="content/public/android/java/src/org/chromium/content/browser/M ediaResourceGetter.java"/> 109 <ignore path="content/public/android/java/src/org/chromium/content/browser/M ediaResourceGetter.java"/>
111 </issue> 110 </issue>
112 <issue id="SetJavaScriptEnabled" severity="ignore"/> 111 <issue id="SetJavaScriptEnabled" severity="ignore"/>
113 <issue id="ViewConstructor" severity="ignore"/> 112 <issue id="ViewConstructor" severity="ignore"/>
114 <issue id="WrongCall" severity="ignore"/> 113 <issue id="WrongCall" severity="ignore"/>
115 </lint> 114 </lint>
OLDNEW
« no previous file with comments | « no previous file | chrome/android/java/res/layout/menu_item.xml » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698