Przeglądaj źródła

LibWeb: Merge Document::layout() and Document::update_layout()

There is now only Document::update_layout().
Andreas Kling 4 lat temu
rodzic
commit
3c9dcec442

+ 3 - 11
Libraries/LibWeb/DOM/Document.cpp

@@ -287,7 +287,7 @@ void Document::attach_to_frame(Badge<Frame>, Frame& frame)
         node.document_did_attach_to_frame(frame);
         return IterationDecision::Continue;
     });
-    layout();
+    update_layout();
 }
 
 void Document::detach_from_frame(Badge<Frame>, Frame& frame)
@@ -376,10 +376,10 @@ void Document::invalidate_layout()
 void Document::force_layout()
 {
     invalidate_layout();
-    layout();
+    update_layout();
 }
 
-void Document::layout()
+void Document::update_layout()
 {
     if (!frame())
         return;
@@ -410,14 +410,6 @@ void Document::update_style()
     update_layout();
 }
 
-void Document::update_layout()
-{
-    if (!frame())
-        return;
-
-    layout();
-}
-
 RefPtr<Layout::Node> Document::create_layout_node(const CSS::StyleProperties*)
 {
     return adopt(*new Layout::InitialContainingBlockBox(*this, CSS::StyleProperties::create()));

+ 0 - 1
Libraries/LibWeb/DOM/Document.h

@@ -117,7 +117,6 @@ public:
     Color visited_link_color() const;
     void set_visited_link_color(Color);
 
-    void layout();
     void force_layout();
     void invalidate_layout();
 

+ 1 - 1
Libraries/LibWeb/HTML/HTMLElement.cpp

@@ -113,7 +113,7 @@ String HTMLElement::inner_text()
     StringBuilder builder;
 
     // innerText for element being rendered takes visibility into account, so force a layout and then walk the layout tree.
-    document().layout();
+    document().update_layout();
     if (!layout_node())
         return text_content();
 

+ 1 - 1
Libraries/LibWeb/Page/Frame.cpp

@@ -103,7 +103,7 @@ void Frame::set_size(const Gfx::IntSize& size)
         return;
     m_size = size;
     if (m_document)
-        m_document->layout();
+        m_document->update_layout();
 }
 
 void Frame::set_viewport_scroll_offset(const Gfx::IntPoint& offset)

+ 1 - 1
Services/WebContent/PageHost.cpp

@@ -102,7 +102,7 @@ void PageHost::set_viewport_rect(const Gfx::IntRect& rect)
 {
     page().main_frame().set_size(rect.size());
     if (page().main_frame().document())
-        page().main_frame().document()->layout();
+        page().main_frame().document()->update_layout();
     page().main_frame().set_viewport_scroll_offset(rect.location());
 }