Rename AbsolutelyPositionedBox::layout to AbsolutelyPositionedBox::to…

…_hoisted This method doesn't actually do any layout, but converts this block to a HoistedAbsolutelyPositionedBox which is hoisted and then laid-out with its containing block later. This makes the code a little easier to read.
servo · Feb 28, 2020 · c20c468 · c20c468
1 parent 756cf66
commit c20c468
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/components/layout_2020/flow/inline.rs b/components/layout_2020/flow/inline.rs
@@ -255,7 +255,7 @@ impl InlineFormattingContext {
                                 },
                             };
                         ifc.positioning_context
-                            .push(box_.layout(initial_start_corner, tree_rank));
+                            .push(box_.to_hoisted(initial_start_corner, tree_rank));
                     },
                     InlineLevelBox::OutOfFlowFloatBox(_box_) => {
                         // TODO

diff --git a/components/layout_2020/flow/mod.rs b/components/layout_2020/flow/mod.rs
@@ -320,7 +320,7 @@ impl BlockLevelBox {
                 ))
             },
             BlockLevelBox::OutOfFlowAbsolutelyPositionedBox(box_) => {
-                positioning_context.push(box_.layout(Vec2::zero(), tree_rank));
+                positioning_context.push(box_.to_hoisted(Vec2::zero(), tree_rank));
                 Fragment::Anonymous(AnonymousFragment::no_op(
                     containing_block.style.writing_mode,
                 ))

diff --git a/components/layout_2020/positioned.rs b/components/layout_2020/positioned.rs
@@ -86,7 +86,7 @@ impl AbsolutelyPositionedBox {
         }
     }
 
-    pub(crate) fn layout(
+    pub(crate) fn to_hoisted(
         &self,
         initial_start_corner: Vec2<Length>,
         tree_rank: usize,