Remove non-MD code from TrayBackgroundView

ash/common/system/tray/tray_background_view.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/tray/tray_background_view.h"
 
 #include <algorithm>
 
 #include "ash/common/ash_constants.h"
-#include "ash/common/material_design/material_design_controller.h"
 #include "ash/common/shelf/shelf_constants.h"
 #include "ash/common/shelf/wm_shelf.h"
 #include "ash/common/shelf/wm_shelf_util.h"
-#include "ash/common/system/tray/system_tray.h"
 #include "ash/common/system/tray/tray_constants.h"
 #include "ash/common/system/tray/tray_event_filter.h"
 #include "ash/common/wm_lookup.h"
-#include "ash/common/wm_shell.h"
 #include "ash/common/wm_window.h"
-#include "ash/public/cpp/shell_window_ids.h"
-#include "ash/resources/grit/ash_resources.h"
 #include "base/memory/ptr_util.h"
 #include "ui/accessibility/ax_node_data.h"
-#include "ui/base/nine_image_painter_factory.h"
 #include "ui/compositor/layer.h"
 #include "ui/compositor/layer_animation_element.h"
 #include "ui/compositor/scoped_layer_animation_settings.h"
 #include "ui/events/event_constants.h"
 #include "ui/gfx/animation/tween.h"
 #include "ui/gfx/canvas.h"
 #include "ui/gfx/geometry/rect.h"
 #include "ui/gfx/geometry/size.h"
-#include "ui/gfx/image/image_skia.h"
-#include "ui/gfx/image/image_skia_operations.h"
-#include "ui/gfx/nine_image_painter.h"
 #include "ui/gfx/scoped_canvas.h"
-#include "ui/gfx/skia_util.h"
 #include "ui/gfx/transform.h"
 #include "ui/views/animation/flood_fill_ink_drop_ripple.h"
 #include "ui/views/animation/ink_drop_highlight.h"
 #include "ui/views/animation/ink_drop_mask.h"
 #include "ui/views/background.h"
 #include "ui/views/layout/box_layout.h"
