Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1496)

Unified Diff: chrome/browser/ui/gtk/tabs/tab_renderer_gtk.cc

Issue 8383028: ui/gfx: Convert Canvas::ClipRectInt() to use gfx::Rect. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix typo Created 9 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | chrome/browser/ui/views/infobars/infobar_view.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/gtk/tabs/tab_renderer_gtk.cc
diff --git a/chrome/browser/ui/gtk/tabs/tab_renderer_gtk.cc b/chrome/browser/ui/gtk/tabs/tab_renderer_gtk.cc
index 8867d752e28764b36f1d3f3bc81350fe273f96cd..26d957f99d42a0d872c89a8cab1cd1ae5f55d52c 100644
--- a/chrome/browser/ui/gtk/tabs/tab_renderer_gtk.cc
+++ b/chrome/browser/ui/gtk/tabs/tab_renderer_gtk.cc
@@ -834,7 +834,8 @@ void TabRendererGtk::PaintIcon(gfx::Canvas* canvas) {
PaintLoadingAnimation(canvas);
} else {
canvas->Save();
- canvas->ClipRectInt(0, 0, width(), height() - kFaviconTitleSpacing);
+ canvas->ClipRectInt(
+ gfx::Rect(0, 0, width(), height() - kFaviconTitleSpacing));
if (should_display_crashed_favicon_) {
canvas->DrawBitmapInt(*crashed_favicon, 0, 0,
crashed_favicon->width(),
@@ -978,8 +979,8 @@ void TabRendererGtk::PaintLoadingAnimation(gfx::Canvas* canvas) {
// NOTE: the clipping is a work around for 69528, it shouldn't be necessary.
canvas->Save();
- canvas->ClipRectInt(
- favicon_bounds_.x(), favicon_bounds_.y(), image_size, image_size);
+ canvas->ClipRectInt(gfx::Rect(favicon_bounds_.x(), favicon_bounds_.y(),
+ image_size, image_size));
canvas->DrawBitmapInt(*frames, image_offset, 0, image_size, image_size,
favicon_bounds_.x(), favicon_bounds_.y(), image_size, image_size,
false);
« no previous file with comments | « no previous file | chrome/browser/ui/views/infobars/infobar_view.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698