Selaa lähdekoodia

LibHTML: Remove unused LayoutInline::layout()

Inline layout now happens in the containing block, specifically in
LayoutBlock::layout_inline_children().
Andreas Kling 5 vuotta sitten
vanhempi
commit
d68c1effcb

+ 0 - 27
Libraries/LibHTML/Layout/LayoutInline.cpp

@@ -1,4 +1,3 @@
-#include <LibHTML/DOM/Element.h>
 #include <LibHTML/Layout/LayoutBlock.h>
 #include <LibHTML/Layout/LayoutInline.h>
 
@@ -11,32 +10,6 @@ LayoutInline::~LayoutInline()
 {
 }
 
-void LayoutInline::layout()
-{
-    Point origin;
-
-    if (previous_sibling() != nullptr) {
-        auto& previous_sibling_rect = previous_sibling()->rect();
-        auto& previous_sibling_style = previous_sibling()->style();
-        origin = previous_sibling_rect.location();
-        // FIXME: Implement proper inline positioning when
-        // there are nodes with different heights. And don't
-        // hardcode font size like we do here.
-        origin.move_by(previous_sibling_rect.width(), previous_sibling_rect.height());
-        origin.move_by(previous_sibling_style.full_margin().right, -11);
-    } else {
-        origin = parent()->rect().location();
-    }
-
-    rect().set_location(origin);
-
-    for_each_child([&](auto& child) {
-        child.layout();
-        rect().set_right(child.rect().right() + child.style().full_margin().right);
-        rect().set_bottom(child.rect().bottom() + child.style().full_margin().bottom);
-    });
-}
-
 void LayoutInline::split_into_lines(LayoutBlock& container)
 {
     for_each_child([&](auto& child) {

+ 0 - 3
Libraries/LibHTML/Layout/LayoutInline.h

@@ -2,7 +2,6 @@
 
 #include <LibHTML/Layout/LayoutNode.h>
 
-class Element;
 class LayoutBlock;
 
 class LayoutInline : public LayoutNode {
@@ -13,8 +12,6 @@ public:
     virtual const char* class_name() const override { return "LayoutInline"; }
     virtual bool is_inline() const override { return true; }
 
-    virtual void layout() override;
-
     virtual void split_into_lines(LayoutBlock& container);
 
 private: