New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 618098 link

Starred by 2 users

Issue metadata

Status: Verified
Owner:
Last visit > 30 days ago
Closed: Dec 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Mac
Pri: 2
Type: Bug



Sign in to add a comment

Sync error notification badge is different on Mac

Project Member Reported by jnaveen@chromium.org, Jun 7 2016

Issue description

ENVIRONMENT and STATS
For syncing clients:
Chrome version: 53.0.2761.0
OS: Mac

REPRO STEPS
1. Sign in to Chrome on multiple clients.
2. Go to advanced sync settings.
3. Change the encryption settings to "All data was encrypted with...."
4. Make sure that "Sync error:Update sync passphrase..." message shows up in the settings menu of all the syncing clients.
 
ACTUAL RESULTS
Observe that on windows machine there is a new orange badge (see the attached screenshot)
On Mac machine there is no orange badge.

EXPECTED RESULTS
Sync error notification badge should look the same on all the desktop clients


 
sync_error_win.JPG
52.3 KB View Download
sync_error_mac.jpg
63.8 KB View Download
Owner: pav...@chromium.org
Status: Assigned (was: Untriaged)
Cc: shrike@chromium.org
Components: UI>Browser>Core
Owner: ----
Status: Untriaged (was: Assigned)
shrike@: could you triage.

On mac UpdateBadgeSeverity is handled in toolbar_controller.mm. It seems not to take BadgeType parameter into account.

Components: -Services>SignIn
Owner: spqc...@chromium.org
Status: Assigned (was: Untriaged)
The new sync notification icons recently landed (replacing the three dots of the old "hamburger menu"). I thought the new icons would just show up for us on the Mac but apparently not. spqchan@ - would you please make the necessary changes to get the icons to show up? This will be for M53 (so no time pressure to try to cherry-pick back to M52).

Comment 6 by shrike@chromium.org, Jun 10 2016

A second cl landed for  Issue 584342  - the badges not showing up may have been because only the first cl had landed. We should see what the next Canary looks like before proceeding.

Comment 7 by plaree@chromium.org, Jun 13 2016

Cc: plaree@chromium.org
Project Member

Comment 8 by sheriffbot@chromium.org, Jul 6 2016

Labels: -M-53 M-54 MovedFrom-53
Moving this nonessential bug to the next milestone.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Status: Verified (was: Assigned)
This has been fixed long time back and also the sync error notification has been moved to the profile menu recently. Closing the bug.

Sign in to add a comment