Support tooltip for HoverHighlightView

ash/system/tray/hover_highlight_view.h

Index: ash/system/tray/hover_highlight_view.h
diff --git a/ash/system/tray/hover_highlight_view.h b/ash/system/tray/hover_highlight_view.h
index 6b9d1a22a42dada37b43bc1e360a19c348de0b48..0bf86d85240876b8ba9987e2a228434ba1769559 100644
--- a/ash/system/tray/hover_highlight_view.h
+++ b/ash/system/tray/hover_highlight_view.h
@@ -66,6 +66,10 @@ class HoverHighlightView : public ActionableView {
 
   bool hover() const { return hover_; }
 
+  bool GetTooltipText(const gfx::Point& p,
+                      base::string16* tooltip) const override;

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

Overrides should be in a separate commented section, typically before other
functions.

// views::View
bool GetTooltipText...

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

Done.

+  void SetTooltipText(const base::string16& tooltip);

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

New methods should have a comment.

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

Updated to a trivial setters in headers with implementations.
And you said trivial setters don't require one.

+
  protected:
   // Overridden from views::View.
   void GetAccessibleState(ui::AXViewState* state) override;
@@ -103,6 +107,7 @@ class HoverHighlightView : public ActionableView {
   bool expandable_;
   bool checkable_;
   bool checked_;
+  base::string16 tooltip_;
 
   DISALLOW_COPY_AND_ASSIGN(HoverHighlightView);
 };