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

Conversation

arbreng
Copy link
Contributor

@arbreng arbreng commented Oct 20, 2021

Add the ability for FakeFlatland to handle View, Transform, and Content operations.
Add flatland_external_view_embedder_unittests.cc, which uses FakeFlatland's new ability in order to test Flutter's graph-production code.

Test: flatland_external_view_embedder_unittests
Bug: https://bugs.fuchsia.dev/p/fuchsia/issues/detail?id=85619
Bug: https://bugs.fuchsia.dev/p/fuchsia/issues/detail?id=85620

@uysalere @freiling

@arbreng
Copy link
Contributor Author

arbreng commented Oct 20, 2021

Still a little WIP.

TODO before I can land:
+Fix the last compile error
+Uncomment the tests in fuchsia_external_view_embedder_unittests and get them working

Copy link
Contributor

@akbiggs akbiggs left a comment

Choose a reason for hiding this comment

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

Thanks for adding me to this, it was a really fun way to learn more about Flatland's protocol. Sorry for the super slow review.

@arbreng arbreng force-pushed the fuchsia-fake-flatland branch from fabf2dc to b9c82d0 Compare October 29, 2021 18:15
@arbreng arbreng marked this pull request as ready for review October 29, 2021 18:38
@arbreng arbreng merged commit b6f773a into flutter:master Oct 29, 2021
@arbreng arbreng deleted the fuchsia-fake-flatland branch October 29, 2021 19:43
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Oct 29, 2021
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Oct 30, 2021
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Oct 30, 2021
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Oct 30, 2021
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Oct 30, 2021
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Oct 30, 2021
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Oct 30, 2021
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Oct 30, 2021
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Oct 30, 2021
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Oct 30, 2021
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Oct 30, 2021
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Oct 31, 2021
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Oct 31, 2021
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Oct 31, 2021
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Oct 31, 2021
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Oct 31, 2021
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Oct 31, 2021
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Nov 1, 2021
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Nov 1, 2021
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Nov 1, 2021
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Nov 1, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants