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

views/desktop/desktop_background.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/desktop/desktop_background.h"
 
 #include "ui/gfx/canvas_skia.h"
 #include "ui/gfx/path.h"
 #include "views/view.h"
 
 namespace views {
 namespace desktop {
 
 DesktopBackground::DesktopBackground() {
 }
 
 DesktopBackground::~DesktopBackground() {
 }
 
 void DesktopBackground::Paint(gfx::Canvas* canvas, View* view) const {
   // Paint the sky.
-  canvas->FillRectInt(SK_ColorCYAN, 0, 0, view->width(), view->width());
+  canvas->FillRect(SK_ColorCYAN, gfx::Rect(0, 0, view->width(), view->width()));

[Peter Kasting, 2011/10/27 19:17:29]

I think the double-use of width() here is a bug (Ben agrees).  I think you
should pass view->GetLocalBounds() here.

[tfarina, 2011/10/27 20:46:03]

Done.

 
   SkPaint paint;
   paint.setAntiAlias(true);
   paint.setStyle(SkPaint::kFill_Style);
 
   // Paint the rolling fields of green.
   {
     gfx::Path path;
     path.moveTo(0, view->height() / 2);
     path.cubicTo(view->height() / 4, view->height() / 4,
@@ skipping 13 matching lines @@
     path.addCircle(view->height() / 4, view->height() / 8, view->height() / 16);
     path.close();
 
     paint.setColor(SK_ColorYELLOW);
     canvas->GetSkCanvas()->drawPath(path, paint);
   }
 }
 
 }  // namespace desktop
 }  // namespace views