RenderTreeBuilder => RenderTreeBuilderForElement and RenderTreeBuilderForText

Source/core/dom/RenderTreeBuilder.h

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2001 Dirk Mueller (mueller@kde.org)
  * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2011 Apple Inc. All rights reserved.
  * Copyright (C) 2008, 2009 Torch Mobile Inc. All rights reserved. (http://www.torchmobile.com/)
  * Copyright (C) 2011 Google Inc. All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
@@ skipping 11 matching lines @@
  * Boston, MA 02110-1301, USA.
  *
  */
 
 #ifndef RenderTreeBuilder_h
 #define RenderTreeBuilder_h
 
 #include "core/dom/Document.h"
 #include "core/dom/Node.h"
 #include "core/dom/NodeRenderingTraversal.h"
+#include "core/dom/Text.h"
 #include "wtf/RefPtr.h"
 
 namespace blink {
 
 class ContainerNode;
 class RenderObject;
 class RenderStyle;
 
 class RenderTreeBuilder {
     STACK_ALLOCATED();
 public:
-    RenderTreeBuilder(Node* node, RenderStyle* style)
+    RenderTreeBuilder(Node* node)
         : m_node(node)
-        , m_renderingParent(nullptr)
-        , m_style(style)
+        , m_renderingParent(NodeRenderingTraversal::parent(node))
     {
-        ASSERT(!node->renderer());

[esprehn, 2014/11/05 23:20:26]

This assert is still valid, we shouldn't create one of these for something that
already has a renderer. You'd also fail the following assert for needsAttach()
if you had one.

         ASSERT(node->needsAttach());
         ASSERT(node->document().inStyleRecalc());
 
         // FIXME: We should be able to ASSERT(node->inActiveDocument()) but childrenChanged is called
         // before ChildNodeInsertionNotifier in ContainerNode's methods and some implementations
         // will trigger a layout inside childrenChanged.
         // Mainly HTMLTextAreaElement::childrenChanged calls HTMLTextFormControlElement::setSelectionRange
         // which does an updateLayoutIgnorePendingStylesheets.
-
-        m_renderingParent = NodeRenderingTraversal::parent(node, &m_parentDetails);
     }
 
-    void createRendererForTextIfNeeded();
-    void createRendererForElementIfNeeded();
+    void createRendererIfNeeded();
+    virtual void createRenderer() = 0;
+    virtual bool rendererIsNeeded() const;

[esprehn, 2014/11/05 23:20:25]

Does the compiler devirtualize this?

 
-private:
-    RenderObject* parentRenderer() const;
+protected:
+    virtual RenderObject* parentRenderer() const;
     RenderObject* nextRenderer() const;
-    bool shouldCreateRenderer() const;
-    RenderStyle& style() const;
 
     RawPtrWillBeMember<Node> m_node;
     RawPtrWillBeMember<ContainerNode> m_renderingParent;
-    NodeRenderingTraversal::ParentDetails m_parentDetails;
-    mutable RefPtr<RenderStyle> m_style;
+};
+
+class RenderTreeBuilderForElement : public RenderTreeBuilder {
+public:
+    RenderTreeBuilderForElement(Element* element, PassRefPtr<RenderStyle> style)
+        : RenderTreeBuilder(element)
+        , m_style(style) { }
+
+    virtual void createRenderer() override final;
+    virtual bool rendererIsNeeded() const override final;
+
+private:
+    virtual RenderObject* parentRenderer() const override final;
+
+    RefPtr<RenderStyle> m_style;
+};
+
+class RenderTreeBuilderForText : public RenderTreeBuilder {
+public:
+    RenderTreeBuilderForText(Text* text)
+        : RenderTreeBuilder(text) { }
+
+    virtual void createRenderer() override final;
+    virtual bool rendererIsNeeded() const override final;
 };
 
 } // namespace blink
 
 #endif