Fixed
Status Update
Comments
nj...@google.com <nj...@google.com>
ap...@google.com <ap...@google.com> #2
Project: platform/frameworks/support
Branch: androidx-master-dev
commit b90079595f33f58fece04026a97faa0d243acdb1
Author: Yuichi Araki <yaraki@google.com>
Date: Wed Sep 18 16:55:49 2019
Change the way to detect mismatch between POJO and query
This fixes cursor mismatch warnings with expandProjection.
Bug: 140759491
Test: QueryMethodProcessorTest
Change-Id: I7659002e5e0d1ef60fc1af2a625c0c36da0664d8
M room/compiler/src/main/kotlin/androidx/room/processor/QueryMethodProcessor.kt
M room/compiler/src/main/kotlin/androidx/room/solver/TypeAdapterStore.kt
M room/compiler/src/main/kotlin/androidx/room/solver/query/result/PojoRowAdapter.kt
M room/compiler/src/test/kotlin/androidx/room/processor/QueryMethodProcessorTest.kt
M room/compiler/src/test/kotlin/androidx/room/testing/TestProcessor.kt
https://android-review.googlesource.com/1123258
https://goto.google.com/android-sha1/b90079595f33f58fece04026a97faa0d243acdb1
Branch: androidx-master-dev
commit b90079595f33f58fece04026a97faa0d243acdb1
Author: Yuichi Araki <yaraki@google.com>
Date: Wed Sep 18 16:55:49 2019
Change the way to detect mismatch between POJO and query
This fixes cursor mismatch warnings with expandProjection.
Bug: 140759491
Test: QueryMethodProcessorTest
Change-Id: I7659002e5e0d1ef60fc1af2a625c0c36da0664d8
M room/compiler/src/main/kotlin/androidx/room/processor/QueryMethodProcessor.kt
M room/compiler/src/main/kotlin/androidx/room/solver/TypeAdapterStore.kt
M room/compiler/src/main/kotlin/androidx/room/solver/query/result/PojoRowAdapter.kt
M room/compiler/src/test/kotlin/androidx/room/processor/QueryMethodProcessorTest.kt
M room/compiler/src/test/kotlin/androidx/room/testing/TestProcessor.kt
Description
Current incorrect behavior
The link to source code ).
painterResource()
currently only supports inflatingandroid:tint
if it references a simpleCOLOR_INT
resource type, meaning thatColorStateList
s or theme attributes pointing toColorStateList
s are simply ignored (Desired correct behavior
android:tint
should inflate the color state list usingTypedArrayUtils#getNamedColor()
instead, which would default to theColorStateList
's default value (sinceCompose
does not have any understanding of drawable states used withView
s).This behavior would mirror the way that here , which returns a here ).
ColorStateList
s are currently handled in a<path>
'sandroid:fillColor
andandroid:strokeColor
, both of which fallback to theColorStateList
's default color (they inflate usingTypedArrayUtils#getNamedComplexColor()
ComplexColor
wrapper around aColorStateList
. Then when the vector is rendered to the display, a simpleSolidColor
using the color state list's default color is returnedThis would closely mirror the way that source code here ).
VectorDrawableCompat
is inflated, which usesTypedArrayUtils#getNamedColorStateList()
(Reasons for fixing the bug
android:fillColor
andandroid:strokeColor
currently support color state lists butandroid:tint
does not. Ideally they should both be supported if possible.Many teams adopting Compose don't have an Icon solution as elegant as Material's (which generates
ImageVector
s as part of a separate artifact automatically). For example, our team, plans to continue inflating icons from our existing icon library defined in XML resources while adopting Compose, and all of these resources currently declare a default tint color usingandroid:tint
.Version used: Compose v1.0.1