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

Unified Diff: third_party/WebKit/Source/core/paint/PaintLayerScrollableArea.h

Issue 2942163002: [Refactor] Moved scrollbar creation and deletion to ScrollbarManager (Closed)
Patch Set: bokan and skobes comments addressed Created 3 years, 6 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: third_party/WebKit/Source/core/paint/PaintLayerScrollableArea.h
diff --git a/third_party/WebKit/Source/core/paint/PaintLayerScrollableArea.h b/third_party/WebKit/Source/core/paint/PaintLayerScrollableArea.h
index 10c6f8b792397cccfd25f0166fb3b00db1f4e138..690667aaac10eced0d547be16005d57f9613bc89 100644
--- a/third_party/WebKit/Source/core/paint/PaintLayerScrollableArea.h
+++ b/third_party/WebKit/Source/core/paint/PaintLayerScrollableArea.h
@@ -143,10 +143,6 @@ class CORE_EXPORT PaintLayerScrollableArea final
void DestroyDetachedScrollbars();
- protected:
- Scrollbar* CreateScrollbar(ScrollbarOrientation) override;
- void DestroyScrollbar(ScrollbarOrientation) override;
-
PaintLayerScrollableArea* ScrollableArea();
};
@@ -228,6 +224,8 @@ class CORE_EXPORT PaintLayerScrollableArea final
bool HasHorizontalScrollbar() const { return HorizontalScrollbar(); }
bool HasVerticalScrollbar() const { return VerticalScrollbar(); }
+ void WillRemoveScrollbar(Scrollbar&, ScrollbarOrientation) override;
+
Scrollbar* HorizontalScrollbar() const override {
return scrollbar_manager_.HorizontalScrollbar();
}
@@ -280,6 +278,7 @@ class CORE_EXPORT PaintLayerScrollableArea final
int VisibleWidth() const override;
IntSize ContentsSize() const override;
void ContentsResized() override;
+ LayoutObject* LayoutObjectForScrollbarStyle() const override;
bool IsScrollable() const override;
IntPoint LastKnownMousePosition() const override;
bool ScrollAnimatorEnabled() const override;

Powered by Google App Engine
This is Rietveld 408576698