Skip to content

[Android] Fix crash on some key repeat events #165307

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.

Sign up for GitHub

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

Jump to bottom
Merged
auto-submit merged 1 commit into from
Mar 18, 2025
Merged

[Android] Fix crash on some key repeat events #165307

auto-submit merged 1 commit into from
Mar 18, 2025

Conversation

@bleroux
Copy link
Contributor

@bleroux bleroux commented Mar 17, 2025 •
edited
Loading

Description

This PR fixes a crash on Android related to specific keyboard events.
It is very similar to http://github.com/flutter/engine/issues/35924 which was related to down events. This PR fixes the similar logic related to repeat events.

Related Issue

Fixes Flutter 3.29 Fatal crash with java.lang.AssertionError

Tests

Adds 1 test.

@github-actions github-actions bot added platform-android Android applications specifically engine flutter/engine related. See also e: labels. labels Mar 17, 2025
@bleroux bleroux changed the title {Android] Fix crash on some key repeat events [Android] Fix crash on some key repeat events Mar 17, 2025
@bleroux bleroux requested a review from dkwingsmt March 17, 2025 20:46
dkwingsmt
dkwingsmt approved these changes Mar 18, 2025
View reviewed changes
Copy link
Contributor

@dkwingsmt dkwingsmt left a comment

Choose a reason for hiding this comment

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

Wow, what a case! Thank you!

