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

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

Issue 1511003003: Use refs for non-null GraphicsContext, Scrollbar, etc. in scrollbar related code (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@ScrollbarRemove
Patch Set: yet another mac fix Created 5 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: 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 46ace1f5842f14e78e52e2089184b8bb27c6ff43..bb1973e992d0cc72c17d57facb325f287b41ec09 100644
--- a/third_party/WebKit/Source/core/paint/PaintLayerScrollableArea.h
+++ b/third_party/WebKit/Source/core/paint/PaintLayerScrollableArea.h
@@ -193,10 +193,10 @@ public:
bool isActive() const override;
bool isScrollCornerVisible() const override;
IntRect scrollCornerRect() const override;
- IntRect convertFromScrollbarToContainingWidget(const Scrollbar*, const IntRect&) const override;
- IntRect convertFromContainingWidgetToScrollbar(const Scrollbar*, const IntRect&) const override;
- IntPoint convertFromScrollbarToContainingWidget(const Scrollbar*, const IntPoint&) const override;
- IntPoint convertFromContainingWidgetToScrollbar(const Scrollbar*, const IntPoint&) const override;
+ IntRect convertFromScrollbarToContainingWidget(const Scrollbar&, const IntRect&) const override;
+ IntRect convertFromContainingWidgetToScrollbar(const Scrollbar&, const IntRect&) const override;
+ IntPoint convertFromScrollbarToContainingWidget(const Scrollbar&, const IntPoint&) const override;
+ IntPoint convertFromContainingWidgetToScrollbar(const Scrollbar&, const IntPoint&) const override;
int scrollSize(ScrollbarOrientation) const override;
IntPoint scrollPosition() const override;
DoublePoint scrollPositionDouble() const override;
@@ -348,7 +348,7 @@ private:
LayoutUnit verticalScrollbarStart(int minX, int maxX) const;
LayoutUnit horizontalScrollbarStart(int minX) const;
- IntSize scrollbarOffset(const Scrollbar*) const;
+ IntSize scrollbarOffset(const Scrollbar&) const;
void setHasHorizontalScrollbar(bool hasScrollbar);
void setHasVerticalScrollbar(bool hasScrollbar);

Powered by Google App Engine
This is Rietveld 408576698