-#include "ui/wm/core/window_animations.h"
 
 namespace {
 
 const int kAnimationDurationForPopupMs = 200;
 
 // Duration of opacity animation for visibility changes.
 const int kAnimationDurationForVisibilityMs = 250;
 
 // When becoming visible delay the animation so that StatusAreaWidgetDelegate
 // can animate sibling views out of the position to be occuped by the
 // TrayBackgroundView.
 const int kShowAnimationDelayMs = 100;
 
-// Additional padding used to adjust the user-visible size of status tray
-// and overview button dark background.
-const int kBackgroundAdjustPadding = 3;
-
 // Switches left and right insets if RTL mode is active.
 void MirrorInsetsIfNecessary(gfx::Insets* insets) {
   if (base::i18n::IsRTL()) {
     insets->Set(insets->top(), insets->right(), insets->bottom(),
                 insets->left());
   }
 }
 
 // Returns background insets relative to the contents bounds of the view and
 // mirrored if RTL mode is active.
@@ skipping 35 matching lines @@
   }
 
  private:
   TrayBackgroundView* host_;
 
   DISALLOW_COPY_AND_ASSIGN(TrayWidgetObserver);
 };
 
 class TrayBackground : public views::Background {
  public:
-  TrayBackground(TrayBackgroundView* tray_background_view, bool draws_active)
+  explicit TrayBackground(TrayBackgroundView* tray_background_view)
       : tray_background_view_(tray_background_view),
-        draws_active_(draws_active),
         color_(SK_ColorTRANSPARENT) {}
 
   ~TrayBackground() override {}
 
   void set_color(SkColor color) { color_ = color; }
 
  private:
   WmShelf* GetShelf() const { return tray_background_view_->shelf(); }
 
-  void PaintMaterial(gfx::Canvas* canvas, views::View* view) const {
+  // Overridden from views::Background.
+  void Paint(gfx::Canvas* canvas, views::View* view) const override {
     cc::PaintFlags background_flags;
     background_flags.setFlags(cc::PaintFlags::kAntiAlias_Flag);
     background_flags.setColor(color_);
     gfx::Insets insets =
         GetMirroredBackgroundInsets(GetShelf()->GetAlignment());
     gfx::Rect bounds = view->GetLocalBounds();
     bounds.Inset(insets);
     canvas->DrawRoundRect(bounds, kTrayRoundedBorderRadius, background_flags);
-
-    if (draws_active_ && tray_background_view_->is_active()) {
-      cc::PaintFlags highlight_flags;
-      highlight_flags.setFlags(cc::PaintFlags::kAntiAlias_Flag);
-      highlight_flags.setColor(kShelfButtonActivatedHighlightColor);
-      canvas->DrawRoundRect(bounds, kTrayRoundedBorderRadius, highlight_flags);
-    }
-  }
-
-  void PaintNonMaterial(gfx::Canvas* canvas, views::View* view) const {
-    const static int kImageTypeDefault = 0;
-    // TODO(estade): leftover type which should be removed along with the rest
-    // of pre-MD code.
-    // const static int kImageTypeOnBlack = 1;
-    const static int kImageTypePressed = 2;
-    const static int kNumStates = 3;
-
-    const static int kImageHorizontal = 0;
-    const static int kImageVertical = 1;
-    const static int kNumOrientations = 2;
-
-    const int kGridSizeForPainter = 9;
-
-    const int kImages[kNumOrientations][kNumStates][kGridSizeForPainter] = {
-        {
-            // Horizontal
-            IMAGE_GRID_HORIZONTAL(IDR_AURA_TRAY_BG_HORIZ),
-            IMAGE_GRID_HORIZONTAL(IDR_AURA_TRAY_BG_HORIZ_ONBLACK),
-            IMAGE_GRID_HORIZONTAL(IDR_AURA_TRAY_BG_HORIZ_PRESSED),
-        },
-        {
-            // Vertical
-            IMAGE_GRID_VERTICAL(IDR_AURA_TRAY_BG_VERTICAL),
-            IMAGE_GRID_VERTICAL(IDR_AURA_TRAY_BG_VERTICAL_ONBLACK),
-            IMAGE_GRID_VERTICAL(IDR_AURA_TRAY_BG_VERTICAL_PRESSED),
-        }};
-
-    WmShelf* shelf = GetShelf();
-    const int orientation = IsHorizontalAlignment(shelf->GetAlignment())
-                                ? kImageHorizontal
-                                : kImageVertical;
-
-    int state = kImageTypeDefault;
-    if (draws_active_ && tray_background_view_->is_active())
-      state = kImageTypePressed;
-    else
-      state = kImageTypeDefault;
-
-    ui::CreateNineImagePainter(kImages[orientation][state])
-        ->Paint(canvas, view->GetLocalBounds());
-  }
-
-  // Overridden from views::Background.
-  void Paint(gfx::Canvas* canvas, views::View* view) const override {
-    if (MaterialDesignController::IsShelfMaterial())
-      PaintMaterial(canvas, view);
-    else
-      PaintNonMaterial(canvas, view);
   }
 
   // Reference to the TrayBackgroundView for which this is a background.
   TrayBackgroundView* tray_background_view_;
 
-  // Determines whether we should draw an active background for the view when it
-  // is active. This is used in non-MD mode. In material design mode, an active
-  // ink drop ripple would indicate if the view is active or not.
-  // TODO(mohsen): This is used only in non-MD version. Remove when non-MD code
-  // is removed (see https://crbug.com/614453).
-  bool draws_active_;
-
   SkColor color_;
 
   DISALLOW_COPY_AND_ASSIGN(TrayBackground);
 };
 
 TrayBackgroundView::TrayContainer::TrayContainer(ShelfAlignment alignment)
     : alignment_(alignment) {
   UpdateLayout();
 }
 
@@ skipping 28 matching lines @@
 
 void TrayBackgroundView::TrayContainer::UpdateLayout() {
   bool is_horizontal = IsHorizontalAlignment(alignment_);
 
   // Adjust the size of status tray dark background by adding additional
   // empty border.
   views::BoxLayout::Orientation orientation =
       is_horizontal ? views::BoxLayout::kHorizontal
                     : views::BoxLayout::kVertical;
 
-  if (ash::MaterialDesignController::IsShelfMaterial()) {
-    const int hit_region_with_separator = kHitRegionPadding + kSeparatorWidth;
-    gfx::Insets insets(
-        is_horizontal
-            ? gfx::Insets(0, kHitRegionPadding, 0, hit_region_with_separator)
-            : gfx::Insets(kHitRegionPadding, 0, hit_region_with_separator, 0));
-    MirrorInsetsIfNecessary(&insets);
-    SetBorder(views::CreateEmptyBorder(insets));
-  } else {
-    SetBorder(views::CreateEmptyBorder(gfx::Insets(kBackgroundAdjustPadding)));
-  }
+  const int hit_region_with_separator = kHitRegionPadding + kSeparatorWidth;
+  gfx::Insets insets(
+      is_horizontal
+          ? gfx::Insets(0, kHitRegionPadding, 0, hit_region_with_separator)
+          : gfx::Insets(kHitRegionPadding, 0, hit_region_with_separator, 0));
+  MirrorInsetsIfNecessary(&insets);
+  SetBorder(views::CreateEmptyBorder(insets));
 
   int horizontal_margin = main_axis_margin_;
   int vertical_margin = cross_axis_margin_;
   if (!is_horizontal)
     std::swap(horizontal_margin, vertical_margin);
   views::BoxLayout* layout =
       new views::BoxLayout(orientation, horizontal_margin, vertical_margin, 0);
 
-  if (!ash::MaterialDesignController::IsShelfMaterial())
-    layout->SetDefaultFlex(1);
   layout->set_minimum_cross_axis_size(kTrayItemSize);
   views::View::SetLayoutManager(layout);
 
   PreferredSizeChanged();
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 // TrayBackgroundView
 
 TrayBackgroundView::TrayBackgroundView(WmShelf* wm_shelf)
@@ skipping 133 matching lines @@
   bounds.set_width(bounds.width() + 2 * icon_size);
   bounds.set_height(bounds.height() + 2 * icon_size);
   std::unique_ptr<views::InkDropHighlight> highlight(
       new views::InkDropHighlight(bounds.size(), 0,
                                   gfx::RectF(bounds).CenterPoint(),
                                   GetInkDropBaseColor()));
   highlight->set_visible_opacity(kTrayPopupInkDropHighlightOpacity);
   return highlight;
 }
 
-void TrayBackgroundView::OnGestureEvent(ui::GestureEvent* event) {
-  // If there is no ink drop, show "touch feedback".
-  // TODO(mohsen): This is used only in non-MD version. Remove when non-MD code
-  // is removed (see https://crbug.com/614453).
-  if (ink_drop_mode() == InkDropMode::OFF) {
-    if (event->type() == ui::ET_GESTURE_TAP_DOWN) {
-      SetIsActive(true);
-    } else if (event->type() == ui::ET_GESTURE_SCROLL_BEGIN ||
-               event->type() == ui::ET_GESTURE_TAP_CANCEL) {
-      SetIsActive(false);
-    }
-  }
-  ActionableView::OnGestureEvent(event);
-}
-
 void TrayBackgroundView::SetContents(views::View* contents) {
   SetLayoutManager(new views::BoxLayout(views::BoxLayout::kVertical, 0, 0, 0));
   AddChildView(contents);
 }
 
-void TrayBackgroundView::SetContentsBackground(bool draws_active) {
-  background_ = new TrayBackground(this, draws_active);
+void TrayBackgroundView::SetContentsBackground() {
+  background_ = new TrayBackground(this);

[Evan Stade, 2017/03/01 04:12:21]

Do you need this function?

[mohsen, 2017/03/01 04:40:41]

It is not called from LogoutButtonTray. So, I think not all subclasses of
TrayBackgroundView need the background.

[Evan Stade, 2017/03/01 07:20:21]

Good point, but I still think we should remove it (set by default) and just have
the logout button clear the background on tray_container_. It was only something
you could dynamically set because of the activat-ability.

[mohsen, 2017/03/01 17:48:05]

I think logout button would still need a similar method to be able to clear
|background_|. How about adding a flag to TrayBackgroundView c'tor?

   tray_container_->set_background(background_);
 }
 
 void TrayBackgroundView::SetShelfAlignment(ShelfAlignment alignment) {
   shelf_alignment_ = alignment;
   tray_container_->SetAlignment(alignment);
 }
 
 void TrayBackgroundView::OnImplicitAnimationsCompleted() {
   // If there is another animation in the queue, the reverse animation was
@@ skipping 33 matching lines @@
   return TrayBubbleView::ANCHOR_ALIGNMENT_BOTTOM;
 }
 
 void TrayBackgroundView::SetIsActive(bool is_active) {
   if (is_active_ == is_active)
     return;
   is_active_ = is_active;
   AnimateInkDrop(is_active_ ? views::InkDropState::ACTIVATED
                             : views::InkDropState::DEACTIVATED,
                  nullptr);
-  if (!background_)
-    return;
-  // TODO(mohsen): This is needed for non-MD version. Remove when non-MD code is
-  // removed (see https://crbug.com/614453).
-  SchedulePaint();
 }
 
 void TrayBackgroundView::UpdateBubbleViewArrow(
     views::TrayBubbleView* bubble_view) {
   // Nothing to do here.
 }
 
 void TrayBackgroundView::UpdateShelfItemBackground(SkColor color) {
   if (background_) {
     background_->set_color(color);
@@ skipping 39 matching lines @@
   // When an action is performed, ink drop ripple is handled in SetIsActive().
   if (action_performed)
     return;
   ActionableView::HandlePerformActionResult(action_performed, event);
 }
 
 void TrayBackgroundView::OnPaintFocus(gfx::Canvas* canvas) {
   // The tray itself expands to the right and bottom edge of the screen to make
   // sure clicking on the edges brings up the popup. However, the focus border
   // should be only around the container.
-  gfx::RectF paint_bounds;
-  if (MaterialDesignController::IsShelfMaterial()) {
-    paint_bounds = gfx::RectF(GetBackgroundBounds());
-    paint_bounds.Inset(gfx::Insets(-kFocusBorderThickness));
-  } else {
-    paint_bounds = gfx::RectF(GetContentsBounds());
-    paint_bounds.Inset(gfx::Insets(1));
-  }
+  gfx::RectF paint_bounds(GetBackgroundBounds());
+  paint_bounds.Inset(gfx::Insets(-kFocusBorderThickness));
   canvas->DrawSolidFocusRect(paint_bounds, kFocusBorderColor,
                              kFocusBorderThickness);
 }
 
 void TrayBackgroundView::OnPaint(gfx::Canvas* canvas) {
   ActionableView::OnPaint(canvas);
-  if (!MaterialDesignController::IsShelfMaterial() ||
-      shelf()->GetBackgroundType() ==
+  if (shelf()->GetBackgroundType() ==
           ShelfBackgroundType::SHELF_BACKGROUND_DEFAULT ||
       !separator_visible_) {
     return;
   }
   // In the given |canvas|, for a horizontal shelf draw a separator line to the
   // right or left of the TrayBackgroundView when the system is LTR or RTL
   // aligned, respectively. For a vertical shelf draw the separator line
   // underneath the items instead.
   const bool horizontal_shelf = IsHorizontalAlignment(shelf_alignment_);
   const gfx::Rect local_bounds = GetLocalBounds();
@@ skipping 38 matching lines @@
 }
 
 gfx::Rect TrayBackgroundView::GetBackgroundBounds() const {
   gfx::Insets insets = GetBackgroundInsets();
   gfx::Rect bounds = GetLocalBounds();
   bounds.Inset(insets);
   return bounds;
 }
 
 }  // namespace ash