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

Unified Diff: third_party/WebKit/Source/core/layout/LayoutFullScreen.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/LayoutFullScreen.cpp
diff --git a/third_party/WebKit/Source/core/layout/LayoutFullScreen.cpp b/third_party/WebKit/Source/core/layout/LayoutFullScreen.cpp
index 1f57ce56f81d6ed5634f8665298b8352c8a029a2..20b7d5c3750fa80ec8c39aa0b9416e2620f9ca71 100644
--- a/third_party/WebKit/Source/core/layout/LayoutFullScreen.cpp
+++ b/third_party/WebKit/Source/core/layout/LayoutFullScreen.cpp
@@ -80,8 +80,8 @@ void LayoutFullScreen::willBeDestroyed() {
DCHECK(!m_placeholder);
}
- // LayoutObjects are unretained, so notify the document (which holds a pointer to a LayoutFullScreen)
- // if its LayoutFullScreen is destroyed.
+ // LayoutObjects are unretained, so notify the document (which holds a pointer
+ // to a LayoutFullScreen) if its LayoutFullScreen is destroyed.
Fullscreen& fullscreen = Fullscreen::from(document());
if (fullscreen.fullScreenLayoutObject() == this)
fullscreen.fullScreenLayoutObjectDestroyed();
@@ -101,7 +101,9 @@ void LayoutFullScreen::updateStyle(LayoutObject* parent) {
fullscreenStyle->setDisplay(EDisplay::Flex);
fullscreenStyle->setJustifyContentPosition(ContentPositionCenter);
- // TODO (lajava): Since the FullScrenn layout object is anonymous, its Default Alignment (align-items) value can't be used to resolve its children Self Alignment 'auto' values.
+ // TODO (lajava): Since the FullScrenn layout object is anonymous, its Default
+ // Alignment (align-items) value can't be used to resolve its children Self
+ // Alignment 'auto' values.
fullscreenStyle->setAlignItemsPosition(ItemPositionCenter);
fullscreenStyle->setFlexDirection(FlowColumn);
@@ -143,17 +145,19 @@ LayoutObject* LayoutFullScreen::wrapLayoutObject(LayoutObject* object,
if (LayoutObject* parent = object->parent()) {
LayoutBlock* containingBlock = object->containingBlock();
DCHECK(containingBlock);
- // Since we are moving the |object| to a new parent |fullscreenLayoutObject|,
- // the line box tree underneath our |containingBlock| is not longer valid.
+ // Since we are moving the |object| to a new parent
+ // |fullscreenLayoutObject|, the line box tree underneath our
+ // |containingBlock| is not longer valid.
if (containingBlock->isLayoutBlockFlow())
toLayoutBlockFlow(containingBlock)->deleteLineBoxTree();
parent->addChildWithWritingModeOfParent(fullscreenLayoutObject, object);
object->remove();
- // Always just do a full layout to ensure that line boxes get deleted properly.
- // Because objects moved from |parent| to |fullscreenLayoutObject|, we want to
- // make new line boxes instead of leaving the old ones around.
+ // Always just do a full layout to ensure that line boxes get deleted
+ // properly.
+ // Because objects moved from |parent| to |fullscreenLayoutObject|, we
+ // want to make new line boxes instead of leaving the old ones around.
parent->setNeedsLayoutAndPrefWidthsRecalcAndFullPaintInvalidation(
LayoutInvalidationReason::Fullscreen);
containingBlock
« no previous file with comments | « third_party/WebKit/Source/core/layout/LayoutFrameSet.cpp ('k') | third_party/WebKit/Source/core/layout/LayoutGeometryMap.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698