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

Unified Diff: ui/native_theme/native_theme_base.cc

Issue 2763373002: Overlay scrollbars flush with window edge (Closed)
Patch Set: bokan comments addressed Created 3 years, 9 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: ui/native_theme/native_theme_base.cc
diff --git a/ui/native_theme/native_theme_base.cc b/ui/native_theme/native_theme_base.cc
index 4a2fb957ed0f5ebfbc1249410ed20b8e6ee846ab..37e87fcea627200bd9aab433f50bad45f1bf61ce 100644
--- a/ui/native_theme/native_theme_base.cc
+++ b/ui/native_theme/native_theme_base.cc
@@ -211,8 +211,7 @@ void NativeThemeBase::Paint(cc::PaintCanvas* canvas,
break;
case kScrollbarHorizontalThumb:
case kScrollbarVerticalThumb:
- PaintScrollbarThumb(canvas, part, state, rect,
- extra.scrollbar_thumb.scrollbar_theme);
+ PaintScrollbarThumb(canvas, part, state, rect, extra.scrollbar_thumb);
break;
case kScrollbarHorizontalTrack:
case kScrollbarVerticalTrack:
@@ -434,11 +433,12 @@ void NativeThemeBase::PaintScrollbarTrack(
DrawBox(canvas, rect, flags);
}
-void NativeThemeBase::PaintScrollbarThumb(cc::PaintCanvas* canvas,
- Part part,
- State state,
- const gfx::Rect& rect,
- ScrollbarOverlayColorTheme) const {
+void NativeThemeBase::PaintScrollbarThumb(
+ cc::PaintCanvas* canvas,
+ Part part,
+ State state,
+ const gfx::Rect& rect,
+ const NativeTheme::ScrollbarThumbExtraParams&) const {
const bool hovered = state == kHovered;
const int midx = rect.x() + rect.width() / 2;
const int midy = rect.y() + rect.height() / 2;
« ui/native_theme/native_theme_aura.cc ('K') | « ui/native_theme/native_theme_base.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698