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

Unified Diff: third_party/WebKit/Source/core/layout/ScrollAnchorTest.cpp

Issue 2403553002: reflow comments in core/layout/[M-Z]*.{cpp,h} (Closed)
Patch Set: 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/core/layout/ScrollAnchorTest.cpp
diff --git a/third_party/WebKit/Source/core/layout/ScrollAnchorTest.cpp b/third_party/WebKit/Source/core/layout/ScrollAnchorTest.cpp
index 2558ed33ab52b4e7e6919a608c794f3ed32b599a..60e58d7a6f792814e153818b217b46b0f6abb112 100644
--- a/third_party/WebKit/Source/core/layout/ScrollAnchorTest.cpp
+++ b/third_party/WebKit/Source/core/layout/ScrollAnchorTest.cpp
@@ -26,7 +26,8 @@ class ScrollAnchorTest : public RenderingTest {
protected:
void update() {
- // TODO(skobes): Use SimTest instead of RenderingTest and move into Source/web?
+ // TODO(skobes): Use SimTest instead of RenderingTest and move into
+ // Source/web?
document().view()->updateAllLifecyclePhases();
}
@@ -281,7 +282,8 @@ TEST_F(ScrollAnchorTest, AnchorWithLayerInScrollingDiv) {
EXPECT_EQ(250, scroller->scrollOffsetInt().height());
EXPECT_EQ(block2->layoutObject(), scrollAnchor(scroller).anchorObject());
- // Test that the anchor object can be destroyed without affecting the scroll position.
+ // Test that the anchor object can be destroyed without affecting the scroll
+ // position.
block2->remove();
update();
EXPECT_EQ(250, scroller->scrollOffsetInt().height());
@@ -414,7 +416,8 @@ TEST_F(ScrollAnchorTest, ExcludeAbsolutePositionThatSticksToViewport) {
scroller->scrollBy(ScrollOffset(0, 25), UserScroll);
setHeight(document().getElementById("a"), 100);
- // When the scroller is position:static, the anchor cannot be position:absolute.
+ // When the scroller is position:static, the anchor cannot be
+ // position:absolute.
EXPECT_EQ(relPos->layoutObject(), scrollAnchor(scroller).anchorObject());
scrollerElement->setAttribute(HTMLNames::styleAttr, "position: relative");
@@ -422,11 +425,13 @@ TEST_F(ScrollAnchorTest, ExcludeAbsolutePositionThatSticksToViewport) {
scroller->scrollBy(ScrollOffset(0, 25), UserScroll);
setHeight(document().getElementById("a"), 125);
- // When the scroller is position:relative, the anchor may be position:absolute.
+ // When the scroller is position:relative, the anchor may be
+ // position:absolute.
EXPECT_EQ(absPos->layoutObject(), scrollAnchor(scroller).anchorObject());
}
-// Test that we descend into zero-height containers that have overflowing content.
+// Test that we descend into zero-height containers that have overflowing
+// content.
TEST_F(ScrollAnchorTest, DescendsIntoContainerWithOverflow) {
setBodyInnerHTML(
"<style>"

Powered by Google App Engine
This is Rietveld 408576698