Fixed
Status Update
Comments
ap...@google.com <ap...@google.com> #3
Project: platform/frameworks/support
Branch: androidx-master-dev
commit c1d450705c6ca493f17ced6566fc244ee34fa3b8
Author: jbwoods <jbwoods@google.com>
Date: Mon Feb 04 17:16:34 2019
Cache @ContentView annotation lookups for activities
Caching the annotation look up so the reflection cost only
needs to be paid once.
This provides a performance benefit.
Test: tested with local activity unit tests
BUG: 123709449
Change-Id: I1dad04cefd9de50aac08e60882c8837472f42b55
M activity/src/main/java/androidx/activity/ComponentActivity.java
https://android-review.googlesource.com/895674
https://goto.google.com/android-sha1/c1d450705c6ca493f17ced6566fc244ee34fa3b8
Branch: androidx-master-dev
commit c1d450705c6ca493f17ced6566fc244ee34fa3b8
Author: jbwoods <jbwoods@google.com>
Date: Mon Feb 04 17:16:34 2019
Cache @ContentView annotation lookups for activities
Caching the annotation look up so the reflection cost only
needs to be paid once.
This provides a performance benefit.
Test: tested with local activity unit tests
BUG: 123709449
Change-Id: I1dad04cefd9de50aac08e60882c8837472f42b55
M activity/src/main/java/androidx/activity/ComponentActivity.java
ap...@google.com <ap...@google.com> #4
Project: platform/frameworks/support
Branch: androidx-master-dev
commit c636f8b7c7e081992813b51e7bd07e5fa47525a1
Author: jbwoods <jbwoods@google.com>
Date: Mon Feb 04 18:31:00 2019
Cache @ContentView annotation lookups for fragments
Caching the annotation look up so the reflection cost only
needs to be paid once.
This provides a performance benefit.
Test: tested with local fragment unit tests
BUG: 123709449
Change-Id: I09a2ee1268c6893ac7bb68f1f74a8bfa5cdf0f16
M fragment/src/main/java/androidx/fragment/app/Fragment.java
https://android-review.googlesource.com/895296
https://goto.google.com/android-sha1/c636f8b7c7e081992813b51e7bd07e5fa47525a1
Branch: androidx-master-dev
commit c636f8b7c7e081992813b51e7bd07e5fa47525a1
Author: jbwoods <jbwoods@google.com>
Date: Mon Feb 04 18:31:00 2019
Cache @ContentView annotation lookups for fragments
Caching the annotation look up so the reflection cost only
needs to be paid once.
This provides a performance benefit.
Test: tested with local fragment unit tests
BUG: 123709449
Change-Id: I09a2ee1268c6893ac7bb68f1f74a8bfa5cdf0f16
M fragment/src/main/java/androidx/fragment/app/Fragment.java
il...@google.com <il...@google.com> #5
This has been fixed internally and will be available in Fragment 1.1.0-alpha05
Description
Version used: 1.0.0-alpha04, 1.1.0-alpha04, respectively
As the @ContentView annotation lookup is done at runtime, it would be a performance benefit to cache the annotation lookup so that you only need to pay the reflection cost once.
This is particularly important for Fragments where onCreateView() can run multiple times (i.e., when coming back from the back stack).