Rename InlineBox::remove() to removeFromParent
authorantti@apple.com <antti@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 13 Oct 2013 11:39:33 +0000 (11:39 +0000)
committerantti@apple.com <antti@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 13 Oct 2013 11:39:33 +0000 (11:39 +0000)
https://bugs.webkit.org/show_bug.cgi?id=122725

Reviewed by Andreas Kling.

Also make it clear from the code that all InlineTextBoxes have behavesLikeText set.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@157367 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebCore/ChangeLog
Source/WebCore/rendering/InlineBox.cpp
Source/WebCore/rendering/InlineBox.h
Source/WebCore/rendering/InlineTextBox.cpp
Source/WebCore/rendering/InlineTextBox.h
Source/WebCore/rendering/RenderBlock.cpp
Source/WebCore/rendering/RenderBox.cpp
Source/WebCore/rendering/RenderInline.cpp
Source/WebCore/rendering/RenderLineBreak.cpp
Source/WebCore/rendering/RenderText.cpp
Source/WebCore/rendering/RenderTextLineBoxes.cpp

index a63811f..b5f1fa3 100644 (file)
@@ -1,5 +1,14 @@
 2013-10-13  Antti Koivisto  <antti@apple.com>
 
 2013-10-13  Antti Koivisto  <antti@apple.com>
 
+        Rename InlineBox::remove() to removeFromParent
+        https://bugs.webkit.org/show_bug.cgi?id=122725
+
+        Reviewed by Andreas Kling.
+
+        Also make it clear from the code that all InlineTextBoxes have behavesLikeText set.
+
+2013-10-13  Antti Koivisto  <antti@apple.com>
+
         Make absoluteQuads/Rects functions return Vectors
         https://bugs.webkit.org/show_bug.cgi?id=122722
 
         Make absoluteQuads/Rects functions return Vectors
         https://bugs.webkit.org/show_bug.cgi?id=122722
 
index f1d0409..1a78c41 100644 (file)
@@ -64,7 +64,7 @@ InlineBox::~InlineBox()
 }
 #endif
 
 }
 #endif
 
-void InlineBox::remove()
+void InlineBox::removeFromParent()
 { 
     if (parent())
         parent()->removeChild(this);
 { 
     if (parent())
         parent()->removeChild(this);
index f9d36b1..ae7037a 100644 (file)
@@ -154,7 +154,7 @@ public:
     void setIsFirstLine(bool firstLine) { m_bitfields.setFirstLine(firstLine); }
     bool isFirstLine() const { return m_bitfields.firstLine(); }
 
     void setIsFirstLine(bool firstLine) { m_bitfields.setFirstLine(firstLine); }
     bool isFirstLine() const { return m_bitfields.firstLine(); }
 
-    void remove();
+    void removeFromParent();
 
     InlineBox* nextOnLine() const { return m_next; }
     InlineBox* prevOnLine() const { return m_prev; }
 
     InlineBox* nextOnLine() const { return m_next; }
     InlineBox* prevOnLine() const { return m_prev; }
index 18e96e8..18c7846 100644 (file)
@@ -99,7 +99,7 @@ void InlineTextBox::setLogicalOverflowRect(const LayoutRect& rect)
 
 int InlineTextBox::baselinePosition(FontBaseline baselineType) const
 {
 
 int InlineTextBox::baselinePosition(FontBaseline baselineType) const
 {
-    if (!behavesLikeText() || !parent())
+    if (!parent())
         return 0;
     if (&parent()->renderer() == renderer().parent())
         return parent()->baselinePosition(baselineType);
         return 0;
     if (&parent()->renderer() == renderer().parent())
         return parent()->baselinePosition(baselineType);
@@ -108,7 +108,7 @@ int InlineTextBox::baselinePosition(FontBaseline baselineType) const
 
 LayoutUnit InlineTextBox::lineHeight() const
 {
 
 LayoutUnit InlineTextBox::lineHeight() const
 {
-    if (!behavesLikeText() || !renderer().parent())
+    if (!renderer().parent())
         return 0;
     if (&parent()->renderer() == renderer().parent())
         return parent()->lineHeight();
         return 0;
     if (&parent()->renderer() == renderer().parent())
         return parent()->lineHeight();
index 3709db7..cac4814 100644 (file)
@@ -51,6 +51,7 @@ public:
         , m_len(0)
         , m_truncation(cNoTruncation)
     {
         , m_len(0)
         , m_truncation(cNoTruncation)
     {
+        setBehavesLikeText(true);
     }
 
     RenderText& renderer() const { return toRenderText(InlineBox::renderer()); }
     }
 
     RenderText& renderer() const { return toRenderText(InlineBox::renderer()); }
@@ -161,16 +162,6 @@ public:
     // Needs to be public, so the static paintTextWithShadows() function can use it.
     static FloatSize applyShadowToGraphicsContext(GraphicsContext*, const ShadowData*, const FloatRect& textRect, bool stroked, bool opaque, bool horizontal);
 
     // Needs to be public, so the static paintTextWithShadows() function can use it.
     static FloatSize applyShadowToGraphicsContext(GraphicsContext*, const ShadowData*, const FloatRect& textRect, bool stroked, bool opaque, bool horizontal);
 
-private:
-    InlineTextBox* m_prevTextBox; // The previous box that also uses our RenderObject
-    InlineTextBox* m_nextTextBox; // The next box that also uses our RenderObject
-
-    int m_start;
-    unsigned short m_len;
-
-    unsigned short m_truncation; // Where to truncate when text overflow is applied.  We use special constants to
-                      // denote no truncation (the whole run paints) and full truncation (nothing paints at all).
-
 protected:
     void paintCompositionBackground(GraphicsContext*, const FloatPoint& boxOrigin, const RenderStyle&, const Font&, int startPos, int endPos);
     void paintDocumentMarkers(GraphicsContext*, const FloatPoint& boxOrigin, const RenderStyle&, const Font&, bool background);
 protected:
     void paintCompositionBackground(GraphicsContext*, const FloatPoint& boxOrigin, const RenderStyle&, const Font&, int startPos, int endPos);
     void paintDocumentMarkers(GraphicsContext*, const FloatPoint& boxOrigin, const RenderStyle&, const Font&, bool background);
@@ -191,6 +182,18 @@ private:
         return (canHaveLeadingExpansion() ? TextRun::AllowLeadingExpansion : TextRun::ForbidLeadingExpansion)
             | (expansion() && nextLeafChild() ? TextRun::AllowTrailingExpansion : TextRun::ForbidTrailingExpansion);
     }
         return (canHaveLeadingExpansion() ? TextRun::AllowLeadingExpansion : TextRun::ForbidLeadingExpansion)
             | (expansion() && nextLeafChild() ? TextRun::AllowTrailingExpansion : TextRun::ForbidTrailingExpansion);
     }
+
+    void behavesLikeText() const WTF_DELETED_FUNCTION;
+
+    InlineTextBox* m_prevTextBox; // The previous box that also uses our RenderObject
+    InlineTextBox* m_nextTextBox; // The next box that also uses our RenderObject
+
+    int m_start;
+    unsigned short m_len;
+
+    // Where to truncate when text overflow is applied. We use special constants to
+    // denote no truncation (the whole run paints) and full truncation (nothing paints at all).
+    unsigned short m_truncation;
 };
 
 inline InlineTextBox* toInlineTextBox(InlineBox* inlineBox)
 };
 
 inline InlineTextBox* toInlineTextBox(InlineBox* inlineBox)
index 0ae75e5..428a9be 100644 (file)
@@ -234,9 +234,9 @@ void RenderBlock::willBeDestroyed()
             // that will outlast this block. In the non-anonymous block case those
             // children will be destroyed by the time we return from this function.
             if (isAnonymousBlock()) {
             // that will outlast this block. In the non-anonymous block case those
             // children will be destroyed by the time we return from this function.
             if (isAnonymousBlock()) {
-                for (InlineFlowBox* box = firstLineBox(); box; box = box->nextLineBox()) {
-                    while (InlineBox* childBox = box->firstChild())
-                        childBox->remove();
+                for (auto box = firstLineBox(); box; box = box->nextLineBox()) {
+                    while (auto childBox = box->firstChild())
+                        childBox->removeFromParent();
                 }
             }
         } else if (parent())
                 }
             }
         } else if (parent())
