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

Unified Diff: ui/views/controls/scrollbar/scroll_bar.h

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/scroll_bar.h
diff --git a/ui/views/controls/scrollbar/scroll_bar.h b/ui/views/controls/scrollbar/scroll_bar.h
index acda1430ff1bd832645f26461f409ed3833c9c9d..ac162b70f5c9ad9078faa84d44694d039e91f064 100644
--- a/ui/views/controls/scrollbar/scroll_bar.h
+++ b/ui/views/controls/scrollbar/scroll_bar.h
@@ -81,14 +81,14 @@ class VIEWS_EXPORT ScrollBar : public View {
// Returns the position of the scrollbar.
virtual int GetPosition() const = 0;
- // Get the width or height of this scrollbar, for use in layout calculations.
- // For a vertical scrollbar, this is the width of the scrollbar, likewise it
- // is the height for a horizontal scrollbar.
- virtual int GetLayoutSize() const = 0;
-
- // Get the width or height for this scrollbar which overlaps with the content.
- // Default is 0.
- virtual int GetContentOverlapSize() const;
+ // Get the width or height of this scrollbar. For a vertical scrollbar, this
+ // is the width of the scrollbar, likewise it is the height for a horizontal
+ // scrollbar.
+ virtual int GetThickness() const = 0;
+
+ // Returns true if the scrollbar should sit on top of the content area (e.g.
+ // for overlay scrollbars).
+ virtual bool OverlapsContent() const = 0;
// Called when a ScrollEvent (in any, or no, direction) is seen by the parent
// ScrollView. E.g., this may reveal an overlay scrollbar to indicate

Powered by Google App Engine
This is Rietveld 408576698