- Notifications
You must be signed in to change notification settings - Fork6k
Don't use rFD in pre-Q versions#28593
Conversation
shell/platform/android/io/flutter/embedding/android/FlutterActivity.java OutdatedShow resolvedHide resolved
Uh oh!
There was an error while loading.Please reload this page.
| // permissions in some Android versions. | ||
| // * rFD behavior isn't tested on pre-Q versions. | ||
| // See https://github.com/flutter/flutter/issues/46172, and | ||
| // https://github.com/flutter/flutter/issues/88767. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
nit: consider adding that there are vendor-specific bugs.
How confident are we that this API works on all vendors after Q?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
I only know that a CTS test was added. Vendors should be running these tests, but we don't really know.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
SGTM
dnfield left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
LGTM with nit.
I am wondering if we know for sure that the Android team's coverage of reportFullyDrawn is good enough to trust it on all vendors after Q.
blasten commentedSep 13, 2021
Android usesCTS for this. |
* 'Update Dart SDK to3300f32'* Don't use rFD in pre-Q versions (#28593)* updated 3rd party license golden* hard-code Build.VERSION_CODE.S* renew cirrus key (#28584)Co-authored-by: Emmanuel Garcia <egarciad@google.com>Co-authored-by: Christopher Fujino <christopherfujino@gmail.com>Co-authored-by: keyonghan <54558023+keyonghan@users.noreply.github.com>
Fixesflutter/flutter#88767
We cannot safely rely on this API since there's no test in Android prior to Q.