ewald@, just to make sure, there are in total 5 sync status messages with email addressed attached to them, do they all need to drop the email?

https://cs.chromium.org/chromium/src/chrome/app/generated_resources.grd?l=11600,11712,11715,11718,11721

Yes. There's a separate bug ( Issue 612595 , also assigned to you) about updating the status/error messages according to our new spec. But in those messages, the email address is gone.

Just a heads up, the sync rows have recently been updated to have "page indicators" (right-facing arrows and open-in-new icons) communicating that a row is clickable. You can see this live in canary today. 

This means that the sync row(s) can have 3 types of embellishments: 

1. Right-facing arrow icon - when clicking the row opens a sub-page within settings, a dialog, or any internal Chrome property (i.e. user manager). 

2. Open-in-new icon - used when clicking takes you to an open web property, i.e. myaccount.google.com 

3. No icon - used when the row is not actionable (passive) 

Please ensure that our error states use the proper embellishment. Let me know if you have any questions. 

Deleted: arrow.png 114 KB

	arrow.png 114 KB View Download

Thanks Alan, what is embellishment for actionable errors?

Each error has a different behavior, right? E.g. the "Passphrase" error opens the subpage/dialogue for entering your passphrase. The auth error opens the sign in flow to re-auth.

I don't think any of them take you to a web property though, right? So they should probably all have the right-facing arrows, according to Alan's spec above.

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/9d08a7d8a72909a0c5bfe0ed5cb0b590876b7dcf

commit 9d08a7d8a72909a0c5bfe0ed5cb0b590876b7dcf
Author: mahmadi <mahmadi@chromium.org>
Date: Tue Oct 18 16:56:10 2016

[MD Settings][People] Updates positions of username and sync status message

before: https://screenshot.googleplex.com/q9gMEb84CDy
after: https://screenshot.googleplex.com/oobv4yKECg5

This CL also updates sync status messages.

BUG= 638464 
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2393643004
Cr-Commit-Position: refs/heads/master@{#425991}

[modify] https://crrev.com/9d08a7d8a72909a0c5bfe0ed5cb0b590876b7dcf/chrome/app/generated_resources.grd
[modify] https://crrev.com/9d08a7d8a72909a0c5bfe0ed5cb0b590876b7dcf/chrome/browser/resources/options/browser_options.js
[modify] https://crrev.com/9d08a7d8a72909a0c5bfe0ed5cb0b590876b7dcf/chrome/browser/resources/options/sync_section.html
[modify] https://crrev.com/9d08a7d8a72909a0c5bfe0ed5cb0b590876b7dcf/chrome/browser/resources/settings/people_page/people_page.html
[modify] https://crrev.com/9d08a7d8a72909a0c5bfe0ed5cb0b590876b7dcf/chrome/browser/resources/settings/people_page/sync_browser_proxy.js
[modify] https://crrev.com/9d08a7d8a72909a0c5bfe0ed5cb0b590876b7dcf/chrome/browser/signin/signin_ui_util.cc
[modify] https://crrev.com/9d08a7d8a72909a0c5bfe0ed5cb0b590876b7dcf/chrome/browser/signin/signin_ui_util.h
[modify] https://crrev.com/9d08a7d8a72909a0c5bfe0ed5cb0b590876b7dcf/chrome/browser/sync/profile_sync_service_android.cc
[modify] https://crrev.com/9d08a7d8a72909a0c5bfe0ed5cb0b590876b7dcf/chrome/browser/sync/sync_ui_util.cc
[modify] https://crrev.com/9d08a7d8a72909a0c5bfe0ed5cb0b590876b7dcf/chrome/browser/ui/webui/options/browser_options_handler.cc
[modify] https://crrev.com/9d08a7d8a72909a0c5bfe0ed5cb0b590876b7dcf/chrome/browser/ui/webui/settings/people_handler.cc

Looks great. Thank you so much! 

Moe - can we mark this as Fixed? We're tracking the error messages separately, right? Or is that also part of this bug.

FYI looks like error messages are being tracked in  Issue 612595 

yep. let's mark this as fixed.