[ash-md] Updates ash shelf tooltips to MD

ash/shelf/shelf_tooltip_manager.cc

 // Copyright 2013 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/shelf/shelf_tooltip_manager.h"
 
+#include "ash/common/material_design/material_design_controller.h"
+#include "ash/common/shelf/shelf_constants.h"
 #include "ash/common/shell_window_ids.h"
 #include "ash/shelf/shelf.h"
 #include "ash/shelf/shelf_layout_manager.h"
 #include "ash/shelf/shelf_view.h"
 #include "ash/shell.h"
 #include "ash/wm/window_animations.h"
 #include "base/bind.h"
 #include "base/strings/string16.h"
 #include "base/threading/thread_task_runner_handle.h"
 #include "base/time/time.h"
 #include "ui/aura/window.h"
 #include "ui/events/event.h"
 #include "ui/events/event_constants.h"
 #include "ui/gfx/geometry/insets.h"
 #include "ui/views/bubble/bubble_dialog_delegate.h"
 #include "ui/views/controls/label.h"
 #include "ui/views/layout/fill_layout.h"
 #include "ui/views/widget/widget.h"
 #include "ui/wm/core/window_animations.h"
 
 namespace ash {
 namespace {
 
-const int kTooltipTopBottomMargin = 3;
-const int kTooltipLeftRightMargin = 10;
 const int kTooltipAppearanceDelay = 1000;  // msec
-const int kTooltipMinHeight = 29 - 2 * kTooltipTopBottomMargin;
 const SkColor kTooltipTextColor = SkColorSetRGB(0x22, 0x22, 0x22);
 
-// The maximum width of the tooltip bubble.  Borrowed the value from
-// ash/tooltip/tooltip_controller.cc
-const int kTooltipMaxWidth = 250;
-
-// The offset for the tooltip bubble - making sure that the bubble is flush
-// with the shelf. The offset includes the arrow size in pixels as well as
-// the activation bar and other spacing elements.
-const int kArrowOffsetLeftRight = 11;
-const int kArrowOffsetTopBottom = 7;
-
 }  // namespace
 
 // The implementation of tooltip of the launcher.
 class ShelfTooltipManager::ShelfTooltipBubble
     : public views::BubbleDialogDelegateView {
  public:
   ShelfTooltipBubble(views::View* anchor,
                      views::BubbleBorder::Arrow arrow,
                      const base::string16& text)
       : views::BubbleDialogDelegateView(anchor, arrow) {
     gfx::Insets insets =
-        gfx::Insets(kArrowOffsetTopBottom, kArrowOffsetLeftRight);
+        gfx::Insets(GetShelfConstant(TOOLTIP_ARROW_TOP_BOTTOM_OFFSET),
+                    GetShelfConstant(TOOLTIP_ARROW_LEFT_RIGHT_OFFSET));
     // Adjust the anchor location for asymmetrical borders of shelf item.
     if (anchor->border())
       insets += anchor->border()->GetInsets();
 
     set_anchor_view_insets(insets);
     set_close_on_deactivate(false);
     set_can_activate(false);
     set_accept_events(false);
-    set_margins(gfx::Insets(kTooltipTopBottomMargin, kTooltipLeftRightMargin));
-    set_shadow(views::BubbleBorder::SMALL_SHADOW);
+    set_margins(gfx::Insets(GetShelfConstant(TOOLTIP_TOP_BOTTOM_MARGIN),
+                            GetShelfConstant(TOOLTIP_LEFT_RIGHT_MARGIN)));
+    const bool material = ash::MaterialDesignController::IsTooltipMaterial();
+    set_shadow(material ? views::BubbleBorder::NO_ASSETS_TOOLTIP

[msw, 2016/07/22 16:51:06]

driveby: Can you use the existing NO_ASSETS setting with different (smaller)
margins? Avoiding a new bubble setting would be nice.

[varkha, 2016/07/22 22:03:56]

views::BubbleBorder::NO_ASSETS results in huge vertical margin that cannot be
reduced below 17. I could plumb through some way to adjust internal margins in
the BubbleBorder but I don't think this is any more elegant. Am I missing
something?

[msw, 2016/07/22 22:14:52]

There's only one user; SubtleNotificationView. Perhaps we can reduce the 17px
|border_interior_thickness| used by NO_ASSETS, and increase the internal
padding/margin of the SubtleNotificationView. +mgiuca

[msw, 2016/07/22 22:16:25]

Oops, there are multiple users; but the same thought holds. I suppose it's nbd
if we need to add another bubble border type just to reduce insets; it just
seems silly.

[varkha, 2016/07/23 01:09:42]

Yes, the margins that are settable with set_margin and the internal bubble
border border_thickness combine in fairly complex ways. Margins affect the
internal layout (define where the content appears) while the border thickness
doesn't. Also the overall height H can be defined by the border_thickness b (H
>= 2*b) but can be also defined by the content c and margins m (insets are
affected by margins but I am not sure if there are NO_ASSET cases with
additional insets from title, close button and such).
So H >= max((c + m) , 2*b).
Setting margins therefore affects the height in combination with the content
which can be different while border thickness imposes a minimum regardless of
contents.

There is probably some way to untangle this but with more than one call site and
multiple scenarios affecting it I can never be sure that I am not breaking some
positioning or layout on some platform.

So I would actually prefer plumbing through access to adjust border thickness at
runtime or alternatively add a new preset which seems like a legitimate
template.

On the other hand I think I need to increase the border thickness to 12 to make
the ash shelf tooltip be same height regardless of content.

+                        : views::BubbleBorder::SMALL_SHADOW);
     SetLayoutManager(new views::FillLayout());
     // The anchor may not have the widget in tests.
     if (anchor->GetWidget() && anchor->GetWidget()->GetNativeWindow()) {
       set_parent_window(ash::Shell::GetContainer(
           anchor->GetWidget()->GetNativeWindow()->GetRootWindow(),
           ash::kShellWindowId_SettingBubbleContainer));
     }
     views::Label* label = new views::Label(text);
     label->SetHorizontalAlignment(gfx::ALIGN_LEFT);
-    label->SetEnabledColor(kTooltipTextColor);
+    ui::NativeTheme* theme = anchor->GetWidget()->GetNativeTheme();
+    SkColor text_color =
+        material ? theme->GetSystemColor(ui::NativeTheme::kColorId_TooltipText)
+                 : kTooltipTextColor;
+    label->SetEnabledColor(text_color);
+    if (material) {
+      SkColor background_color =
+          theme->GetSystemColor(ui::NativeTheme::kColorId_TooltipBackground);
+      set_color(background_color);
+      label->SetBackgroundColor(background_color);
+    }
     AddChildView(label);
     views::BubbleDialogDelegateView::CreateBubble(this);
+    // The bubble border has its own background so the background created by the
+    // BubbleDialogDelegateView is redundant and would cause extra opacity.
+    if (material)
+      set_background(nullptr);
+    SetArrowPaintType(material ? views::BubbleBorder::PAINT_TRANSPARENT
+                               : views::BubbleBorder::PAINT_NORMAL);
   }
 
  private:
   // BubbleDialogDelegateView overrides:
   gfx::Size GetPreferredSize() const override {
     const gfx::Size size = BubbleDialogDelegateView::GetPreferredSize();
+    const int kTooltipMinHeight =
+        GetShelfConstant(TOOLTIP_HEIGHT) -
+        2 * GetShelfConstant(TOOLTIP_TOP_BOTTOM_MARGIN);
+    const int kTooltipMaxWidth = GetShelfConstant(TOOLTIP_MAX_WIDTH);
     return gfx::Size(std::min(size.width(), kTooltipMaxWidth),
                      std::max(size.height(), kTooltipMinHeight));
   }
 
   int GetDialogButtons() const override { return ui::DIALOG_BUTTON_NONE; }
 
   DISALLOW_COPY_AND_ASSIGN(ShelfTooltipBubble);
 };
 
 ShelfTooltipManager::ShelfTooltipManager(ShelfView* shelf_view)
@@ skipping 154 matching lines @@
         base::Bind(&ShelfTooltipManager::Close, weak_factory_.GetWeakPtr()));
   }
 }
 
 bool ShelfTooltipManager::ShouldShowTooltipForView(views::View* view) {
   return shelf_view_ && shelf_view_->ShouldShowTooltipForView(view) &&
          shelf_layout_manager_ && shelf_layout_manager_->IsVisible();
 }
 
 }  // namespace ash