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

Revert "[Android] Return keyboard pressed state" #42616

Conversation

dkwingsmt
Copy link
Contributor

Reverts #41695 due to internal bug report b/284945818.

cc @bleroux let's investigate some day.

@dkwingsmt dkwingsmt merged commit 61b2b50 into main Jun 7, 2023
@dkwingsmt dkwingsmt deleted the revert-41695-android_listen_and_answer_to_keyboard_pressed_state_queries branch June 7, 2023 08:38
@jiahaog
Copy link
Member

jiahaog commented Jun 7, 2023

We should try to create a repro from the internal bug so that it's possible for us to debug the issue and reland this :)

@jiahaog
Copy link
Member

jiahaog commented Jun 7, 2023

Do we need to revert flutter/flutter#122885 as well as it looks like that framework change depends on this?

@bleroux
Copy link
Contributor

bleroux commented Jun 7, 2023

Do we need to revert flutter/flutter#122885 as well as it looks like that framework change depends on this?

This engine PR depends on flutter/flutter#122885, but flutter/flutter#122885 does not depend on engine implementations.

If it is possible to share part of b/284945818 description, I would be happy to understand what is in play here.

engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Jun 7, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Jun 7, 2023
zanderso pushed a commit to flutter/flutter that referenced this pull request Jun 7, 2023
…128460)

flutter/engine@4f4486b...1089ce6

2023-06-07 jonahwilliams@google.com [Impeller] Add buffer to texture
blit for Vulkan. (flutter/engine#41706)
2023-06-07 chris@bracken.jp [macOS] Add platformview creation parameter
support (flutter/engine#42607)
2023-06-07 30870216+gaaclarke@users.noreply.github.com [Impeller] Added
a switch to turn on vulkan (flutter/engine#42585)
2023-06-07 skia-flutter-autoroll@skia.org Roll ANGLE from f8220fa3a729
to 15a29438b099 (1 revision) (flutter/engine#42627)
2023-06-07 jacksongardner@google.com Bump Chrome version to 114 for
testing (flutter/engine#42623)
2023-06-07 skia-flutter-autoroll@skia.org Roll Skia from d607cbb0db78 to
773765ca1dd2 (7 revisions) (flutter/engine#42624)
2023-06-07 jpnurmi@gmail.com [Linux][a11y] implement
AtkText::get_text/string_at_offset() (flutter/engine#38144)
2023-06-07 skia-flutter-autoroll@skia.org Roll ANGLE from 176989ad00cc
to f8220fa3a729 (1 revision) (flutter/engine#42621)
2023-06-07 skia-flutter-autoroll@skia.org Roll Skia from ee90e9ae2e62 to
d607cbb0db78 (1 revision) (flutter/engine#42618)
2023-06-07 skia-flutter-autoroll@skia.org Roll ANGLE from 1ad4ae4d63bf
to 176989ad00cc (1 revision) (flutter/engine#42617)
2023-06-07 dkwingsmt@users.noreply.github.com Revert "[Android] Return
keyboard pressed state" (flutter/engine#42616)
2023-06-07 skia-flutter-autoroll@skia.org Roll Skia from bde894438f29 to
ee90e9ae2e62 (1 revision) (flutter/engine#42614)
2023-06-07 skia-flutter-autoroll@skia.org Roll Fuchsia Linux SDK from
lpbkSRJBMkPs0FM7_... to sEHtHM1iFt79roP-x... (flutter/engine#42613)
2023-06-07 skia-flutter-autoroll@skia.org Roll Skia from cd3e1665dcd1 to
bde894438f29 (1 revision) (flutter/engine#42612)
2023-06-07 skia-flutter-autoroll@skia.org Roll ANGLE from 16841d6256da
to 1ad4ae4d63bf (1 revision) (flutter/engine#42611)
2023-06-07 bdero@google.com [Impeller] Reland 2: Add Impeller Metal
support in the embedder API (#42411) (flutter/engine#42597)
2023-06-07 skia-flutter-autoroll@skia.org Roll Skia from a01f49f539ab to
cd3e1665dcd1 (1 revision) (flutter/engine#42610)
2023-06-07 skia-flutter-autoroll@skia.org Roll Fuchsia Mac SDK from
atrYtfHWi2cmV9B_C... to ojwVlxZWrbsG4WGSE... (flutter/engine#42609)
2023-06-07 skia-flutter-autoroll@skia.org Roll Skia from 521b8c4bb011 to
a01f49f539ab (4 revisions) (flutter/engine#42608)
2023-06-07 skia-flutter-autoroll@skia.org Roll Skia from cef18d10b363 to
521b8c4bb011 (1 revision) (flutter/engine#42605)

Also rolling transitive DEPS:
  fuchsia/sdk/core/linux-amd64 from lpbkSRJBMkPs to sEHtHM1iFt79
  fuchsia/sdk/core/mac-amd64 from atrYtfHWi2cm to ojwVlxZWrbsG

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 jacksongardner@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://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
CaseyHillers pushed a commit to CaseyHillers/engine that referenced this pull request Jun 8, 2023
Reverts flutter#41695 due to internal bug report b/284945818.

cc @bleroux let's investigate some day.
CaseyHillers pushed a commit that referenced this pull request Jun 8, 2023
Reverts #41695 due to internal bug report b/284945818.

b/286341060

Co-authored-by: Tong Mu <dkwingsmt@users.noreply.github.com>
CaseyHillers pushed a commit to CaseyHillers/engine that referenced this pull request Jun 13, 2023
Reverts flutter#41695 due to internal bug report b/284945818.

cc @bleroux let's investigate some day.
CaseyHillers pushed a commit that referenced this pull request Jun 13, 2023
b/286341060

Co-authored-by: Tong Mu <dkwingsmt@users.noreply.github.com>
XilaiZhang pushed a commit to XilaiZhang/engine that referenced this pull request Jun 13, 2023
Reverts flutter#41695 due to internal bug report b/284945818.

cc @bleroux let's investigate some day.
CaseyHillers pushed a commit that referenced this pull request Jun 13, 2023
Reverts #41695 due to internal bug
report b/284945818.

cherry pick request was filed at b/287043578

Co-authored-by: Tong Mu <dkwingsmt@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants