MacViews: support backgrounds for selected rows

ui/views/controls/tree/tree_view.h

 // 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.
 
 #ifndef UI_VIEWS_CONTROLS_TREE_TREE_VIEW_VIEWS_H_
 #define UI_VIEWS_CONTROLS_TREE_TREE_VIEW_VIEWS_H_
 
 #include <memory>
 #include <vector>
 
@@ skipping 276 matching lines @@
   void PaintExpandControl(gfx::Canvas* canvas,
                           const gfx::Rect& node_bounds,
                           bool expanded);
 
   // Returns the InternalNode for a model node. |create_type| indicates wheter
   // this should load InternalNode or not.
   InternalNode* GetInternalNodeForModelNode(
       ui::TreeModelNode* model_node,
       GetInternalNodeCreateType create_type);
 
+  // Returns the bounds for a node's background.
+  gfx::Rect GetBackgroundBoundsForNode(InternalNode* node);
+
   // Returns the bounds for a node.

[tapted, 2016/06/15 03:17:50]

nit: update comment

[Elly Fong-Jones, 2016/06/16 14:42:34]

Done.

-  gfx::Rect GetBoundsForNode(InternalNode* node);
+  gfx::Rect GetTextBoundsForNode(InternalNode* node);
 
   // Implementation of GetBoundsForNode. Separated out as some callers already

[tapted, 2016/06/15 03:17:50]

nit: update comment

[Elly Fong-Jones, 2016/06/16 14:42:34]

Done.

   // know the row/depth.
-  gfx::Rect GetBoundsForNodeImpl(InternalNode* node, int row, int depth);
+  gfx::Rect GetTextBoundsForNodeImpl(InternalNode* node, int row, int depth);
 
   // Returns the row and depth of a node.
   int GetRowForInternalNode(InternalNode* node, int* depth);
 
   // Returns the row and depth of the specified node.
   InternalNode* GetNodeByRow(int row, int* depth);
 
   // Implementation of GetNodeByRow. |curent_row| is updated as we iterate.
   InternalNode* GetNodeByRowImpl(InternalNode* node,
                                  int target_row,
@@ skipping 11 matching lines @@
   // If the selected node is collapsed, it's expanded. Otherwise the first child
   // is seleected.
   void ExpandOrSelectChild();
 
   // Implementation of Expand(). Returns true if at least one node was expanded
   // that previously wasn't.
   bool ExpandImpl(ui::TreeModelNode* model_node);
 
   PrefixSelector* GetPrefixSelector();
 
+  // This function returns the InternalNode (if any) lying under |point|. If no

[tapted, 2016/06/15 03:17:50]

nit: remove "This function" / "this function". 

[Elly Fong-Jones, 2016/06/16 14:42:34]

Done.

+  // node lies under |point|, this function returns nullptr.
+  InternalNode* NodeAtPoint(const gfx::Point& point);
+
+  // This function returns whether |point| is in the bounds of |node|'s

[tapted, 2016/06/15 03:17:50]

nit: Returns whether..

[Elly Fong-Jones, 2016/06/16 14:42:34]

Done.

+  // expand/collapse arrow.
+  bool HitsNodeArrow(InternalNode* node, const gfx::Point& point);
+
   // The model, may be null.
   ui::TreeModel* model_;
 
   // Default icons for closed/open.
   gfx::ImageSkia closed_icon_;
   gfx::ImageSkia open_icon_;
 
   // Icons from the model.
   std::vector<gfx::ImageSkia> icons_;
 
@@ skipping 43 matching lines @@
   int text_offset_;
 
   std::unique_ptr<PrefixSelector> selector_;
 
   DISALLOW_COPY_AND_ASSIGN(TreeView);
 };
 
 }  // namespace views
 
 #endif  // UI_VIEWS_CONTROLS_TREE_TREE_VIEW_VIEWS_H_