Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

Conversation

blasten
Copy link

@blasten blasten commented Apr 21, 2020

Wires up the external view embedder for Android, which just does the bare minimum work. It doesn't handle most of the features required in hybrid composition.

The external view embedder is used if AndroidView is constructed with compositeMode: AndroidViewCompositeMode.hybrid after flutter/flutter#55175 is merged.

@blasten
Copy link
Author

blasten commented Apr 21, 2020

The Android native shell doesn't have unit tests. I filed flutter/flutter#55269. I could add one unit test to check that GetExternalViewEmbedder returns a valid instance.

@blasten blasten marked this pull request as ready for review April 21, 2020 04:41
@auto-assign auto-assign bot requested a review from cbracken April 21, 2020 04:41
@blasten blasten requested a review from amirh April 21, 2020 04:42
@blasten blasten merged commit b1c51cf into flutter:master Apr 22, 2020
@blasten blasten deleted the android_external_view branch April 22, 2020 02:14
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Apr 22, 2020
dnfield pushed a commit to flutter/flutter that referenced this pull request Apr 22, 2020
* e83f30c Pass amber-files directory to tests. (flutter/engine#17815)

* b1c51cf Extend external view embedder on Android (flutter/engine#17839)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants