ui/native_theme/native_theme_switches.h - Issue 2175163005: Add flag to hide scrollbars completely.

Unified Diff: ui/native_theme/native_theme_switches.h

Issue 2175163005: Add flag to hide scrollbars completely. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master

Patch Set: add tests for scrollbar behavior without animator. Created 4 years, 5 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:

Index: ui/native_theme/native_theme_switches.h

diff --git a/ui/native_theme/native_theme_switches.h b/ui/native_theme/native_theme_switches.h

index 026ee70eca378be31944d0b8c56fd8d6d96583fc..b19caaeecc6cc4ae977e435b7dc89a6c34d4006d 100644

--- a/ui/native_theme/native_theme_switches.h

+++ b/ui/native_theme/native_theme_switches.h

@@ -13,12 +13,14 @@ namespace switches {

NATIVE_THEME_EXPORT extern const char kDisableOverlayScrollbar[];

NATIVE_THEME_EXPORT extern const char kEnableOverlayScrollbar[];

+NATIVE_THEME_EXPORT extern const char kEnableHiddenScrollbar[];

} // namespace switches

namespace ui {

NATIVE_THEME_EXPORT bool IsOverlayScrollbarEnabled();

+NATIVE_THEME_EXPORT bool IsHiddenScrollbarEnabled();

sky 2016/08/02 20:30:24 AFAICT ui/native_theme doesn't handle this switch

Eric Seckler 2016/08/03 17:34:54 It's used by IsOverlayScrollbarEnabled(), which is

sky 2016/08/03 23:44:03 Does it need to be? It seems like the call sites c

Eric Seckler 2016/08/04 08:07:25 Yeah, there are quite a few other callsites of IsO

} // namespace ui