Attachment 356818 Details for Bug 192500 – patch

[patch] patch

formattingcontext-rename.patch (text/plain), 6.42 KB, created by Antti Koivisto on 2018-12-07 10:16:21 PST

(hide)

Description:

Filename:

MIME Type:

Creator: Antti Koivisto

Created: 2018-12-07 10:16:21 PST

Size: 6.42 KB

patch

obsolete

>Index: Source/WebCore/ChangeLog
>===================================================================
>--- Source/WebCore/ChangeLog	(revision 238952)
>+++ Source/WebCore/ChangeLog	(working copy)
>@@ -1,3 +1,23 @@
>+2018-12-07  Antti Koivisto  <antti@apple.com>
>+
>+        [LFC] Rename formattingContext() to createFormattingContext()
>+        https://bugs.webkit.org/show_bug.cgi?id=192500
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        * layout/FormattingContext.cpp:
>+        (WebCore::Layout::FormattingContext::layoutOutOfFlowDescendants const):
>+        * layout/FormattingState.h:
>+        * layout/LayoutFormattingState.cpp:
>+        (WebCore::Layout::LayoutState::layoutFormattingContextSubtree):
>+        * layout/blockformatting/BlockFormattingState.cpp:
>+        (WebCore::Layout::BlockFormattingState::createFormattingContext):
>+        (WebCore::Layout::BlockFormattingState::formattingContext): Deleted.
>+        * layout/blockformatting/BlockFormattingState.h:
>+        * layout/inlineformatting/InlineFormattingContext.cpp:
>+        (WebCore::Layout::InlineFormattingContext::layoutFormattingContextRoot const):
>+        * layout/inlineformatting/InlineFormattingState.h:
>+
> 2018-12-07  Eric Carlson  <eric.carlson@apple.com>
> 
>         [iOS] Don't update AVPlayerViewController currentTime while scrubbing
>Index: Source/WebCore/layout/FormattingContext.cpp
>===================================================================
>--- Source/WebCore/layout/FormattingContext.cpp	(revision 238859)
>+++ Source/WebCore/layout/FormattingContext.cpp	(working copy)
>@@ -154,7 +154,7 @@ void FormattingContext::layoutOutOfFlowD
>         computeBorderAndPadding(layoutBox);
>         computeOutOfFlowHorizontalGeometry(layoutBox);
> 
>-        layoutState.createFormattingStateForFormattingRootIfNeeded(layoutBox).formattingContext(layoutBox)->layout();
>+        layoutState.createFormattingStateForFormattingRootIfNeeded(layoutBox).createFormattingContext(layoutBox)->layout();
> 
>         computeOutOfFlowVerticalGeometry(layoutBox);
>         layoutOutOfFlowDescendants(layoutBox);
>Index: Source/WebCore/layout/FormattingState.h
>===================================================================
>--- Source/WebCore/layout/FormattingState.h	(revision 238859)
>+++ Source/WebCore/layout/FormattingState.h	(working copy)
>@@ -46,7 +46,7 @@ class FormattingState {
> public:
>     virtual ~FormattingState();
> 
>-    virtual std::unique_ptr<FormattingContext>formattingContext(const Box& formattingContextRoot) = 0;
>+    virtual std::unique_ptr<FormattingContext> createFormattingContext(const Box& formattingContextRoot) = 0;
> 
>     FloatingState& floatingState() const { return m_floatingState; }
> 
>Index: Source/WebCore/layout/LayoutFormattingState.cpp
>===================================================================
>--- Source/WebCore/layout/LayoutFormattingState.cpp	(revision 238859)
>+++ Source/WebCore/layout/LayoutFormattingState.cpp	(working copy)
>@@ -78,7 +78,7 @@ void LayoutState::layoutFormattingContex
> {
>     RELEASE_ASSERT(layoutRoot.establishesFormattingContext());
>     auto& formattingState = createFormattingStateForFormattingRootIfNeeded(layoutRoot);
>-    auto formattingContext = formattingState.formattingContext(layoutRoot);
>+    auto formattingContext = formattingState.createFormattingContext(layoutRoot);
>     formattingContext->layout();
>     formattingContext->layoutOutOfFlowDescendants(layoutRoot);
> }
>Index: Source/WebCore/layout/blockformatting/BlockFormattingState.cpp
>===================================================================
>--- Source/WebCore/layout/blockformatting/BlockFormattingState.cpp	(revision 238859)
>+++ Source/WebCore/layout/blockformatting/BlockFormattingState.cpp	(working copy)
>@@ -46,7 +46,7 @@ BlockFormattingState::~BlockFormattingSt
> {
> }
> 
>-std::unique_ptr<FormattingContext> BlockFormattingState::formattingContext(const Box& formattingContextRoot)
>+std::unique_ptr<FormattingContext> BlockFormattingState::createFormattingContext(const Box& formattingContextRoot)
> {
>     ASSERT(formattingContextRoot.establishesBlockFormattingContext());
>     return std::make_unique<BlockFormattingContext>(formattingContextRoot, *this);
>Index: Source/WebCore/layout/blockformatting/BlockFormattingState.h
>===================================================================
>--- Source/WebCore/layout/blockformatting/BlockFormattingState.h	(revision 238859)
>+++ Source/WebCore/layout/blockformatting/BlockFormattingState.h	(working copy)
>@@ -41,7 +41,7 @@ public:
>     BlockFormattingState(Ref<FloatingState>&&, LayoutState&);
>     virtual ~BlockFormattingState();
> 
>-    std::unique_ptr<FormattingContext> formattingContext(const Box& formattingContextRoot) override;
>+    std::unique_ptr<FormattingContext> createFormattingContext(const Box& formattingContextRoot) override;
> };
> 
> }
>Index: Source/WebCore/layout/inlineformatting/InlineFormattingContext.cpp
>===================================================================
>--- Source/WebCore/layout/inlineformatting/InlineFormattingContext.cpp	(revision 238859)
>+++ Source/WebCore/layout/inlineformatting/InlineFormattingContext.cpp	(working copy)
>@@ -373,7 +373,7 @@ void InlineFormattingContext::layoutForm
>     computeBorderAndPadding(root);
>     computeWidthAndMargin(root);
>     // Swich over to the new formatting context (the one that the root creates).
>-    auto formattingContext = layoutState().createFormattingStateForFormattingRootIfNeeded(root).formattingContext(root);
>+    auto formattingContext = layoutState().createFormattingStateForFormattingRootIfNeeded(root).createFormattingContext(root);
>     formattingContext->layout();
>     // Come back and finalize the root's height and margin.
>     computeHeightAndMargin(root);
>Index: Source/WebCore/layout/inlineformatting/InlineFormattingState.h
>===================================================================
>--- Source/WebCore/layout/inlineformatting/InlineFormattingState.h	(revision 238859)
>+++ Source/WebCore/layout/inlineformatting/InlineFormattingState.h	(working copy)
>@@ -43,7 +43,7 @@ public:
>     InlineFormattingState(Ref<FloatingState>&&, LayoutState&);
>     virtual ~InlineFormattingState();
> 
>-    std::unique_ptr<FormattingContext> formattingContext(const Box& formattingContextRoot) override;
>+    std::unique_ptr<FormattingContext> createFormattingContext(const Box& formattingContextRoot) override;
> 
>     InlineContent& inlineContent() { return m_inlineContent; }
>     InlineItem* lastInlineItem() const { return m_inlineContent.isEmpty() ? nullptr : m_inlineContent.last().get(); }

Flags:

zalan: review+

Actions: View | Formatted Diff | Diff

Attachments on bug 192500: 356818 | 356819