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

ui/gfx/canvas.h

Index: ui/gfx/canvas.h
diff --git a/ui/gfx/canvas.h b/ui/gfx/canvas.h
index 96159578e5150433ccc64d1e805cdf36327105c3..7c056f1ff907955d230f8a19b717ce0722128e2c 100644
--- a/ui/gfx/canvas.h
+++ b/ui/gfx/canvas.h
@@ -122,20 +122,18 @@ class UI_EXPORT Canvas {
   // color, using a transfer mode of SkXfermode::kSrcOver_Mode.
   //
   // NOTE: if you need a single pixel line, use DrawLineInt.
-  virtual void DrawRectInt(const SkColor& color,
-                           int x, int y, int w, int h) = 0;
+  virtual void DrawRect(const SkColor& color, const gfx::Rect& rect) = 0;

[Peter Kasting, 2011/11/05 01:10:10]

Nit: Kinda drives me crazy that some of these functions put colors or brushes or
paints first, some last, etc.  Maybe we can standardize all the rect and line
drawing/filling functions to put coordinates first, other stuff after?

[tfarina, 2011/11/05 02:23:24]

Me too. Addressed for DrawRect. Can I fix the other functions in a follow up CL?

[Peter Kasting, 2011/11/21 19:26:03]

Sure.

 
   // Draws a single pixel rect in the specified region with the specified
   // color and transfer mode.
   //
   // NOTE: if you need a single pixel line, use DrawLineInt.
-  virtual void DrawRectInt(const SkColor& color,
-                           int x, int y, int w, int h,
-                           SkXfermode::Mode mode) = 0;
+  virtual void DrawRect(const SkColor& color,
+                        const gfx::Rect& rect,
+                        SkXfermode::Mode mode) = 0;
 
   // Draws the given rectangle with the given paint's parameters.
-  virtual void DrawRectInt(int x, int y, int w, int h,
-                           const SkPaint& paint) = 0;
+  virtual void DrawRect(const gfx::Rect& rect, const SkPaint& paint) = 0;
 
   // Draws a single pixel line with the specified color.
   virtual void DrawLineInt(const SkColor& color,