@bleroux bleroux added the autosubmit Merge PR when tree becomes green via auto submit App label Mar 18, 2025
@auto-submit auto-submit bot added this pull request to the merge queue Mar 18, 2025
Merged via the queue into flutter:master with commit d0ef1c1 Mar 18, 2025
168 checks passed
@flutter-dashboard flutter-dashboard bot removed the autosubmit Merge PR when tree becomes green via auto submit App label Mar 18, 2025
@bleroux bleroux deleted the fix_android_crash_on_key_repeat_event_having_zeroed_scan_code branch March 18, 2025 21:53
@Piinks Piinks mentioned this pull request Mar 21, 2025
Open
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 25, 2025
@engine-flutter-autoroll
d0ef1c1 [Android] Fix crash on some key repeat events (flutter/flutte…
5eab710 
…r#165307)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 25, 2025
@engine-flutter-autoroll
d0ef1c1 [Android] Fix crash on some key repeat events (flutter/flutte…
2a422c4 
…r#165307)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 26, 2025
@engine-flutter-autoroll
d0ef1c1 [Android] Fix crash on some key repeat events (flutter/flutte…
c38d083 
…r#165307)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 26, 2025
@engine-flutter-autoroll
d0ef1c1 [Android] Fix crash on some key repeat events (flutter/flutte…
4d11ec1 
…r#165307)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 26, 2025
@engine-flutter-autoroll
d0ef1c1 [Android] Fix crash on some key repeat events (flutter/flutte…
3dbc21d 
…r#165307)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 26, 2025
@engine-flutter-autoroll
d0ef1c1 [Android] Fix crash on some key repeat events (flutter/flutte…
4274ee4 
…r#165307)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 26, 2025
@engine-flutter-autoroll
d0ef1c1 [Android] Fix crash on some key repeat events (flutter/flutte…
e5f59f8 
…r#165307)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 26, 2025
@engine-flutter-autoroll
d0ef1c1 [Android] Fix crash on some key repeat events (flutter/flutte…
88648f9 
…r#165307)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 26, 2025
@engine-flutter-autoroll
d0ef1c1 [Android] Fix crash on some key repeat events (flutter/flutte…
0dbdb3e 
…r#165307)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 26, 2025
@engine-flutter-autoroll
d0ef1c1 [Android] Fix crash on some key repeat events (flutter/flutte…
ab0a4a3 
…r#165307)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 27, 2025
@engine-flutter-autoroll
d0ef1c1 [Android] Fix crash on some key repeat events (flutter/flutte…
a543e7a 
…r#165307)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 27, 2025
@engine-flutter-autoroll
d0ef1c1 [Android] Fix crash on some key repeat events (flutter/flutte…
54822c7 
…r#165307)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 27, 2025
@engine-flutter-autoroll
d0ef1c1 [Android] Fix crash on some key repeat events (flutter/flutte…
bc26aa7 
…r#165307)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 27, 2025
@engine-flutter-autoroll
d0ef1c1 [Android] Fix crash on some key repeat events (flutter/flutte…
12c45f8 
…r#165307)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 27, 2025
@engine-flutter-autoroll
d0ef1c1 [Android] Fix crash on some key repeat events (flutter/flutte…
2378b7b 
…r#165307)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 27, 2025
@engine-flutter-autoroll
d0ef1c1 [Android] Fix crash on some key repeat events (flutter/flutte…
48c04a2 
…r#165307)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 28, 2025
@engine-flutter-autoroll
d0ef1c1 [Android] Fix crash on some key repeat events (flutter/flutte…
9541657 
…r#165307)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 28, 2025
@engine-flutter-autoroll
d0ef1c1 [Android] Fix crash on some key repeat events (flutter/flutte…
26a68ef 
…r#165307)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 28, 2025
@engine-flutter-autoroll
d0ef1c1 [Android] Fix crash on some key repeat events (flutter/flutte…
a3e212a 
…r#165307)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 28, 2025
@engine-flutter-autoroll
d0ef1c1 [Android] Fix crash on some key repeat events (flutter/flutte…
7098187 
…r#165307)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 29, 2025
@engine-flutter-autoroll
d0ef1c1 [Android] Fix crash on some key repeat events (flutter/flutte…
7dd750b 
…r#165307)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 29, 2025
@engine-flutter-autoroll
d0ef1c1 [Android] Fix crash on some key repeat events (flutter/flutte…
004e918 
…r#165307)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 30, 2025
@engine-flutter-autoroll
d0ef1c1 [Android] Fix crash on some key repeat events (flutter/flutte…
3cff564 
…r#165307)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 30, 2025
@engine-flutter-autoroll
d0ef1c1 [Android] Fix crash on some key repeat events (flutter/flutte…
5c26d26 
…r#165307)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 31, 2025
@engine-flutter-autoroll
d0ef1c1 [Android] Fix crash on some key repeat events (flutter/flutte…
890092c 
…r#165307)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 31, 2025
@engine-flutter-autoroll
d0ef1c1 [Android] Fix crash on some key repeat events (flutter/flutte…
c5008b8 
…r#165307)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 31, 2025
@engine-flutter-autoroll
d0ef1c1 [Android] Fix crash on some key repeat events (flutter/flutte…
20be98c 
…r#165307)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 31, 2025
@engine-flutter-autoroll
d0ef1c1 [Android] Fix crash on some key repeat events (flutter/flutte…
da6bfa8 
…r#165307)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 31, 2025
@engine-flutter-autoroll
d0ef1c1 [Android] Fix crash on some key repeat events (flutter/flutte…
084010e 
…r#165307)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 31, 2025
@engine-flutter-autoroll
d0ef1c1 [Android] Fix crash on some key repeat events (flutter/flutte…
3893b9a 
…r#165307)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Mar 31, 2025
@engine-flutter-autoroll
d0ef1c1 [Android] Fix crash on some key repeat events (flutter/flutte…
0996d82 
…r#165307)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Apr 1, 2025
@engine-flutter-autoroll
d0ef1c1 [Android] Fix crash on some key repeat events (flutter/flutte…
005d00b 
…r#165307)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Apr 1, 2025
@engine-flutter-autoroll
d0ef1c1 [Android] Fix crash on some key repeat events (flutter/flutte…
1929ed3 
…r#165307)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Apr 1, 2025
@engine-flutter-autoroll
d0ef1c1 [Android] Fix crash on some key repeat events (flutter/flutte…
2e151ac 
…r#165307)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Apr 1, 2025
@engine-flutter-autoroll
d0ef1c1 [Android] Fix crash on some key repeat events (flutter/flutte…
635f037 
…r#165307)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 20, 2025
@engine-flutter-autoroll
d0ef1c1 [Android] Fix crash on some key repeat events (flutter/flutte…
24d89ac 
…r#165307)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 20, 2025
@engine-flutter-autoroll
d0ef1c1 [Android] Fix crash on some key repeat events (flutter/flutte…
cb17e53 
…r#165307)
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request May 21, 2025
@engine-flutter-autoroll
d0ef1c1 [Android] Fix crash on some key repeat events (flutter/flutte…
65e18f6 
…r#165307)
romanejaquez pushed a commit to romanejaquez/flutter that referenced this pull request Aug 14, 2025
@bleroux @romanejaquez
[Android] Fix crash on some key repeat events (flutter#165307)
f338999 
## Description

This PR fixes a crash on Android related to specific keyboard events.
It is very similar to http://github.com/flutter/engine/issues/35924
which was related to down events. This PR fixes the similar logic
related to repeat events.

## Related Issue

Fixes [Flutter 3.29 Fatal crash with
java.lang.AssertionError](flutter#164626)

## Tests

Adds 1 test.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dkwingsmt dkwingsmt dkwingsmt approved these changes

Assignees

No one assigned

Labels

engine flutter/engine related. See also e: labels. platform-android Android applications specifically

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Flutter 3.29 Fatal crash with java.lang.AssertionError

2 participants

@bleroux @dkwingsmt