Remove kEnablePasswordChangeSupport and clean-up components/autofill/content/renderer/form_autofill_util.cc |
||
Issue descriptionpassword_manager::features::kEnablePasswordChangeSupport is no longer used since r414350, we should remove it completely. Also, in components/autofill/content/renderer/form_autofill_util.cc, IsFormVisible still contains code excluded from OS_ANDROID. This code can potentially lead to Chrome understanding more submission events as success (by returning false). The code was not used on Android when Android did not have the "Update password?" prompt, to protect users from rewriting a good credential by some misunderstood other content. Now that Android has the update prompt, there is no need for running different checks on Android, and we should remove the non-Android #ifdefs.
,
Oct 11 2016
,
Oct 12 2016
https://codereview.chromium.org/2409123002/ just landed. Sadly the BUG= annotation got lost during editing the description, so it won't be posted here automatically. The commit is: https://chromium.googlesource.com/chromium/src/+/3a03743c7be2c81af3390823488db30fa7c1dd29 Thanks for landing this, peary2@. |
||
►
Sign in to add a comment |
||
Comment 1 by vabr@chromium.org
, Oct 10 2016