Issue metadata
Sign in to add a comment
|
Regression:Sign-out doesn't reflect until the page is refreshed in md-settings. |
||||||||||||||||||||||
Issue descriptionVersion: 52.0.2730.0 dev OS: Ubuntu 12.04,14.04, Windows What steps will reproduce the problem? (1)Launch chrome and sign-in to chrome. (2)Navigate to chrome://md-settings,click on sign-out and Observe. Expected: Sign-out should be successful. Actual: Instead sign-out doesn't reflect until the page is refreshed. This is a regression issue broken in M52. Unable to provide tool bisect as we cannot sign-in into chromium builds. Hence providing manual bisect info: Good Build: 52.0.2729.0 Dev Bad Build: 52.0.2730.0 Dev CHANGELOG URL: https://chromium.googlesource.com/chromium/src/+log/60cf59abead78d7389b922951abb5b9081f93497..58b07f5147cb47dc85302bd3a112efb627c972ff Suspecting https://codereview.chromium.org/1960063003 from changelog. @tommycli : Please help in reassigning if it is not related to your change.
,
May 11 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/573ca551d58dee154aa216536bfbbc1bd0ca22a3 commit 573ca551d58dee154aa216536bfbbc1bd0ca22a3 Author: tommycli <tommycli@chromium.org> Date: Wed May 11 21:39:22 2016 Settings People Revamp: Fix some small bugs in the handler BUG=563721, 610599 , 610579 CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation Review-Url: https://codereview.chromium.org/1967883002 Cr-Commit-Position: refs/heads/master@{#393069} [modify] https://crrev.com/573ca551d58dee154aa216536bfbbc1bd0ca22a3/chrome/browser/resources/settings/people_page/sync_browser_proxy.js [modify] https://crrev.com/573ca551d58dee154aa216536bfbbc1bd0ca22a3/chrome/browser/ui/webui/settings/people_handler.cc [modify] https://crrev.com/573ca551d58dee154aa216536bfbbc1bd0ca22a3/chrome/browser/ui/webui/settings/people_handler.h [modify] https://crrev.com/573ca551d58dee154aa216536bfbbc1bd0ca22a3/chrome/browser/ui/webui/settings/people_handler_unittest.cc
,
May 23 2016
Since MD Settings has not launched, setting "regression" bugs to P2
,
Jun 1 2016
Moving this nonessential bug to the next milestone. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Jun 23 2016
Fixed, but could a TE verify please? thank you! |
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by tommycli@chromium.org
, May 10 2016