ui/gfx: Convert Canvas::DrawFocusRect() to use gfx::Rect.

views/view.cc

 // Copyright (c) 2011 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 "views/view.h"
 
 #include <algorithm>
 
 #include "base/debug/trace_event.h"
 #include "base/logging.h"
@@ skipping 1061 matching lines @@
     background_->Paint(canvas, this);
 }
 
 void View::OnPaintBorder(gfx::Canvas* canvas) {
   if (border_.get())
     border_->Paint(*this, canvas);
 }
 
 void View::OnPaintFocusBorder(gfx::Canvas* canvas) {
   if ((IsFocusable() || IsAccessibilityFocusableInRootView()) && HasFocus())
-    canvas->DrawFocusRect(0, 0, width(), height());
+    canvas->DrawFocusRect(gfx::Rect(0, 0, width(), height()));

[Peter Kasting, 2011/10/24 22:15:13]

Nit: Just pass GetLocalBounds() instead.

(While you're in this file, feel free to fix that function to return
"gfx::Rect(gfx::Point(), size());".)

[tfarina, 2011/10/25 02:04:53]

Done.

 }
 
 // Accelerated Painting --------------------------------------------------------
 
 void View::SetFillsBoundsOpaquely(bool fills_bounds_opaquely) {
   // This method should not have the side-effect of creating the layer.
   if (layer())
     layer()->SetFillsBoundsOpaquely(fills_bounds_opaquely);
 }
 
@@ skipping 979 matching lines @@
 
   OSExchangeData data;
   WriteDragData(press_pt, &data);
 
   // Message the RootView to do the drag and drop. That way if we're removed
   // the RootView can detect it and avoid calling us back.
   GetWidget()->RunShellDrag(this, data, drag_operations);
 }
 
 }  // namespace views