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

ash/common/system/status_area_widget.h

Index: ash/common/system/status_area_widget.h
diff --git a/ash/common/system/status_area_widget.h b/ash/common/system/status_area_widget.h
index 9ee9143f9ce1fdb6a16801891a727810208bba0b..63d6a511ed4b15d7ed3584fd8cbb05161f41dc57 100644
--- a/ash/common/system/status_area_widget.h
+++ b/ash/common/system/status_area_widget.h
@@ -17,6 +17,7 @@ class OverviewButtonTray;
 class ShellDelegate;
 class StatusAreaWidgetDelegate;
 class SystemTray;
+class TrayBackgroundView;
 class WebNotificationTray;
 class WmShelf;
 class WmWindow;
@@ -48,6 +49,12 @@ class ASH_EXPORT StatusAreaWidget : public views::Widget,
   // notification tray.
   void UpdateAfterLoginStatusChange(LoginStatus login_status);
 
+  // Based on visibility of |tray| and a tray item on its right, sets |tray|'s
+  // separator visibility either to true or false. Note that no leading
+  // separator is required before the logout_button_tray because that
+  // button's red background is distinctive on its own.
+  void OnTrayVisibilityChanged(TrayBackgroundView* tray);
+
   StatusAreaWidgetDelegate* status_area_widget_delegate() {
     return status_area_widget_delegate_;
   }
@@ -91,6 +98,11 @@ class ASH_EXPORT StatusAreaWidget : public views::Widget,
   void AddPaletteTray();
   void AddVirtualKeyboardTray();
   void AddImeMenuTray();
+
+  // Check if |tray| is next visible view in the children() list, i.e. check if

[varkha, 2016/08/18 16:33:24]

nit: is this accurate?
// Checks if |tray| is the tray to the left of |logout_button_tray_|. Returns
true if both are visible, false otherwise.

[yiyix, 2016/08/18 20:01:43]

|tray| may not be the tray after |logout_button_tray_| in the list of children.
I want to be clear that |tray| is the next visible tray after
|logout_button_tray_|.

I can update to:
// Checks if |tray| is the next visible tray to the left of
|logout_button_tray_|. Returns
true if both are visible, false otherwise.

+  // the |tray| is the visible tray on the left to the |logout_button_tray_| in
+  // Chrome OS. If |logout_button_tray_| is not visible, then it returns false.
+  bool IsNextVisibleTrayToLogout(TrayBackgroundView* tray);

[yoshiki, 2016/08/18 08:39:08]

nit: can we make this const?

[yiyix, 2016/08/18 20:01:43]

Done.

 #endif
   void AddOverviewButtonTray();