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

[web] DomManager for each FlutterView #47388

Merged
merged 2 commits into from Oct 27, 2023

Conversation

mdebbar
Copy link
Contributor

@mdebbar mdebbar commented Oct 27, 2023

The PR may seem large, but the main changes are simple:

  • Introduce a DomManager that aims to take over all DOM responsibilities from FlutterViewEmbedder.
  • Update all references to flutterViewEmbedder.*domElement* to domManager.*domElement*.
  • Describe the general DOM structure of a Flutter View in a doc comment.

Next steps (in future PRs):

  • Move all DOM manipulation methods out of FlutterViewEmbedder into DomManager.
  • Move DOM creation logic out of FlutterViewEmbedder into DomManager.

@github-actions github-actions bot added the platform-web Code specifically for the web engine label Oct 27, 2023
Copy link
Contributor

@harryterkelsen harryterkelsen left a comment

Choose a reason for hiding this comment

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

LGTM

@mdebbar mdebbar added the autosubmit Merge PR when tree becomes green via auto submit App label Oct 27, 2023
@auto-submit auto-submit bot merged commit 0dc05b1 into flutter:main Oct 27, 2023
24 checks passed
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Oct 28, 2023
auto-submit bot pushed a commit to flutter/flutter that referenced this pull request Oct 28, 2023
…137462)

flutter/engine@f1e30b4...a768211

2023-10-28 32242716+ricardoamador@users.noreply.github.com Add Android Emulator dependencies as "test_dependencies" to Android tests (flutter/engine#47384)
2023-10-27 mdebbar@google.com [web] DomManager for each FlutterView (flutter/engine#47388)
2023-10-27 skia-flutter-autoroll@skia.org Roll Skia from 77abd518ad4a to 64f13d5be3f7 (2 revisions) (flutter/engine#47406)

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC aaclarke@google.com,rmistry@google.com,zra@google.com on the revert to ensure that a human
is aware of the problem.

To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
@mdebbar mdebbar deleted the view_dom_manager branch November 17, 2023 17:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
autosubmit Merge PR when tree becomes green via auto submit App platform-web Code specifically for the web engine
Projects
None yet
2 participants