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

Conversation

@iskakaushik
Copy link
Contributor

@iskakaushik iskakaushik commented Nov 3, 2020

This decouples surfaces from view embedders.

@iskakaushik
Copy link
Contributor Author

// note to self, local branch ios-remove-eve-comp.

@iskakaushik
Copy link
Contributor Author

This is now ready for review @gaaclarke , @chinmaygarde

Copy link
Member

@gaaclarke gaaclarke left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

This decouples surfaces from view embedders.
@iskakaushik iskakaushik force-pushed the surface-remove-get-external-view-embedder branch from d6c59e0 to 8dfe774 Compare November 11, 2020 20:49
@iskakaushik iskakaushik added the waiting for tree to go green This PR is approved and tested, but waiting for the tree to be green to land. label Nov 11, 2020
@iskakaushik iskakaushik merged commit 5419f70 into flutter:master Nov 11, 2020
@iskakaushik iskakaushik deleted the surface-remove-get-external-view-embedder branch November 11, 2020 21:45
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Nov 12, 2020
iskakaushik added a commit to iskakaushik/engine that referenced this pull request Nov 12, 2020
zanderso pushed a commit that referenced this pull request Nov 12, 2020
…edder (#22468)

* Revert "Remove GetExternalViewEmbedder from surface (#22272)"

This reverts commit 5419f70.

* Revert "Rasterizer is initialized with an external view embedder (#22405)"

This reverts commit f95df42.
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Nov 13, 2020
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Nov 13, 2020
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Nov 13, 2020
chaselatta pushed a commit to chaselatta/engine that referenced this pull request Nov 30, 2020
This decouples surfaces from view embedders.
chaselatta pushed a commit to chaselatta/engine that referenced this pull request Nov 30, 2020
…edder (flutter#22468)

* Revert "Remove GetExternalViewEmbedder from surface (flutter#22272)"

This reverts commit 5419f70.

* Revert "Rasterizer is initialized with an external view embedder (flutter#22405)"

This reverts commit f95df42.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla: yes platform-android platform-ios waiting for tree to go green This PR is approved and tested, but waiting for the tree to be green to land.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants