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

Unified Diff: trunk/src/chrome/browser/ui/views/tabs/tab_strip.cc

Issue 24262008: Revert 224473 "Remove dependency on ui::ScaleFactor from ui/gfx" (Closed) Base URL: svn://svn.chromium.org/chrome/
Patch Set: Created 7 years, 3 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
Index: trunk/src/chrome/browser/ui/views/tabs/tab_strip.cc
===================================================================
--- trunk/src/chrome/browser/ui/views/tabs/tab_strip.cc (revision 224498)
+++ trunk/src/chrome/browser/ui/views/tabs/tab_strip.cc (working copy)
@@ -393,8 +393,7 @@
#endif
void NewTabButton::OnPaint(gfx::Canvas* canvas) {
- gfx::ImageSkia image =
- GetImageForScale(ui::GetSupportedScaleFactor(canvas->image_scale()));
+ gfx::ImageSkia image = GetImageForScale(canvas->scale_factor());
canvas->DrawImageInt(image, 0, height() - image.height());
}
@@ -444,12 +443,12 @@
GetThemeProvider()->GetImageSkiaNamed(IDR_NEWTAB_BUTTON_MASK);
int height = mask->height();
int width = mask->width();
- float scale = ui::GetImageScale(scale_factor);
+
// The canvas and mask has to use the same scale factor.
- if (!mask->HasRepresentation(scale))
+ if (!mask->HasRepresentation(scale_factor))
scale_factor = ui::SCALE_FACTOR_100P;
- gfx::Canvas canvas(gfx::Size(width, height), scale, false);
+ gfx::Canvas canvas(gfx::Size(width, height), scale_factor, false);
// For custom images the background starts at the top of the tab strip.
// Otherwise the background starts at the top of the frame.
@@ -495,9 +494,7 @@
gfx::ImageSkia* overlay = GetThemeProvider()->GetImageSkiaNamed(overlay_id);
gfx::Canvas canvas(
- gfx::Size(overlay->width(), overlay->height()),
- ui::GetImageScale(scale_factor),
- false);
+ gfx::Size(overlay->width(), overlay->height()), scale_factor, false);
canvas.DrawImageInt(GetBackgroundImage(state, scale_factor), 0, 0);
// Draw the button border with a slight alpha.
« no previous file with comments | « trunk/src/chrome/browser/ui/views/tabs/tab.cc ('k') | trunk/src/chrome/browser/ui/web_applications/web_app_ui.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698