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

[Android] Send " did gain focus" message from engine to framework #47114

Merged
merged 8 commits into from
Nov 17, 2023

Conversation

hannah-hyj
Copy link
Member

@hannah-hyj hannah-hyj commented Oct 19, 2023

issue:flutter/flutter#97747

framework pr:flutter/flutter#135771

Pre-launch Checklist

  • I read the Contributor Guide and followed the process outlined there for submitting PRs.
  • I read the Tree Hygiene wiki page, which explains my responsibilities.
  • I read and followed the Flutter Style Guide and the C++, Objective-C, Java style guides.
  • I listed at least one issue that this PR fixes in the description above.
  • I added new tests to check the change I am making or feature I am adding, or the PR is test-exempt. See testing the engine for instructions on writing and running engine tests.
  • I updated/added relevant documentation (doc comments with ///).
  • I signed the CLA.
  • All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

@flutter-dashboard

This comment was marked as outdated.

Update AccessibilityBridge.java

1

1

1
@hannah-hyj hannah-hyj changed the title Send focus message from engine to framework Send did gain focus message from engine to framework Oct 25, 2023
@hannah-hyj hannah-hyj changed the title Send did gain focus message from engine to framework Send " did gain focus message" from engine to framework Oct 25, 2023
@hannah-hyj hannah-hyj changed the title Send " did gain focus message" from engine to framework Send " did gain focus" message from engine to framework Oct 25, 2023
@hannah-hyj hannah-hyj requested a review from chunhtai October 25, 2023 17:56
Copy link
Contributor

@chunhtai chunhtai left a comment

Choose a reason for hiding this comment

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

Can you also link framework pr so that we can review the pr as a whole?

@zanderso
Copy link
Member

zanderso commented Nov 2, 2023

From Engine PR triage: @Hangyujin are you still working on this? Is it ready for review?

@hannah-hyj
Copy link
Member Author

@zanderso Yes this pr is ready for review and I am still working on the framework pr.

@chinmaygarde
Copy link
Member

@chunhtai This one in addition to flutter/flutter#135771 please.

Copy link
Contributor

@chunhtai chunhtai left a comment

Choose a reason for hiding this comment

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

LGTM

@chunhtai
Copy link
Contributor

Do you have iOS pr?

@hannah-hyj hannah-hyj changed the title Send " did gain focus" message from engine to framework [Android] Send " did gain focus" message from engine to framework Nov 17, 2023
@hannah-hyj hannah-hyj merged commit 714987f into flutter:main Nov 17, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Nov 17, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Nov 17, 2023
auto-submit bot pushed a commit to flutter/flutter that referenced this pull request Nov 17, 2023
…138647)

flutter/engine@e010f17...c38272b

2023-11-17 matanlurey@users.noreply.github.com Make `fml/...` compatible with `.clang-tidy`. (flutter/engine#48150)
2023-11-17 skia-flutter-autoroll@skia.org Roll Dart SDK from 03cddb5d740d to 3d4886a2e218 (1 revision) (flutter/engine#48187)
2023-11-17 skia-flutter-autoroll@skia.org Roll Skia from dd7a26ead897 to 04b0ac194443 (8 revisions) (flutter/engine#48186)
2023-11-17 jhy03261997@gmail.com [Android] Send " did gain focus" message from engine to framework (flutter/engine#47114)

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 jonahwilliams@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
auto-submit bot pushed a commit to flutter/flutter that referenced this pull request Nov 27, 2023
caseycrogers pushed a commit to caseycrogers/flutter that referenced this pull request Dec 29, 2023
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.

5 participants