Synced Notifications Layout Changes

Synced Notifications Layout Changes

Per latest designs from UX, move to the 3 line format provided by the
Notification Center, remove all multi-line notification data, and always
use the first profile image photo instead of the app icon for multiple
senders.

BUG=281520
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=222157

[dewittj, 2013-09-05 23:39:43]

https://codereview.chromium.org/23981004/diff/1/chrome/browser/notifications/...
File chrome/browser/notifications/sync_notifier/synced_notification.cc (left):

https://codereview.chromium.org/23981004/diff/1/chrome/browser/notifications/...
chrome/browser/notifications/sync_notifier/synced_notification.cc:222: int
notification_count = GetNotificationCount();
If we're not showing this anymore, would it make more sense to remove this
method entirely and not count the inner notification data in the Equals method
below?

[Pete Williamson, 2013-09-06 21:01:42]

https://codereview.chromium.org/23981004/diff/1/chrome/browser/notifications/...
File chrome/browser/notifications/sync_notifier/synced_notification.cc (left):

https://codereview.chromium.org/23981004/diff/1/chrome/browser/notifications/...
chrome/browser/notifications/sync_notifier/synced_notification.cc:222: int
notification_count = GetNotificationCount();
On 2013/09/05 23:39:43, dewittj wrote:
> If we're not showing this anymore, would it make more sense to remove this
> method entirely and not count the inner notification data in the Equals method
> below?

I'd prefer to leave it as is.  I want to detect any differences and show an
updated notification, so I prefer to leave it in the equals method.  Also, we
will need the method someday in the future for expanded notifications, and it is
perhaps more generally useful.

[dewittj, 2013-09-09 16:07:04]

lgtm

[commit-bot: I haz the power, 2013-09-09 17:24:50]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/petewil@chromium.org/23981004/1

[commit-bot: I haz the power, 2013-09-09 18:29:06]

Retried try job too often on linux_aura for step(s) unit_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_aura...

[commit-bot: I haz the power, 2013-09-09 20:32:03]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/petewil@chromium.org/23981004/27001

[commit-bot: I haz the power, 2013-09-10 00:41:13]

Change committed as 222157