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

ash/common/system/tray/tray_background_view.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef ASH_COMMON_SYSTEM_TRAY_TRAY_BACKGROUND_VIEW_H_
 #define ASH_COMMON_SYSTEM_TRAY_TRAY_BACKGROUND_VIEW_H_
 
 #include <memory>
 
 #include "ash/ash_export.h"
 #include "ash/common/shelf/shelf_types.h"
 #include "ash/common/system/tray/actionable_view.h"
 #include "ash/common/wm/background_animator.h"
 #include "base/macros.h"
 #include "ui/compositor/layer_animation_observer.h"
 #include "ui/views/bubble/tray_bubble_view.h"
 
 namespace ash {
 class ShelfLayoutManager;
 class TrayEventFilter;
 class TrayBackground;
 class WmShelf;
 
 // Base class for children of StatusAreaWidget: SystemTray, WebNotificationTray,
 // LogoutButtonTray, OverviewButtonTray.
 // This class handles setting and animating the background when the Launcher
 // his shown/hidden. It also inherits from ActionableView so that the tray

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

nit: s/his/is

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

Done.

 // items can override PerformAction when clicked on.
 class ASH_EXPORT TrayBackgroundView : public ActionableView,
                                       public BackgroundAnimatorDelegate,
                                       public ui::ImplicitAnimationObserver {
  public:
   static const char kViewClassName[];
 
   // Base class for tray containers. Sets the border and layout. The container
   // auto-resizes the widget when necessary.
   class TrayContainer : public views::View {
@@ skipping 90 matching lines @@
   bool draw_background_as_active() const { return draw_background_as_active_; }
 
   TrayContainer* tray_container() const { return tray_container_; }
   ShelfAlignment shelf_alignment() const { return shelf_alignment_; }
   TrayEventFilter* tray_event_filter() { return tray_event_filter_.get(); }
   WmShelf* shelf() { return wm_shelf_; }
 
   // Updates the arrow visibility based on the launcher visibility.
   void UpdateBubbleViewArrow(views::TrayBubbleView* bubble_view);
 
+ protected:
+  // In the given |canvas|, draw a 1px wide and 32dp high separator line to the

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

nit: s/draw/draws
nit: s/high/tall

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

Done.

+  // right of the tray item.

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

nit: clarify which tray item.

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

Done.

+  void DrawSeparator(gfx::Canvas* canvas);
+
  private:
   class TrayWidgetObserver;
 
-  // Called from Initialize after all status area trays have been created.
-  // Sets the border based on the position of the view.
-  void SetTrayBorder();
-
   // ui::ImplicitAnimationObserver:
   void OnImplicitAnimationsCompleted() override;
   bool RequiresNotificationWhenAnimatorDestroyed() const override;
 
   // Applies transformations to the |layer()| to animate the view when
   // SetVisible(false) is called.
   void HideTransformation();
 
+  // For Material Design, some extra padding is added to tray items so there is

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

nit: drop "some"

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

Done.

+  // enough places to draw separator between this tray and the tray to the left
+  // (if horizontally aligned) or to the bottom (if vertically aligned) if
+  // separator needed. No border is set for non-MD.

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

nit: s/separator/a separator is

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

Done.

+  void CalculateAndSetTrayContainerBorder();
+
   // The shelf containing the system tray for this view.
   WmShelf* wm_shelf_;
 
   // Convenience pointer to the contents view.
   TrayContainer* tray_container_;
 
   // Shelf alignment.
   // TODO(jamescook): Don't cache this, get it from WmShelf.
   ShelfAlignment shelf_alignment_;
 
   // Owned by the view passed to SetContents().
   TrayBackground* background_;
 
   // This variable stores the activation override which will tint the background
   // differently if set to true.
   bool draw_background_as_active_;
 
   std::unique_ptr<TrayWidgetObserver> widget_observer_;
   std::unique_ptr<TrayEventFilter> tray_event_filter_;
 
   DISALLOW_COPY_AND_ASSIGN(TrayBackgroundView);
 };
 
 }  // namespace ash
 
 #endif  // ASH_COMMON_SYSTEM_TRAY_TRAY_BACKGROUND_VIEW_H_