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

ash/common/system/web_notification/web_notification_tray.cc

 // 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.
 
 #include "ash/common/system/web_notification/web_notification_tray.h"
 
 #include "ash/common/ash_switches.h"
 #include "ash/common/material_design/material_design_controller.h"
 #include "ash/common/session/session_state_delegate.h"
 #include "ash/common/shelf/shelf_constants.h"
@@ skipping 159 matching lines @@
         parent()->RemoveChildView(this);
       base::ThreadTaskRunnerHandle::Get()->DeleteSoon(FROM_HERE, this);
     } else {
       delete_after_animation_ = true;
     }
   }
 
  protected:
   // Overridden from views::View:
   gfx::Size GetPreferredSize() const override {
-    if (!animation_.get() || !animation_->is_animating())

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

Please don't remove this. This is a short circuit for the non-animated case.

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

I am so sorry, i removed it as I tray different solutions to update the
background and as i review my own cl, i missed this change. So sorry.

-      return kTrayItemOuterSize;
-
     // Animate the width (or height) when this item shows (or hides) so that
     // the icons on the left are shifted with the animation.
     // Note that TrayItemView does the same thing.
     gfx::Size size = kTrayItemOuterSize;
     if (IsHorizontalLayout()) {
       size.set_width(std::max(
           1, gfx::ToRoundedInt(size.width() * animation_->GetCurrentValue())));
     } else {
       size.set_height(std::max(
           1, gfx::ToRoundedInt(size.height() * animation_->GetCurrentValue())));
@@ skipping 475 matching lines @@
 
 message_center::MessageCenterBubble*
 WebNotificationTray::GetMessageCenterBubbleForTest() {
   if (!message_center_bubble())
     return NULL;
   return static_cast<message_center::MessageCenterBubble*>(
       message_center_bubble()->bubble());
 }
 
 }  // namespace ash