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

Unified Diff: third_party/WebKit/Source/platform/scroll/ScrollableArea.cpp

Issue 2387053003: reflow comments in platform/{mediastream,scroll} (Closed)
Patch Set: blah 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/platform/scroll/ScrollableArea.cpp
diff --git a/third_party/WebKit/Source/platform/scroll/ScrollableArea.cpp b/third_party/WebKit/Source/platform/scroll/ScrollableArea.cpp
index c2506d5401a4a1841e96e1dbba5957325c013a08..1184db856fce0f2a876889e27720a0f51ce5dcf5 100644
--- a/third_party/WebKit/Source/platform/scroll/ScrollableArea.cpp
+++ b/third_party/WebKit/Source/platform/scroll/ScrollableArea.cpp
@@ -217,9 +217,10 @@ void ScrollableArea::setScrollPositionSingleAxis(
else
newPosition = DoublePoint(scrollAnimator().currentPosition().x(), position);
- // TODO(bokan): Note, this doesn't use the derived class versions since this method is currently used
- // exclusively by code that adjusts the position by the scroll origin and the derived class versions
- // differ on whether they take that into account or not.
+ // TODO(bokan): Note, this doesn't use the derived class versions since this
+ // method is currently used exclusively by code that adjusts the position by
+ // the scroll origin and the derived class versions differ on whether they
+ // take that into account or not.
ScrollableArea::setScrollPosition(newPosition, scrollType, behavior);
}
@@ -245,10 +246,11 @@ void ScrollableArea::userScrollHelper(const DoublePoint& position,
? position.y()
: scrollAnimator().currentPosition().y();
- // Smooth user scrolls (keyboard, wheel clicks) are handled via the userScroll method.
- // TODO(bokan): The userScroll method should probably be modified to call this method
- // and ScrollAnimatorBase to have a simpler animateToOffset method like the
- // ProgrammaticScrollAnimator.
+ // Smooth user scrolls (keyboard, wheel clicks) are handled via the userScroll
+ // method.
+ // TODO(bokan): The userScroll method should probably be modified to call this
+ // method and ScrollAnimatorBase to have a simpler
+ // animateToOffset method like the ProgrammaticScrollAnimator.
ASSERT(scrollBehavior == ScrollBehaviorInstant);
scrollAnimator().scrollToOffsetWithoutAnimation(FloatPoint(x, y));
}
@@ -257,7 +259,8 @@ LayoutRect ScrollableArea::scrollIntoView(const LayoutRect& rectInContent,
const ScrollAlignment& alignX,
const ScrollAlignment& alignY,
ScrollType) {
- // TODO(bokan): This should really be implemented here but ScrollAlignment is in Core which is a dependency violation.
+ // TODO(bokan): This should really be implemented here but ScrollAlignment is
+ // in Core which is a dependency violation.
ASSERT_NOT_REACHED();
return LayoutRect();
}
@@ -360,7 +363,8 @@ void ScrollableArea::didAddScrollbar(Scrollbar& scrollbar,
else
scrollAnimator().didAddHorizontalScrollbar(scrollbar);
- // <rdar://problem/9797253> AppKit resets the scrollbar's style when you attach a scrollbar
+ // <rdar://problem/9797253> AppKit resets the scrollbar's style when you
+ // attach a scrollbar
setScrollbarOverlayStyle(getScrollbarOverlayStyle());
}

Powered by Google App Engine
This is Rietveld 408576698