Magic-Mirror/gecko-dev
1
0
Fork 0
mirror of https://github.com/mozilla/gecko-dev.git synced 2024-10-25 19:25:43 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Bug 1365878 - Change severity of NewApi lint checks from warning to error; r=nalexander

Browse Source 
Change severity of newapi to error and white list current not fixed newapi problems
So future unchecked NewApi issues will be blocked

MozReview-Commit-ID: LglV50mILn4

--HG--
extra : rebase_source : 9191d61c3cc934212153346e017e1c37c1972b20
This commit is contained in:
Teng-pao Yu 2017-05-16 17:10:10 +08:00
parent 6c54dea640
commit 23032a9245
1 changed files with 24 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

25
mobile/android/app/lint.xml
View File

@ -33,7 +33,6 @@
<issue id="Instantiatable" severity="warning" />
<issue id="LongLogTag" severity="warning" />
<issue id="MissingPermission" severity="warning" />
<issue id="NewApi" severity="warning" />
<issue id="OnClick" severity="warning" />
<issue id="ReferenceType" severity="warning" />
<issue id="ResourceAsColor" severity="warning" />
@ -41,6 +40,30 @@
<issue id="ValidFragment" severity="warning" />
<issue id="WrongConstant" severity="warning" />
<!-- Fixes are in progress but we would like to block future candidates.
The ** in the path are wildcards. We need these wildcards to not change our code structure.
See: http://stackoverflow.com/questions/43994420/what-path-is-the-issue-ignore-path-element-in-lint-xml-relative-to -->
<issue id="NewApi" severity="error">
<ignore path="**/mozilla-central/mobile/android/base/java/org/mozilla/gecko/activitystream/ActivityStreamPreference.java"/>
<ignore path="**/mozilla-central/mobile/android/base/java/org/mozilla/gecko/dlc/DownloadContentTelemetry.java"/>
<ignore path="**/mozilla-central/mobile/android/base/java/org/mozilla/gecko/preferences/LocaleListPreference.java"/>
<ignore path="**/mozilla-central/mobile/android/base/java/org/mozilla/gecko/media/MediaControlService.java"/>
<ignore path="**/mozilla-central/mobile/android/base/java/org/mozilla/gecko/db/PerProfileDatabaseProvider.java"/>
<ignore path="**/mozilla-central/mobile/android/base/java/org/mozilla/gecko/PrintHelper.java"/>
<ignore path="**/mozilla-central/mobile/android/base/java/org/mozilla/gecko/animation/PropertyAnimator.java"/>
<ignore path="**/mozilla-central/mobile/android/base/java/org/mozilla/gecko/RemotePresentationService.java"/>
<ignore path="**/mozilla-central/mobile/android/base/java/org/mozilla/gecko/db/SharedBrowserDatabaseProvider.java"/>
<ignore path="**/mozilla-central/mobile/android/base/java/org/mozilla/gecko/toolbar/TabCounter.java"/>
<ignore path="**/mozilla-central/mobile/android/base/java/org/mozilla/gecko/tabs/TabPanelBackButton.java"/>
<ignore path="**/mozilla-central/mobile/android/base/java/org/mozilla/gecko/toolbar/ToolbarEditText.java"/>
<ignore path="**/mozilla-central/mobile/android/base/java/org/mozilla/gecko/util/ViewUtil.java"/>
<ignore path="**/mozilla-central/mobile/android/base/java/org/mozilla/gecko/webapps/WebAppActivity.java"/>
<ignore path="**/mozilla-central/media/webrtc/trunk/webrtc/modules/audio_device/android/java/src/org/webrtc/voiceengine/WebRtcAudioEffects.java"/>
<ignore path="**/mozilla-central/media/webrtc/trunk/webrtc/modules/audio_device/android/java/src/org/webrtc/voiceengine/WebRtcAudioRecord.java"/>
<ignore path="**/mozilla-central/mobile/android/base/resources/values/styles.xml"/>
<ignore path="**/mozilla-central/mobile/android/base/resources/values/themes.xml"/>
</issue>
<!-- We fixed all "Registered" lint errors. However the current gradle plugin has a bug where
it ignores @SuppressLint annotations for this check. See CrashReporter class and
https://code.google.com/p/android/issues/detail?id=204846 -->