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

Unified Diff: third_party/WebKit/Source/web/WebLocalFrameImpl.cpp

Issue 2387883002: Use float for scroll offset. (Closed)
Patch Set: rebase Created 4 years, 2 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/web/WebLocalFrameImpl.cpp
diff --git a/third_party/WebKit/Source/web/WebLocalFrameImpl.cpp b/third_party/WebKit/Source/web/WebLocalFrameImpl.cpp
index 327ccfd7a8de28705e711b5d2d29e61d826b903c..2a243a2a95242d4a129a5349d51f69479a7b2561 100644
--- a/third_party/WebKit/Source/web/WebLocalFrameImpl.cpp
+++ b/third_party/WebKit/Source/web/WebLocalFrameImpl.cpp
@@ -605,14 +605,15 @@ bool WebLocalFrameImpl::isFocused() const {
WebSize WebLocalFrameImpl::scrollOffset() const {
if (ScrollableArea* scrollableArea = layoutViewportScrollableArea())
- return toIntSize(scrollableArea->scrollPosition());
+ return scrollableArea->scrollOffsetInt();
return WebSize();
}
void WebLocalFrameImpl::setScrollOffset(const WebSize& offset) {
- if (ScrollableArea* scrollableArea = layoutViewportScrollableArea())
- scrollableArea->setScrollPosition(IntPoint(offset.width, offset.height),
- ProgrammaticScroll);
+ if (ScrollableArea* scrollableArea = layoutViewportScrollableArea()) {
+ scrollableArea->setScrollOffset(ScrollOffset(offset.width, offset.height),
+ ProgrammaticScroll);
+ }
}
WebSize WebLocalFrameImpl::contentsSize() const {

Powered by Google App Engine
This is Rietveld 408576698