Support tooltip for HoverHighlightView

ash/system/tray/hover_highlight_view.cc

 // Copyright 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/system/tray/hover_highlight_view.h"
 
 #include "ash/system/tray/fixed_sized_image_view.h"
 #include "ash/system/tray/tray_constants.h"
 #include "ash/system/tray/view_click_listener.h"
 #include "ui/accessibility/ax_view_state.h"
@@ skipping 222 matching lines @@
 
 void HoverHighlightView::OnPaintBackground(gfx::Canvas* canvas) {
   canvas->DrawColor(hover_ ? highlight_color_ : default_color_);
 }
 
 void HoverHighlightView::OnFocus() {
   ScrollRectToVisible(gfx::Rect(gfx::Point(), size()));
   ActionableView::OnFocus();
 }
 
+bool HoverHighlightView::GetTooltipText(const gfx::Point& p,
+                                        base::string16* tooltip) const {
+  if (tooltip_.length()) {
+    *tooltip = tooltip_;
+    return true;
+  } else {
+    return false;
+  }

[stevenjb, 2015/11/23 20:26:24]

if (tooltip_.empty())
  return false;
tooltip = tooltip_;
return true;

[fqj, 2015/11/23 20:39:03]

Done.

+}
+
+void HoverHighlightView::SetTooltipText(const base::string16& tooltip) {
+  tooltip_ = tooltip;
+}

[stevenjb, 2015/11/23 20:26:24]

Trivial setters like this should be inlined in the header (and don't require a
comment):

void set_tooltip(...) { tooltip_ = tooltip; }

[fqj, 2015/11/23 20:39:03]

Done.

+
 }  // namespace ash