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

Unified Diff: ui/views/controls/scrollbar/cocoa_scroll_bar.mm

Issue 2555213004: Improving the appearance of overlay scrollbars (Closed)
Patch Set: attempt to fix mac Created 4 years 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/views/controls/scrollbar/cocoa_scroll_bar.mm
diff --git a/ui/views/controls/scrollbar/cocoa_scroll_bar.mm b/ui/views/controls/scrollbar/cocoa_scroll_bar.mm
index 40390cd1b95227b67b4369d2cb1166805f861ab6..9d54b673b7772dac0d8b2fdf12c75d8f286a4249 100644
--- a/ui/views/controls/scrollbar/cocoa_scroll_bar.mm
+++ b/ui/views/controls/scrollbar/cocoa_scroll_bar.mm
@@ -208,12 +208,12 @@ gfx::Rect CocoaScrollBar::GetTrackBounds() const {
//////////////////////////////////////////////////////////////////
// CocoaScrollBar, ScrollBar:
-int CocoaScrollBar::GetLayoutSize() const {
- return scroller_style_ == NSScrollerStyleOverlay ? 0 : ScrollbarThickness();
+int CocoaScrollBar::GetThickness() const {
+ return ScrollbarThickness();
}
-int CocoaScrollBar::GetContentOverlapSize() const {
- return scroller_style_ == NSScrollerStyleLegacy ? 0 : ScrollbarThickness();
+bool CocoaScrollBar::OverlapsContent() const {
+ return scroller_style_ == NSScrollerStyleOverlay;
}
//////////////////////////////////////////////////////////////////

Powered by Google App Engine
This is Rietveld 408576698