Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adds a platfromViewId to SemanticsNode #8055

Merged
merged 1 commit into from
Mar 11, 2019

Conversation

goderbauer
Copy link
Member

Framework-side changes to follow.

Copy link
Contributor

@amirh amirh left a comment

Choose a reason for hiding this comment

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

LGTM

Thanks!

lib/ui/semantics/semantics_node.h Show resolved Hide resolved
@amirh
Copy link
Contributor

amirh commented Mar 6, 2019

flutter/flutter#19418

@goderbauer goderbauer merged commit bd0f908 into flutter:master Mar 11, 2019
@goderbauer goderbauer deleted the platformViewId branch March 11, 2019 18:30
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Mar 11, 2019
engine-flutter-autoroll added a commit to flutter/flutter that referenced this pull request Mar 12, 2019
flutter/engine@a71609f...56d8634

git log a71609f..56d8634 --no-merges --oneline
56d8634 Improve shadow doc in PhysicalShapeLayer (flutter/engine#8114)
bd0f908 Adds a platfromViewId to SemanticsNode (flutter/engine#8055)

The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, please contact the current sheriff (mklim@google.com), and stop
the roller if necessary.
RBogie pushed a commit to RBogie/flutter-engine that referenced this pull request Apr 8, 2019
RBogie added a commit to RBogie/flutter-engine that referenced this pull request Apr 8, 2019
RBogie added a commit to RBogie/flutter-engine that referenced this pull request Apr 8, 2019
cbracken added a commit to cbracken/flutter_engine that referenced this pull request Oct 25, 2019
This exposes platform_view_id on the embedder API's FlutterSemanticNode.

In bd0f908 (flutter#8055), platformViewId was
added to SemanticsNode. This field is non-zero when the SemanticsNode
represents a platform view and is typically used by embedders as a means
of identifying locations where a platform view's 'native' accessibility
tree should be injected into the platform-specific accessibility tree
constructed by the embedder.

Due to the intended use of this field, the Flutter framework is meant to
enforce that this node has a child count of zero.
cbracken added a commit to cbracken/flutter_engine that referenced this pull request Oct 25, 2019
This exposes platform_view_id on the embedder API's FlutterSemanticNode.

In bd0f908 (flutter#8055), platformViewId was
added to SemanticsNode. This field is non-zero when the SemanticsNode
represents a platform view and is typically used by embedders as a means
of identifying locations where a platform view's 'native' accessibility
tree should be injected into the platform-specific accessibility tree
constructed by the embedder.

Due to the intended use of this field, the Flutter framework is meant to
enforce that this node has a child count of zero.
cbracken added a commit that referenced this pull request Oct 25, 2019
This exposes platform_view_id on the embedder API's FlutterSemanticNode.

In bd0f908 (#8055), platformViewId was
added to SemanticsNode. This field is non-zero when the SemanticsNode
represents a platform view and is typically used by embedders as a means
of identifying locations where a platform view's 'native' accessibility
tree should be injected into the platform-specific accessibility tree
constructed by the embedder.

Due to the intended use of this field, the Flutter framework is meant to
enforce that this node has a child count of zero.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
3 participants