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

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

Issue 2394263004: Reformat comments in core/layout up until LayoutMultiColumnFlowThread (Closed)
Patch Set: Rebase w/HEAD 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/LayoutBoxTest.cpp
diff --git a/third_party/WebKit/Source/core/layout/LayoutBoxTest.cpp b/third_party/WebKit/Source/core/layout/LayoutBoxTest.cpp
index 26c301096d7324e61ec69d6740c42156da5b10b6..7cbd7e182023fe976ff1ba80796b613ebbf2ef6b 100644
--- a/third_party/WebKit/Source/core/layout/LayoutBoxTest.cpp
+++ b/third_party/WebKit/Source/core/layout/LayoutBoxTest.cpp
@@ -55,15 +55,16 @@ TEST_F(LayoutBoxTest, BackgroundRect) {
"<div id='target5'></div>"
"<div id='target6'></div>");
- // #target1's opaque background color only fills the content box but its translucent image extends to the borders.
+ // #target1's opaque background color only fills the content box but its
+ // translucent image extends to the borders.
LayoutBox* layoutBox = toLayoutBox(getLayoutObjectByElementId("target1"));
EXPECT_EQ(LayoutRect(20, 20, 100, 100),
layoutBox->backgroundRect(BackgroundKnownOpaqueRect));
EXPECT_EQ(LayoutRect(0, 0, 140, 140),
layoutBox->backgroundRect(BackgroundClipRect));
- // #target2's background color is opaque but only fills the padding-box because it has local attachment.
- // This eclipses the content-box image.
+ // #target2's background color is opaque but only fills the padding-box
+ // because it has local attachment. This eclipses the content-box image.
layoutBox = toLayoutBox(getLayoutObjectByElementId("target2"));
EXPECT_EQ(LayoutRect(10, 10, 120, 120),
layoutBox->backgroundRect(BackgroundKnownOpaqueRect));
@@ -82,17 +83,17 @@ TEST_F(LayoutBoxTest, BackgroundRect) {
EXPECT_EQ(LayoutRect(0, 0, 140, 140),
layoutBox->backgroundRect(BackgroundClipRect));
- // #target5's solid background only covers the content-box but it has a "none" background
- // covering the border box.
+ // #target5's solid background only covers the content-box but it has a "none"
+ // background covering the border box.
layoutBox = toLayoutBox(getLayoutObjectByElementId("target5"));
EXPECT_EQ(LayoutRect(20, 20, 100, 100),
layoutBox->backgroundRect(BackgroundKnownOpaqueRect));
EXPECT_EQ(LayoutRect(0, 0, 140, 140),
layoutBox->backgroundRect(BackgroundClipRect));
- // Because it can scroll due to local attachment, the opaque local background in #target6
- // is treated as padding box for the clip rect, but remains the content box for the known
- // opaque rect.
+ // Because it can scroll due to local attachment, the opaque local background
+ // in #target6 is treated as padding box for the clip rect, but remains the
+ // content box for the known opaque rect.
layoutBox = toLayoutBox(getLayoutObjectByElementId("target6"));
EXPECT_EQ(LayoutRect(20, 20, 100, 100),
layoutBox->backgroundRect(BackgroundKnownOpaqueRect));

Powered by Google App Engine
This is Rietveld 408576698