DescriptionAdd support for setting password value gated on user's gesture in a page
A new way to set value is exposed for password input elements: setValueAfterUserGesture -- unlike setValue, which sets the new value immediately, setValueAfterUserGesture delays the effect until the user interacts with the page in any way.
This has security motivation, detailed here:
https://docs.google.com/document/d/1_Es0qQVrKZSZoXuQBuPmNW8-EAUDsuQGSMIlRFPzs_0/edit?usp=sharing
This improves the current status, when a workaround involving setSuggestedValue and checking for a user gesture outside of Blink is used. That workaround does not work for Android, and can be seen as a layering violation in the sense, that checking for user gesture is a Blink internal thing.
This is the Blink part, two more CLs are planned as follow-ups:
1) Chromium part: PasswordAutofillAgent using setValueAfterUserGesture instead of setSuggestedValue + clean-up (basically getting rid of content/ changes from https://codereview.chromium.org/163843002)
2) Blink clean-up: making WebInputEvent::isUserGestureEventType Blink-internal
This CL was closed without landing, in favour of https://codereview.chromium.org/235983016/.
BUG=163072
Patch Set 1 : #
Messages
Total messages: 5 (0 generated)
|