-
Notifications
You must be signed in to change notification settings - Fork 29.7k
Add deleteBackwardByDecomposingPreviousCharacter mapping for tests #132919
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
Conversation
|
@Renzo-Olivares Could you take a look? |
Renzo-Olivares
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM thanks for the contribution!
justinmc
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍 Whoops, thanks for fixing this!
|
I don't think that selector is mapped to an intent in the framework. Do you have plans to dispatch that event as an |
|
@LongCatIsLooong I'm handling the selector in performSelector directly. |
|
I see. Could you rebase the branch to fix Google testing? |
9d170a6 to
08c554a
Compare
|
@LongCatIsLooong Done. |
|
auto label is removed for flutter/flutter/132919, due to - The status or check suite Linux web_canvaskit_tests_2 has failed. Please fix the issues identified (or deflake) before re-applying this label.
|
flutter/flutter@5e6d88d...54c98d7 2023-08-22 [email protected] Roll Flutter Engine from 447981acbf36 to f5f099a4a5e3 (1 revision) (flutter/flutter#133035) 2023-08-22 [email protected] Roll Flutter Engine from 5e84d733ef67 to 447981acbf36 (2 revisions) (flutter/flutter#133031) 2023-08-22 [email protected] Roll Flutter Engine from e183e8a94093 to 5e84d733ef67 (1 revision) (flutter/flutter#133030) 2023-08-22 [email protected] Roll Flutter Engine from 454e0e35f8e4 to e183e8a94093 (2 revisions) (flutter/flutter#133026) 2023-08-22 [email protected] Roll Flutter Engine from 857840db5574 to 454e0e35f8e4 (1 revision) (flutter/flutter#133020) 2023-08-22 [email protected] Roll Flutter Engine from 1ab054967c1d to 857840db5574 (2 revisions) (flutter/flutter#133018) 2023-08-22 [email protected] Roll Flutter Engine from 5a292e42a593 to 1ab054967c1d (1 revision) (flutter/flutter#133014) 2023-08-22 [email protected] Fix visual overflow for SliverMainAxisGroup (flutter/flutter#132989) 2023-08-22 [email protected] Roll Flutter Engine from e5929d83e9b7 to 5a292e42a593 (3 revisions) (flutter/flutter#133010) 2023-08-22 [email protected] Always require `--local-engine-host` if `--local-engine` (flutter/flutter#133003) 2023-08-22 [email protected] Roll Flutter Engine from 55b84268f2a6 to e5929d83e9b7 (3 revisions) (flutter/flutter#133005) 2023-08-21 [email protected] Roll Flutter Engine from e5f690b829e5 to 55b84268f2a6 (1 revision) (flutter/flutter#133000) 2023-08-21 [email protected] Add deleteBackwardByDecomposingPreviousCharacter mapping for tests (flutter/flutter#132919) 2023-08-21 [email protected] Roll Flutter Engine from fc75109c3a1e to e5f690b829e5 (1 revision) (flutter/flutter#132996) 2023-08-21 [email protected] Roll Flutter Engine from 2388b9db2be7 to fc75109c3a1e (3 revisions) (flutter/flutter#132992) 2023-08-21 [email protected] Roll Flutter Engine from 210f5a8aa245 to 2388b9db2be7 (6 revisions) (flutter/flutter#132986) 2023-08-21 [email protected] Update `ExpansionPanel` example for the updated `expansionCallback` callback (flutter/flutter#132837) 2023-08-21 [email protected] Change gradle lockfile generation script to use --config-only flag (flutter/flutter#132967) 2023-08-21 [email protected] Roll Flutter Engine from 0290d0bd79ee to 210f5a8aa245 (1 revision) (flutter/flutter#132973) 2023-08-21 [email protected] Docs on iOS's directional quote characters (flutter/flutter#132869) 2023-08-21 [email protected] Bump memory usage for flutter gallery (flutter/flutter#132968) 2023-08-21 [email protected] When resized network image has error, all future unrelated images using the same url will fail, even if the network becomes OK (flutter/flutter#127456) 2023-08-21 [email protected] Roll Flutter Engine from a9b00783bc7c to 0290d0bd79ee (1 revision) (flutter/flutter#132962) 2023-08-21 [email protected] Roll Packages from 4c16f3e to e26f3b9 (5 revisions) (flutter/flutter#132960) If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/flutter-packages Please CC [email protected],[email protected],[email protected] on the revert to ensure that a human is aware of the problem. To file a bug in Packages: 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
In
MacOSTestTextInputKeyHandler, there are some mappings from shortcuts to generate macOS selectors. However,ctrl + backspaceisn't mapped todeleteBackwardByDecomposingPreviousCharacter:.This PR adds the mapping for
deleteBackwardByDecomposingPreviousCharacter:.Fixes: #132917
Pre-launch Checklist
///).