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

Unified Diff: ui/native_theme/native_theme_aura.cc

Issue 2426793002: Aura overlay scrollbars adjust color for dark backgrounds (Closed)
Patch Set: rename to ScrollbarOverlayColorTheme Created 4 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
Index: ui/native_theme/native_theme_aura.cc
diff --git a/ui/native_theme/native_theme_aura.cc b/ui/native_theme/native_theme_aura.cc
index d03a0d3211edc0f216a157fa5ff93b7ce22cab68..30e3b68caf08913ae56914b15f101e903280763a 100644
--- a/ui/native_theme/native_theme_aura.cc
+++ b/ui/native_theme/native_theme_aura.cc
@@ -35,7 +35,8 @@ constexpr int kOverlayScrollbarMinimumLength = 12;
constexpr SkAlpha kOverlayScrollbarAlphaNormal = 0x4D;
constexpr SkAlpha kOverlayScrollbarAlphaHovered = 0x80;
constexpr SkAlpha kOverlayScrollbarAlphaPressed = 0x80;
-constexpr SkColor kOverlayScrollbarThumbColor = SK_ColorBLACK;
+constexpr SkColor kOverlayScrollbarThumbDefaultColor = SK_ColorBLACK;
+constexpr SkColor kOverlayScrollbarThumbLightColor = SK_ColorWHITE;
constexpr SkColor kOverlayScrollbarStrokeColor = SK_ColorWHITE;
SkAlpha ThumbAlphaForState(NativeTheme::State state) {
@@ -177,15 +178,18 @@ void NativeThemeAura::PaintScrollbarTrack(
canvas->drawIRect(gfx::RectToSkIRect(rect), paint);
}
-void NativeThemeAura::PaintScrollbarThumb(SkCanvas* canvas,
- Part part,
- State state,
- const gfx::Rect& rect) const {
+void NativeThemeAura::PaintScrollbarThumb(
+ SkCanvas* canvas,
+ Part part,
+ State state,
+ const gfx::Rect& rect,
+ ScrollbarOverlayColorTheme theme) const {
// Do not paint if state is disabled.
if (state == kDisabled)
return;
- PaintScrollbarThumbStateTransition(canvas, part, state, state, 1.0, rect);
+ PaintScrollbarThumbStateTransition(canvas, part, state, state, 1.0, rect,
+ theme);
}
void NativeThemeAura::PaintScrollbarThumbStateTransition(
@@ -194,7 +198,8 @@ void NativeThemeAura::PaintScrollbarThumbStateTransition(
State start_state,
State end_state,
double progress,
- const gfx::Rect& rect) const {
+ const gfx::Rect& rect,
+ ScrollbarOverlayColorTheme theme) const {
TRACE_EVENT0("blink", "NativeThemeAura::PaintScrollbarThumbStateTransition");
gfx::Rect thumb_rect(rect);
SkColor thumb_color;
@@ -216,7 +221,10 @@ void NativeThemeAura::PaintScrollbarThumbStateTransition(
// Inset the all the edges edges so we fill-in the stroke below.
thumb_rect.Inset(kStrokeWidth, kStrokeWidth);
- thumb_color = kOverlayScrollbarThumbColor;
+ if (theme == ScrollbarOverlayColorThemeLight)
+ thumb_color = kOverlayScrollbarThumbLightColor;
+ else
+ thumb_color = kOverlayScrollbarThumbDefaultColor;
} else {
// If there are no scrollbuttons then provide some padding so that the thumb
// doesn't touch the top of the track.

Powered by Google App Engine
This is Rietveld 408576698