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

Unified Diff: ui/native_theme/native_theme_aura.cc

Issue 2467693002: Implement overlay scrollbar fade out for non-composited scrollers. (Closed)
Patch Set: overlay-scrollbar-mouse-capture now works on Mac Created 4 years, 1 month 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 dc7166e6c491181340db32218f2f768dc48c7bab..6fa6f6c86569a1a8f01ced5a92034c5b7822c305 100644
--- a/ui/native_theme/native_theme_aura.cc
+++ b/ui/native_theme/native_theme_aura.cc
@@ -197,6 +197,9 @@ void NativeThemeAura::PaintScrollbarThumb(
SkAlpha thumb_alpha = ThumbAlphaForState(state);
if (IsOverlayScrollbarEnabled()) {
+ if (state == kDisabled)
skobes 2016/11/02 21:02:43 This seems redundant with an identical check at th
bokan 2016/11/02 22:41:57 Doh! Missed that :)
+ return;
+
thumb_color = kOverlayScrollbarThumbColor[theme];
// In overlay mode, draw a stroke (border).

Powered by Google App Engine
This is Rietveld 408576698