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

Unified Diff: third_party/WebKit/Source/core/layout/LayoutMultiColumnSpannerPlaceholder.h

Issue 2400863005: Reformat comments in core/layout up until LayoutTableRow (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/LayoutMultiColumnSpannerPlaceholder.h
diff --git a/third_party/WebKit/Source/core/layout/LayoutMultiColumnSpannerPlaceholder.h b/third_party/WebKit/Source/core/layout/LayoutMultiColumnSpannerPlaceholder.h
index 450e6a8b823286a79bf69c1da4a4738a12d79677..e164460adc7bc49dc998439774c631614108c106 100644
--- a/third_party/WebKit/Source/core/layout/LayoutMultiColumnSpannerPlaceholder.h
+++ b/third_party/WebKit/Source/core/layout/LayoutMultiColumnSpannerPlaceholder.h
@@ -9,10 +9,12 @@
namespace blink {
-// Placeholder layoutObject for column-span:all elements. The column-span:all layoutObject itself is a
-// descendant of the flow thread, but due to its out-of-flow nature, we need something on the
-// outside to take care of its positioning and sizing. LayoutMultiColumnSpannerPlaceholder objects
-// are siblings of LayoutMultiColumnSet objects, i.e. direct children of the multicol container.
+// Placeholder layoutObject for column-span:all elements. The column-span:all
+// layoutObject itself is a descendant of the flow thread, but due to its
+// out-of-flow nature, we need something on the outside to take care of its
+// positioning and sizing. LayoutMultiColumnSpannerPlaceholder objects are
+// siblings of LayoutMultiColumnSet objects, i.e. direct children of the
+// multicol container.
class LayoutMultiColumnSpannerPlaceholder final : public LayoutBox {
public:
bool isOfType(LayoutObjectType type) const override {
@@ -34,8 +36,9 @@ class LayoutMultiColumnSpannerPlaceholder final : public LayoutBox {
void markForLayoutIfObjectInFlowThreadNeedsLayout() {
if (!m_layoutObjectInFlowThread->needsLayout())
return;
- // The containing block of a spanner is the multicol container (our parent here), but the
- // spanner is laid out via its spanner set (us), so we need to make sure that we enter it.
+ // The containing block of a spanner is the multicol container (our parent
+ // here), but the spanner is laid out via its spanner set (us), so we need
+ // to make sure that we enter it.
setChildNeedsLayout(MarkOnlyThis);
}
@@ -67,8 +70,8 @@ class LayoutMultiColumnSpannerPlaceholder final : public LayoutBox {
private:
LayoutMultiColumnSpannerPlaceholder(LayoutBox*);
- LayoutBox*
- m_layoutObjectInFlowThread; // The actual column-span:all layoutObject inside the flow thread.
+ // The actual column-span:all layoutObject inside the flow thread.
+ LayoutBox* m_layoutObjectInFlowThread;
};
DEFINE_LAYOUT_OBJECT_TYPE_CASTS(LayoutMultiColumnSpannerPlaceholder,

Powered by Google App Engine
This is Rietveld 408576698