LibWeb: Make Paintable ref-counted

This will allow us to use a protective NonnullRefPtr to keep paintables
alive while running arbitrary JavaScript in response to events.

Author: awesomekling

Userland/Libraries/LibWeb/Layout/BlockContainer.cpp

@@ -85,7 +85,7 @@ Painting::PaintableWithLines const* BlockContainer::paint_box() const
     return static_cast<Painting::PaintableWithLines const*>(Box::paint_box());
 }
 
-OwnPtr<Painting::Paintable> BlockContainer::create_paintable() const
+RefPtr<Painting::Paintable> BlockContainer::create_paintable() const
 {
     return Painting::PaintableWithLines::create(*this);
 }

Userland/Libraries/LibWeb/Layout/BlockContainer.h

@@ -31,7 +31,7 @@ class BlockContainer : public Box {
 
     Painting::PaintableWithLines const* paint_box() const;
 
-    virtual OwnPtr<Painting::Paintable> create_paintable() const override;
+    virtual RefPtr<Painting::Paintable> create_paintable() const override;
 
 private:
     virtual bool is_block_container() const final { return true; }

Userland/Libraries/LibWeb/Layout/Box.cpp

@@ -86,7 +86,7 @@ bool Box::is_body() const
     return dom_node() && dom_node() == document().body();
 }
 
-OwnPtr<Painting::Paintable> Box::create_paintable() const
+RefPtr<Painting::Paintable> Box::create_paintable() const
 {
     return Painting::PaintableBox::create(*this);
 }

Userland/Libraries/LibWeb/Layout/Box.h

@@ -41,7 +41,7 @@ class Box : public NodeWithStyleAndBoxModelMetrics {
 
     virtual void did_set_rect() { }
 
-    virtual OwnPtr<Painting::Paintable> create_paintable() const override;
+    virtual RefPtr<Painting::Paintable> create_paintable() const override;
 
 protected:
     Box(DOM::Document&, DOM::Node*, NonnullRefPtr<CSS::StyleProperties>);

Userland/Libraries/LibWeb/Layout/ButtonBox.cpp

@@ -106,7 +106,7 @@ void ButtonBox::handle_associated_label_mousemove(Badge<Label>, bool is_inside_n
     set_needs_display();
 }
 
-OwnPtr<Painting::Paintable> ButtonBox::create_paintable() const
+RefPtr<Painting::Paintable> ButtonBox::create_paintable() const
 {
     return Painting::ButtonPaintable::create(*this);
 }

Userland/Libraries/LibWeb/Layout/ButtonBox.h

@@ -23,7 +23,7 @@ class ButtonBox : public LabelableNode {
 
     bool being_pressed() const { return m_being_pressed; }
 
-    virtual OwnPtr<Painting::Paintable> create_paintable() const override;
+    virtual RefPtr<Painting::Paintable> create_paintable() const override;
 
 private:
     virtual bool wants_mouse_events() const override { return true; }

Userland/Libraries/LibWeb/Layout/CanvasBox.cpp

@@ -25,7 +25,7 @@ void CanvasBox::prepare_for_replaced_layout()
     set_intrinsic_height(dom_node().height());
 }
 
-OwnPtr<Painting::Paintable> CanvasBox::create_paintable() const
+RefPtr<Painting::Paintable> CanvasBox::create_paintable() const
 {
     return Painting::CanvasPaintable::create(*this);
 }

Userland/Libraries/LibWeb/Layout/CanvasBox.h

@@ -20,7 +20,7 @@ class CanvasBox : public ReplacedBox {
 
     const HTML::HTMLCanvasElement& dom_node() const { return static_cast<const HTML::HTMLCanvasElement&>(ReplacedBox::dom_node()); }
 
-    virtual OwnPtr<Painting::Paintable> create_paintable() const override;
+    virtual RefPtr<Painting::Paintable> create_paintable() const override;
 };
 
 }

Userland/Libraries/LibWeb/Layout/CheckBox.cpp

@@ -108,7 +108,7 @@ void CheckBox::handle_associated_label_mousemove(Badge<Label>, bool is_inside_no
     set_needs_display();
 }
 
-OwnPtr<Painting::Paintable> CheckBox::create_paintable() const
+RefPtr<Painting::Paintable> CheckBox::create_paintable() const
 {
     return Painting::CheckBoxPaintable::create(*this);
 }

Userland/Libraries/LibWeb/Layout/CheckBox.h

@@ -21,7 +21,7 @@ class CheckBox : public LabelableNode {
 
     bool being_pressed() const { return m_being_pressed; }
 
-    virtual OwnPtr<Painting::Paintable> create_paintable() const override;
+    virtual RefPtr<Painting::Paintable> create_paintable() const override;
 
 private:
     virtual bool wants_mouse_events() const override { return true; }