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

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

Issue 2207103003: Removing unused resources from chrome/java/res (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Added comment about suppression Created 4 years, 4 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/java/res/drawable/notification_icon_bg.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 90 matching lines...) Expand 10 before | Expand all | Expand 10 after
101 <issue id="RtlCompat" severity="ignore"/> 101 <issue id="RtlCompat" severity="ignore"/>
102 <issue id="RtlEnabled" severity="ignore"/> 102 <issue id="RtlEnabled" severity="ignore"/>
103 <issue id="RtlSymmetry" severity="ignore"/> 103 <issue id="RtlSymmetry" severity="ignore"/>
104 <issue id="SdCardPath"> 104 <issue id="SdCardPath">
105 <ignore path="content/public/android/java/src/org/chromium/content/browser/M ediaResourceGetter.java"/> 105 <ignore path="content/public/android/java/src/org/chromium/content/browser/M ediaResourceGetter.java"/>
106 </issue> 106 </issue>
107 <issue id="SetJavaScriptEnabled" severity="ignore"/> 107 <issue id="SetJavaScriptEnabled" severity="ignore"/>
108 <issue id="UnusedResources"> 108 <issue id="UnusedResources">
109 <!-- Used by Android's policies system --> 109 <!-- Used by Android's policies system -->
110 <ignore regexp="restriction_values.xml" /> 110 <ignore regexp="restriction_values.xml" />
111
112 <ignore path="android_webview/tools/automated_ui_tests/java/res/" /> 111 <ignore path="android_webview/tools/automated_ui_tests/java/res/" />
112 <!-- This file isn't used if the target SDK version is less than 24 -->
Ted C 2016/08/05 20:48:33 maybe add a TODO to remove this suppression once w
estevenson 2016/08/05 21:21:45 Done.
113 <ignore path="chrome/android/java/res/xml/network_security_config.xml" />
113 </issue> 114 </issue>
114 <issue id="SignatureOrSystemPermissions" severity="ignore"/> 115 <issue id="SignatureOrSystemPermissions" severity="ignore"/>
115 <issue id="UnusedAttribute" severity="ignore"/> 116 <issue id="UnusedAttribute" severity="ignore"/>
116 <issue id="ViewConstructor" severity="ignore"/> 117 <issue id="ViewConstructor" severity="ignore"/>
117 <issue id="WrongCall" severity="ignore"/> 118 <issue id="WrongCall" severity="ignore"/>
118 <issue id="UselessParent"> 119 <issue id="UselessParent">
119 <ignore path="chrome/android/java/res/layout/data_reduction_promo_screen.xml " /> 120 <ignore path="chrome/android/java/res/layout/data_reduction_promo_screen.xml " />
120 </issue> 121 </issue>
121 </lint> 122 </lint>
OLDNEW
« no previous file with comments | « no previous file | chrome/android/java/res/drawable/notification_icon_bg.xml » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698