index d71aee0..99ce933 100644 (file)
@@ -1968,7 +1968,7 @@ void RenderBox::positionLineBox(InlineBox* box)
         }
 
         // Nuke the box.
         }
 
         // Nuke the box.
-        box->remove();
+        box->removeFromParent();
         box->destroy(renderArena());
     } else if (isReplaced()) {
         setLocation(roundedLayoutPoint(box->topLeft()));
         box->destroy(renderArena());
     } else if (isReplaced()) {
         setLocation(roundedLayoutPoint(box->topLeft()));
@@ -1984,7 +1984,7 @@ void RenderBox::deleteLineBoxWrapper()
 {
     if (m_inlineBoxWrapper) {
         if (!documentBeingDestroyed())
 {
     if (m_inlineBoxWrapper) {
         if (!documentBeingDestroyed())
-            m_inlineBoxWrapper->remove();
+            m_inlineBoxWrapper->removeFromParent();
         m_inlineBoxWrapper->destroy(renderArena());
         m_inlineBoxWrapper = 0;
     }
         m_inlineBoxWrapper->destroy(renderArena());
         m_inlineBoxWrapper = 0;
     }
index e4b6ad9..0fce14b 100644 (file)
@@ -107,8 +107,8 @@ void RenderInline::willBeDestroyed()
             // not have a parent that means they are either already disconnected or
             // root lines that can just be destroyed without disconnecting.
             if (firstLineBox()->parent()) {
             // not have a parent that means they are either already disconnected or
             // root lines that can just be destroyed without disconnecting.
             if (firstLineBox()->parent()) {
-                for (InlineFlowBox* box = firstLineBox(); box; box = box->nextLineBox())
-                    box->remove();
+                for (auto box = firstLineBox(); box; box = box->nextLineBox())
+                    box->removeFromParent();
             }
         } else if (parent()) 
             parent()->dirtyLinesFromChangedChild(this);
             }
         } else if (parent()) 
             parent()->dirtyLinesFromChangedChild(this);
index b4e8927..2a5594f 100644 (file)
@@ -90,7 +90,7 @@ void RenderLineBreak::deleteInlineBoxWrapper()
     if (!m_inlineBoxWrapper)
         return;
     if (!documentBeingDestroyed())
     if (!m_inlineBoxWrapper)
         return;
     if (!documentBeingDestroyed())
-        m_inlineBoxWrapper->remove();
+        m_inlineBoxWrapper->removeFromParent();
     m_inlineBoxWrapper->destroy(renderArena());
     m_inlineBoxWrapper = nullptr;
 }
     m_inlineBoxWrapper->destroy(renderArena());
     m_inlineBoxWrapper = nullptr;
 }
index ee7470e..7042f27 100644 (file)
@@ -1002,7 +1002,7 @@ void RenderText::positionLineBox(InlineBox* box)
     // FIXME: should not be needed!!!
     if (!textBox->len()) {
         // We want the box to be destroyed.
     // FIXME: should not be needed!!!
     if (!textBox->len()) {
         // We want the box to be destroyed.
-        textBox->remove();
+        textBox->removeFromParent();
         m_lineBoxes.remove(*textBox);
         textBox->destroy(renderArena());
         return;
         m_lineBoxes.remove(*textBox);
         textBox->destroy(renderArena());
         return;
index 3689ca5..eea8360 100644 (file)
@@ -51,7 +51,6 @@ InlineTextBox* RenderTextLineBoxes::createAndAppendLineBox(RenderText& renderTex
         textBox->setPreviousTextBox(m_last);
         m_last = textBox;
     }
         textBox->setPreviousTextBox(m_last);
         m_last = textBox;
     }
-    textBox->setBehavesLikeText(true);
     return textBox;
 }
 
     return textBox;
 }
 
@@ -114,7 +113,7 @@ void RenderTextLineBoxes::removeAllFromParent(RenderText& renderer)
         return;
     }
     for (auto box = m_first; box; box = box->nextTextBox())
         return;
     }
     for (auto box = m_first; box; box = box->nextTextBox())
-        box->remove();
+        box->removeFromParent();
 }
 
 void RenderTextLineBoxes::deleteAll(RenderText& renderer)
 }
 
 void RenderTextLineBoxes::deleteAll(RenderText& renderer)