[Chrome OS MD] Draw a 1px separator between 2 tray items

ash/common/system/web_notification/web_notification_tray.h

Index: ash/common/system/web_notification/web_notification_tray.h
diff --git a/ash/common/system/web_notification/web_notification_tray.h b/ash/common/system/web_notification/web_notification_tray.h
index 0527238243a85d098c2ca18b540797dbee9fba5a..a398695883240e288571f276157aa974415e7cc8 100644
--- a/ash/common/system/web_notification/web_notification_tray.h
+++ b/ash/common/system/web_notification/web_notification_tray.h
@@ -168,6 +168,9 @@ class ASH_EXPORT WebNotificationTray
   // for adding a secondary user.
   bool IsLoggedIn() const;
 
+  // views:View
+  void OnPaint(gfx::Canvas* canvas) override;
+

[varkha, 2016/07/14 21:34:48]

I think it would be better to make it protected because you are hiding it from
possible overrides in subclasses. That or move down since overrides should be
the last methods declared (a pattern I observe even though I could not find it
mentioned in the style guide).

[yiyix, 2016/07/26 20:27:32]

N/A with new approach

   // Testing accessors.
   bool IsPopupVisible() const;
   message_center::MessageCenterBubble* GetMessageCenterBubbleForTest();