Pass const gfx::Rect& as the first parameter to FillRect.

ui/views/controls/tree/tree_view_views.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 "ui/views/controls/tree/tree_view_views.h"
 
 #include <algorithm>
 
 #include "base/i18n/rtl.h"
 #include "base/message_loop.h"
@@ skipping 643 matching lines @@
   canvas->DrawBitmapInt(
       icon, icon_x,
       bounds.y() + (bounds.height() - icon.height()) / 2);
 
   if (!editor_ || node != selected_node_) {
     gfx::Rect text_bounds(bounds.x() + text_offset_, bounds.y(),
                           bounds.width() - text_offset_, bounds.height());
     if (base::i18n::IsRTL())
       text_bounds.set_x(bounds.x());
     if (node == selected_node_) {
-      canvas->FillRect(kSelectedBackgroundColor, text_bounds);
+      canvas->FillRect(text_bounds, kSelectedBackgroundColor);
       if (HasFocus())
         canvas->DrawFocusRect(text_bounds);
     }
     canvas->DrawStringInt(node->model_node()->GetTitle(), font_, kTextColor,
                           text_bounds.x() + kTextHorizontalPadding,
                           text_bounds.y() + kTextVerticalPadding,
                           text_bounds.width() - kTextHorizontalPadding * 2,
                           text_bounds.height() - kTextVerticalPadding * 2);
   }
 }
 
 void TreeView::PaintExpandControl(gfx::Canvas* canvas,
                                   const gfx::Rect& node_bounds,
                                   bool expanded) {
   int center_x;
   if (base::i18n::IsRTL()) {
     center_x = node_bounds.right() - kArrowRegionSize +
                (kArrowRegionSize - 4) / 2;
   } else {
     center_x = node_bounds.x() + (kArrowRegionSize - 4) / 2;
   }
   int center_y = node_bounds.y() + node_bounds.height() / 2;
   // TODO: this should come from an image.
   if (!expanded) {
     int delta = base::i18n::IsRTL() ? 1 : -1;
     for (int i = 0; i < 4; ++i) {
-      canvas->FillRect(kArrowColor,
-                       gfx::Rect(center_x + delta * (2 - i),
-                                 center_y - (3 - i), 1, (3 - i) * 2 + 1));
+      canvas->FillRect(gfx::Rect(center_x + delta * (2 - i),
+                                 center_y - (3 - i), 1, (3 - i) * 2 + 1),
+                       kArrowColor);
     }
   } else {
     center_y -= 2;
     for (int i = 0; i < 4; ++i) {
-      canvas->FillRect(kArrowColor,
-                       gfx::Rect(center_x - (3 - i), center_y + i,
-                                 (3 - i) * 2 + 1, 1));
+      canvas->FillRect(gfx::Rect(center_x - (3 - i), center_y + i,
+                                 (3 - i) * 2 + 1, 1), kArrowColor);
     }
   }
 }
 
 TreeView::InternalNode* TreeView::GetInternalNodeForModelNode(
     ui::TreeModelNode* model_node,
     GetInternalNodeCreateType create_type) {
   if (model_node == root_.model_node())
     return &root_;
   InternalNode* parent_internal_node =
@@ skipping 189 matching lines @@
   if (!is_expanded_)
     return max_width;
   for (int i = 0; i < child_count(); ++i) {
     max_width = std::max(max_width,
                          GetChild(i)->GetMaxWidth(indent, depth + 1));
   }
   return max_width;
 }
 
 }  // namespace views