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

chrome/browser/ui/views/tabs/tab.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 "chrome/browser/ui/views/tabs/tab.h"
 
 #include <limits>
 
 #include "base/utf_string_conversions.h"
 #include "chrome/browser/defaults.h"
@@ skipping 384 matching lines @@
   colors[1] = kMiniTitleChangeGradientColor2;
   SkShader* shader = SkGradientShader::CreateRadial(
       loc,
       SkIntToScalar(radius),
       colors,
       NULL,
       2,
       SkShader::kClamp_TileMode);
   paint.setShader(shader);
   shader->unref();
-  hover_canvas.DrawRectInt(x - radius, -radius, radius * 2, radius * 2, paint);
+  hover_canvas.DrawRect(gfx::Rect(x - radius, -radius, radius * 2, radius * 2),
+                        paint);
 
   // Draw the radial gradient clipped to the background into hover_image.
   SkBitmap hover_image = SkBitmapOperations::CreateMaskedBitmap(
       hover_canvas.ExtractBitmap(), background_image);
 
   // Draw the tab background to the canvas.
   canvas->DrawBitmapInt(background_image, 0, 0);
 
   // And then the gradient on top of that.
   if (mini_title_animation_->current_part_index() == 2) {
@@ skipping 159 matching lines @@
       SkIntToScalar(radius),
       colors,
       NULL,
       2,
       SkShader::kClamp_TileMode);
   // Shader can end up null when radius = 0.
   // If so, this results in default full tab glow behavior.
   if (shader) {
     paint.setShader(shader);
     shader->unref();
-    hover_canvas.DrawRectInt(hover_point_.x() - radius,
-                             hover_point_.y() - radius,
-                             radius * 2, radius * 2, paint);
+    hover_canvas.DrawRect(gfx::Rect(hover_point_.x() - radius,

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

Nit: Seems cleaner:

    gfx::Rect paint_rect(hover_point_, gfx::Size());
    paint_rect.Inset(-radius, -radius);
    hover_canvas.DrawRect(paint_rect, paint);

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

Done.

+                                    hover_point_.y() - radius,
+                                    radius * 2, radius * 2), paint);
   }
   return hover_canvas.ExtractBitmap();
 }
 
 int Tab::IconCapacity() const {
   if (height() < GetMinimumUnselectedSize().height())
     return 0;
   return (width() - kLeftPadding - kRightPadding) / kTabIconSize;
 }
 
@@ skipping 57 matching lines @@
   tab_active_.image_r = rb.GetBitmapNamed(IDR_TAB_ACTIVE_RIGHT);
   tab_active_.l_width = tab_active_.image_l->width();
   tab_active_.r_width = tab_active_.image_r->width();
 
   tab_inactive_.image_l = rb.GetBitmapNamed(IDR_TAB_INACTIVE_LEFT);
   tab_inactive_.image_c = rb.GetBitmapNamed(IDR_TAB_INACTIVE_CENTER);
   tab_inactive_.image_r = rb.GetBitmapNamed(IDR_TAB_INACTIVE_RIGHT);
   tab_inactive_.l_width = tab_inactive_.image_l->width();
   tab_inactive_.r_width = tab_inactive_.image_r->width();
 }