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

Conversation

@amirh
Copy link
Contributor

@amirh amirh commented Jun 10, 2019

#9203 broke the keyboard_resize integration test(see more details in flutter/flutter#34085 (comment)).

This relands @9203 and fixes the issue the integration test uncovered by always allowing to hide the keyboard.

The difference from the original change is 07d2598

@amirh
Copy link
Contributor Author

amirh commented Jun 10, 2019

@jason-simmons

@amirh amirh merged commit 259d334 into flutter:master Jun 10, 2019
@amirh amirh deleted the reland_keyboard branch June 10, 2019 19:56
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Jun 10, 2019
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Jun 10, 2019
engine-flutter-autoroll added a commit to flutter/flutter that referenced this pull request Jun 10, 2019
flutter/engine@36b7123...f3ab2e4

git log 36b7123..f3ab2e4 --no-merges --oneline
f3ab2e4 Roll src/third_party/skia 5ec1b8ff3f45..cb1adb40d0f3 (10 commits) (flutter/engine#9258)
b622d43 Make flow layers' attributes immutable (flutter/engine#9176)
259d334 Reland "Keyboard support for embedded Android views. (#9203) (flutter/engine#9257)

The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, please contact the current sheriff (bmparr@google.com), and stop
the roller if necessary.
huqiuser pushed a commit to huqiuser/engine that referenced this pull request Jun 12, 2019
…lutter#9257)

flutter#9203 broke the keyboard_resize integration test(see more details in flutter/flutter#34085 (comment)).

This re-lands @9203 and fixes the issue the integration test uncovered by always allowing to hide the keyboard.

The difference from the original change is 07d2598
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants