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

Side by Side Diff: third_party/WebKit/Source/core/paint/ScrollbarManager.h

Issue 2454913003: MainFrame scrollbars should work with RFV instead of FV (Closed)
Patch Set: Fix broken chromeos bot 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 unified diff | Download patch
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef ScrollbarManager_h 5 #ifndef ScrollbarManager_h
6 #define ScrollbarManager_h 6 #define ScrollbarManager_h
7 7
8 #include "core/CoreExport.h" 8 #include "core/CoreExport.h"
9 #include "platform/scroll/ScrollableArea.h" 9 #include "platform/scroll/ScrollableArea.h"
10 10
11 namespace blink { 11 namespace blink {
12 12
13 class CORE_EXPORT ScrollbarManager { 13 class CORE_EXPORT ScrollbarManager {
14 DISALLOW_NEW(); 14 DISALLOW_NEW();
15 15
16 // Helper class to manage the life cycle of Scrollbar objects. 16 // Helper class to manage the life cycle of Scrollbar objects.
17 public: 17 public:
18 ScrollbarManager(ScrollableArea&); 18 ScrollbarManager(ScrollableArea&);
19 19
20 // TODO(crbug.com/661236): The scroller should be the one that owns this
21 // ScrollbarManager and this setter should be removed.
22 // The scroller that this scrollbar it attached to.
23 void setScroller(ScrollableArea*);
24
20 void dispose(); 25 void dispose();
21 26
22 Scrollbar* horizontalScrollbar() const { 27 Scrollbar* horizontalScrollbar() const {
23 return m_hBarIsAttached ? m_hBar.get() : nullptr; 28 return m_hBarIsAttached ? m_hBar.get() : nullptr;
24 } 29 }
25 Scrollbar* verticalScrollbar() const { 30 Scrollbar* verticalScrollbar() const {
26 return m_vBarIsAttached ? m_vBar.get() : nullptr; 31 return m_vBarIsAttached ? m_vBar.get() : nullptr;
27 } 32 }
28 bool hasHorizontalScrollbar() const { return horizontalScrollbar(); } 33 bool hasHorizontalScrollbar() const { return horizontalScrollbar(); }
29 bool hasVerticalScrollbar() const { return verticalScrollbar(); } 34 bool hasVerticalScrollbar() const { return verticalScrollbar(); }
(...skipping 17 matching lines...) Expand all
47 Member<Scrollbar> m_hBar; 52 Member<Scrollbar> m_hBar;
48 Member<Scrollbar> m_vBar; 53 Member<Scrollbar> m_vBar;
49 54
50 unsigned m_hBarIsAttached : 1; 55 unsigned m_hBarIsAttached : 1;
51 unsigned m_vBarIsAttached : 1; 56 unsigned m_vBarIsAttached : 1;
52 }; 57 };
53 58
54 } // namespace blink 59 } // namespace blink
55 60
56 #endif // ScrollbarManager_h 61 #endif // ScrollbarManager_h
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698