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

Unified Diff: talk/app/webrtc/java/lint/suppressions.xml

Issue 1412673008: Introduces Android API level linting, fixes all current API lint errors. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc@master
Patch Set: Add comments, rebase Created 5 years, 1 month 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 side-by-side diff with in-line comments
Download patch
Index: talk/app/webrtc/java/lint/suppressions.xml
diff --git a/talk/app/webrtc/java/lint/suppressions.xml b/talk/app/webrtc/java/lint/suppressions.xml
new file mode 100644
index 0000000000000000000000000000000000000000..7d1df4ca8ab5503a2d6db1ed02dcef1946622ec9
--- /dev/null
+++ b/talk/app/webrtc/java/lint/suppressions.xml
@@ -0,0 +1,18 @@
+<?xml version="1.0" encoding="utf-8"?>
kjellander_webrtc 2015/11/12 13:02:18 I think it makes more sense for us to have a globa
phoglund 2015/11/12 13:21:34 Done.
+<lint>
+ <issue id="NewApi"></issue>
+
+ <issue id="Locale" severity="ignore"/>
+ <issue id="SdCardPath" severity="ignore"/>
+ <issue id="UseValueOf" severity="ignore"/>
+ <issue id="InlinedApi" severity="ignore"/>
+ <issue id="DefaultLocale" severity="ignore"/>
+ <issue id="Assert" severity="ignore"/>
+ <issue id="UseSparseArrays" severity="ignore"/>
+
+ <!-- These are just from the dummy AndroidManifest.xml we use for linting.
+ That file is located at app/webrtc/java/lint/AndroidManifest.xml. -->
kjellander_webrtc 2015/11/12 13:02:18 If you agree with the above this line needs to be
phoglund 2015/11/12 13:21:34 Well, they're still needed, but the file will be i
+ <issue id="MissingApplicationIcon" severity="ignore"/>
+ <issue id="AllowBackup" severity="ignore"/>
+ <issue id="MissingVersion" severity="ignore"/>
+</lint>

Powered by Google App Engine
This is Rietveld 408576698