"Turn off" button in first run experience for the credential manager on Android doesn't work. |
|||||||
Issue description"Turn off" button in first run experience for the credential manager on Android doesn't work. Expected behavior: after button is clicked credentials_enable_autosignin has value false Observed: credentials_enable_autosignin doesn't change its value
,
Apr 25 2016
,
Apr 25 2016
Your change meets the bar and is auto-approved for M51 (branch: 2704)
,
Apr 25 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/104930a015639e64b33f7cb5fed66ffadba844eb commit 104930a015639e64b33f7cb5fed66ffadba844eb Author: Tatiana Gornak <melandory@chromium.org> Date: Mon Apr 25 13:31:21 2016 [Smart Lock] Fix "turn off" button in auto-signin first run experience. BUG= 606206 Review URL: https://codereview.chromium.org/1917713003 Cr-Commit-Position: refs/heads/master@{#389457} (cherry picked from commit da55ab76c24c3da0179daddc424443a1328e9abd) Review URL: https://codereview.chromium.org/1917813002 . Cr-Commit-Position: refs/branch-heads/2704@{#213} Cr-Branched-From: 6e53600def8f60d8c632fadc70d7c1939ccea347-refs/heads/master@{#386251} [modify] https://crrev.com/104930a015639e64b33f7cb5fed66ffadba844eb/chrome/browser/password_manager/auto_signin_first_run_dialog_android_unittest.cc [modify] https://crrev.com/104930a015639e64b33f7cb5fed66ffadba844eb/components/password_manager/core/browser/password_bubble_experiment.cc
,
Apr 25 2016
,
Apr 25 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/da55ab76c24c3da0179daddc424443a1328e9abd commit da55ab76c24c3da0179daddc424443a1328e9abd Author: melandory <melandory@chromium.org> Date: Mon Apr 25 12:37:46 2016 [Smart Lock] Fix "turn off" button in auto-signin first run experience. BUG= 606206 Review URL: https://codereview.chromium.org/1917713003 Cr-Commit-Position: refs/heads/master@{#389457} [modify] https://crrev.com/da55ab76c24c3da0179daddc424443a1328e9abd/chrome/browser/password_manager/auto_signin_first_run_dialog_android_unittest.cc [modify] https://crrev.com/da55ab76c24c3da0179daddc424443a1328e9abd/components/password_manager/core/browser/password_bubble_experiment.cc
,
Apr 26 2016
Verified fix in M51 - 51.0.2704.28. Thanks! |
|||||||
►
Sign in to add a comment |
|||||||
Comment 1 by bugdroid1@chromium.org
, Apr 25 2016