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

Conversation

@justinmc
Copy link
Contributor

@justinmc justinmc commented Oct 11, 2019

This PR allows Android's TYPE_TEXT_FLAG_NO_SUGGESTIONS flag to be controlled by the new enableSuggestions parameter. Previously, it was only indirectly controllable by setting obscureText to true.

Note that there is no iOS equivalent.

Issue: flutter/flutter#22828
Used in framework PR flutter/flutter#42550.

Copy link
Contributor

@GaryQian GaryQian left a comment

Choose a reason for hiding this comment

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

LGTM!

@GaryQian
Copy link
Contributor

Looks like some tests need to be fixed?

@justinmc
Copy link
Contributor Author

Yup thanks, I forgot to add the new parameter to those tests.

@justinmc justinmc merged commit 7071dc7 into flutter:master Oct 23, 2019
@justinmc justinmc deleted the no-suggestions branch October 23, 2019 17:47
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Oct 24, 2019
engine-flutter-autoroll added a commit to flutter/flutter that referenced this pull request Oct 24, 2019
[email protected]:flutter/engine.git/compare/195425e04480...400e3b0

git log 195425e..400e3b0 --no-merges --oneline
2019-10-23 [email protected] Wires the locale provided by Fuchsia. (flutter/engine#13045)
2019-10-23 [email protected] Add FlutterEngineRunsAOTCompiledDartCode to the embedder API. (flutter/engine#13319)
2019-10-23 [email protected] [web] [test] Adding firefox install functionality to the test platform (flutter/engine#13272)
2019-10-23 [email protected] Roll src/third_party/skia 32803ff74448..6863bb0930cf (7 commits) (flutter/engine#13318)
2019-10-23 [email protected] Roll src/third_party/dart 5fd6c8a3c1..b359ac0a1e (2 commits)
2019-10-23 [email protected] [recipe] Upload opt flutter_tester (flutter/engine#13311)
2019-10-23 [email protected] Update the dependencies for the Fuchsia build of flutter_frontend_server (flutter/engine#13316)
2019-10-23 [email protected] NO_SUGGESTIONS keyboard flag in Android (flutter/engine#13099)


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 [email protected] on the revert to ensure that a human
is aware of the problem.

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/+/master/autoroll/README.md
Inconnu08 pushed a commit to Inconnu08/flutter that referenced this pull request Nov 26, 2019
[email protected]:flutter/engine.git/compare/195425e04480...400e3b0

git log 195425e..400e3b0 --no-merges --oneline
2019-10-23 [email protected] Wires the locale provided by Fuchsia. (flutter/engine#13045)
2019-10-23 [email protected] Add FlutterEngineRunsAOTCompiledDartCode to the embedder API. (flutter/engine#13319)
2019-10-23 [email protected] [web] [test] Adding firefox install functionality to the test platform (flutter/engine#13272)
2019-10-23 [email protected] Roll src/third_party/skia 32803ff74448..6863bb0930cf (7 commits) (flutter/engine#13318)
2019-10-23 [email protected] Roll src/third_party/dart 5fd6c8a3c1..b359ac0a1e (2 commits)
2019-10-23 [email protected] [recipe] Upload opt flutter_tester (flutter/engine#13311)
2019-10-23 [email protected] Update the dependencies for the Fuchsia build of flutter_frontend_server (flutter/engine#13316)
2019-10-23 [email protected] NO_SUGGESTIONS keyboard flag in Android (flutter/engine#13099)


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 [email protected] on the revert to ensure that a human
is aware of the problem.

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/+/master/autoroll/README.md
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