Issue metadata
Sign in to add a comment
|
Regression: Unnecessary Sync icon is seen for all Profiles.
Reported by
abom...@etouch.net,
Jun 13 2017
|
||||||||||||||||||||||
Issue descriptionChrome Version:1.0.3129.0 (Official Build)b7e12bae8572c1ed03d0959ed71df8e4efe2f8ee-refs/heads/master@{#478840} OS: Mac (10.12.3,10.11.6),Windows(7,8,10) What steps will reproduce the problem? 1. Launch chrome and sign into chrome with valid credentials which contain passphrase. 2. Add 2 person from Manage people overlay and observe profile avatar button. 3. Remove first profile(i.e. signed in profile) and observe remaining profile avatar button. Actual:Unnecessary Sync icon is seen for all Profiles after step 2 and step 3. Expected: Sync icon should not be seen for all other Profiles except profile first(Signed in once). This is regression issue, broken in ‘M 61’ and below is manual bisect: Good build: 61.0.3128.0 Bad build: 61.0.3129.0 Note: Will soon provide Linux OS info.
,
Jun 13 2017
,
Jun 14 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/b0f420e81d69cf2712f7847a364b2d7d3d59195c commit b0f420e81d69cf2712f7847a364b2d7d3d59195c Author: pnoland <pnoland@chromium.org> Date: Wed Jun 14 20:01:26 2017 [sync] Don't show an error icon when sync isn't signed in BUG= 732688 , 732684 R=skym@chromium.org Review-Url: https://codereview.chromium.org/2942473002 Cr-Commit-Position: refs/heads/master@{#479477} [modify] https://crrev.com/b0f420e81d69cf2712f7847a364b2d7d3d59195c/components/browser_sync/profile_sync_service.cc [modify] https://crrev.com/b0f420e81d69cf2712f7847a364b2d7d3d59195c/components/browser_sync/profile_sync_service.h [modify] https://crrev.com/b0f420e81d69cf2712f7847a364b2d7d3d59195c/components/sync/base/pref_names.cc
,
Jun 15 2017
,
Jun 20 2017
Tested this issue on Mac 10.12.5 & Windows 10 with chrome #61.0.3135.4 and observed Sync icon is not on seen for all other Profiles except profile first(Signed in once), hence adding TE-Verified labels. Attaching the screen-cast for reference. |
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by kkaluri@chromium.org
, Jun 13 2017Components: Services>Sync
Labels: hasbisect-per-revision
Owner: pnoland@chromium.org
Status: Assigned (was: Unconfirmed)