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

chrome/browser/ui/views/tabs/tab_strip.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_strip.h"
 
 #include <algorithm>
 #include <iterator>
 #include <vector>
 
@@ skipping 315 matching lines @@
     bool multiple_tabs_selected = (!selected_tabs.empty() ||
                                    tabs_dragging.size() > 1);
     // Make sure non-active tabs are somewhat transparent.
     SkPaint paint;
     // If there are multiple tabs selected, fade non-selected tabs more to make
     // the selected tabs more noticable.
     paint.setColor(SkColorSetARGB(
                        multiple_tabs_selected ? 150 : 200, 255, 255, 255));
     paint.setXfermodeMode(SkXfermode::kDstIn_Mode);
     paint.setStyle(SkPaint::kFill_Style);
-    canvas->DrawRectInt(0, 0, width(),
-        height() - 2,  // Visible region that overlaps the toolbar.
-        paint);
+    canvas->DrawRect(gfx::Rect(0, 0, width(),
+       height() - 2),  // Visible region that overlaps the toolbar.

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

Nit: I'd hoist this comment above the line as "// The tabstrip area overlaps the
toolbar area by 2 px.", then all the code can go on one line.

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

Done.

+       paint);
   }
 
   // Now selected but not active. We don't want these dimmed if using native
   // frame, so they're painted after initial pass.
   for (size_t i = 0; i < selected_tabs.size(); ++i)
     selected_tabs[i]->Paint(canvas);
 
   // Next comes the active tab.
   if (active_tab && !is_dragging)
     active_tab->Paint(canvas);
@@ skipping 696 matching lines @@
   }
   return mini_count;
 }
 
 bool TabStrip::IsPointInTab(Tab* tab,
                             const gfx::Point& point_in_tabstrip_coords) {
   gfx::Point point_in_tab_coords(point_in_tabstrip_coords);
   View::ConvertPointToView(this, tab, &point_in_tab_coords);
   return tab->HitTest(point_in_tab_coords);
 }