New issue
Advanced search Search tips

Issue 602627 link

Starred by 4 users

Issue metadata

Status: Assigned
Owner:
Cc:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 3
Type: Bug



Sign in to add a comment

NewApi lint warning about <vector> even though they are now supported through the support library

Project Member Reported by dgn@chromium.org, Apr 12 2016

Issue description

chrome/android/java/res/drawable/ic_snippet_thumbnail_placeholder.xml:2 `<vector>` requires API level 21 (current min is 16): NewApi [warning]


From https://crrev.com/1871343006

The new code runs just fine on my KitKat test device. In the meantime I muted the warning in the drawable file, but I can't imagine it being a good long term solution, especially if we start replacing dpi-duplicated icons with Vector Drawables.

I tried searching online for similar issues, but I can't find much, which makes it look like it's not an issue for Android Studio users. Do we have to do something special other than suppressing those warnings then?
 
Owner: dgn@chromium.org
We have several lint checks completely disabled. Sounds like we want to do the same here. E.g.: https://code.google.com/p/chromium/codesearch#chromium/src/build/android/lint/suppressions.xml&q=suppr&sq=package:chromium&l=73 (note they are alphabetical in the file)
I don't think we should completely disable the lint check for vector's because depending on how it's used this could actually be a problem (e.g. used with a non-AppCompat ImageView). I think it'd be better to add a lint suppression for just the ic_snippet_thumbnail_placeholder.xml file.
Project Member

Comment 3 by sheriffbot@chromium.org, Apr 14 2017

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
This issue has been available for more than 365 days, and should be re-evaluated. Please re-triage this issue.
The Hotlist-Recharge-Cold label is applied for tracking purposes, and should not be removed after re-triaging the issue.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Comment 4 by dgn@chromium.org, Feb 26 2018

Cc: -jbudorick@chromium.org
Owner: jbudorick@chromium.org
Status: Assigned (was: Untriaged)
This bug has an owner, thus, it's been triaged. Changing status to "assigned".

Sign in to add a comment