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

Unified Diff: ui/base/test/scoped_preferred_scroller_style_mac.h

Issue 1671313002: MacViews: Overlay Scrollbars with Show/Hide Animations (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: nits Created 4 years, 10 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/base/test/scoped_preferred_scroller_style_mac.h
diff --git a/ui/base/test/scoped_preferred_scroller_style_legacy_mac.h b/ui/base/test/scoped_preferred_scroller_style_mac.h
similarity index 59%
rename from ui/base/test/scoped_preferred_scroller_style_legacy_mac.h
rename to ui/base/test/scoped_preferred_scroller_style_mac.h
index 99b0028db8fccad760749cb072caa12773280947..9e56130dd6b8cd068107b312d1f62ecb8b6ee3a0 100644
--- a/ui/base/test/scoped_preferred_scroller_style_legacy_mac.h
+++ b/ui/base/test/scoped_preferred_scroller_style_mac.h
@@ -17,17 +17,21 @@ class ScopedObjCClassSwizzler;
namespace ui {
namespace test {
-// Overrides system setting for scrollbar style with NSScrollerStyleLegacy,
-// which means "always show scrollbars".
-class ScopedPreferredScrollerStyleLegacy {
+// Overrides system setting for scrollbar style with NSScrollerOverlay if we
+// want the scrollbar to overlay. Otherwise, override with
+// NSScrollerStyleLegacy which means "always show scrollbars".
+class ScopedPreferredScrollerStyle {
public:
- ScopedPreferredScrollerStyleLegacy();
- ~ScopedPreferredScrollerStyleLegacy();
+ explicit ScopedPreferredScrollerStyle(bool overlay);
+ ~ScopedPreferredScrollerStyle();
private:
scoped_ptr<base::mac::ScopedObjCClassSwizzler> swizzler_;
- DISALLOW_COPY_AND_ASSIGN(ScopedPreferredScrollerStyleLegacy);
+ // True if the scrollbar style should overlay.
+ bool overlay_;
+
+ DISALLOW_COPY_AND_ASSIGN(ScopedPreferredScrollerStyle);
};
} // namespace test
« no previous file with comments | « ui/base/test/scoped_preferred_scroller_style_legacy_mac.mm ('k') | ui/base/test/scoped_preferred_scroller_style_mac.mm » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698