Convert first letter into a pseudo element.

Source/core/editing/TextIterator.cpp

 /*
  * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2012 Apple Inc. All rights reserved.
  * Copyright (C) 2005 Alexey Proskuryakov.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
@@ skipping 12 matching lines @@
  * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
 #include "config.h"
 #include "core/editing/TextIterator.h"
 
 #include "bindings/core/v8/ExceptionStatePlaceholder.h"
 #include "core/HTMLNames.h"
 #include "core/dom/Document.h"
+#include "core/dom/FirstLetterPseudoElement.h"
 #include "core/dom/NodeTraversal.h"
 #include "core/dom/shadow/ShadowRoot.h"
 #include "core/editing/VisiblePosition.h"
 #include "core/editing/VisibleUnits.h"
 #include "core/editing/htmlediting.h"
 #include "core/frame/FrameView.h"
 #include "core/html/HTMLElement.h"
 #include "core/html/HTMLTextFormControlElement.h"
 #include "core/rendering/InlineTextBox.h"
 #include "core/rendering/RenderImage.h"
@@ skipping 628 matching lines @@
         m_textBox = m_sortedTextBoxes.isEmpty() ? 0 : m_sortedTextBoxes[0];
     }
 
     handleTextBox();
     return true;
 }
 
 void TextIterator::handleTextBox()
 {
     RenderText* renderer = m_firstLetterText ? m_firstLetterText.get() : toRenderText(m_node->renderer());
+
     if (renderer->style()->visibility() != VISIBLE && !m_ignoresStyleVisibility) {
         m_textBox = 0;
-        return;
-    }
-    String str = renderer->text();
-    unsigned start = m_offset;
-    unsigned end = (m_node == m_endContainer) ? static_cast<unsigned>(m_endOffset) : INT_MAX;
-    while (m_textBox) {
-        unsigned textBoxStart = m_textBox->start();
-        unsigned runStart = std::max(textBoxStart, start);
+    } else {
+        String str = renderer->text();
+        unsigned start = m_offset;
+        unsigned end = (m_node == m_endContainer) ? static_cast<unsigned>(m_endOffset) : INT_MAX;
+        while (m_textBox) {
+            unsigned textBoxStart = m_textBox->start();
+            unsigned runStart = std::max(textBoxStart, start);
 
-        // Check for collapsed space at the start of this run.
-        InlineTextBox* firstTextBox = renderer->containsReversedText() ? (m_sortedTextBoxes.isEmpty() ? 0 : m_sortedTextBoxes[0]) : renderer->firstTextBox();
-        bool needSpace = m_lastTextNodeEndedWithCollapsedSpace
-            || (m_textBox == firstTextBox && textBoxStart == runStart && runStart > 0);
-        if (needSpace && !renderer->style()->isCollapsibleWhiteSpace(m_lastCharacter) && m_lastCharacter) {
-            if (m_lastTextNode == m_node && runStart > 0 && str[runStart - 1] == ' ') {
-                unsigned spaceRunStart = runStart - 1;
-                while (spaceRunStart > 0 && str[spaceRunStart - 1] == ' ')
-                    --spaceRunStart;
-                emitText(m_node, renderer, spaceRunStart, spaceRunStart + 1);
+            // Check for collapsed space at the start of this run.
+            InlineTextBox* firstTextBox = renderer->containsReversedText() ? (m_sortedTextBoxes.isEmpty() ? 0 : m_sortedTextBoxes[0]) : renderer->firstTextBox();
+            bool needSpace = m_lastTextNodeEndedWithCollapsedSpace
+                || (m_textBox == firstTextBox && textBoxStart == runStart && runStart > 0);
+            if (needSpace && !renderer->style()->isCollapsibleWhiteSpace(m_lastCharacter) && m_lastCharacter) {
+                if (m_lastTextNode == m_node && runStart > 0 && str[runStart - 1] == ' ') {
+                    unsigned spaceRunStart = runStart - 1;
+                    while (spaceRunStart > 0 && str[spaceRunStart - 1] == ' ')
+                        --spaceRunStart;
+                    emitText(m_node, renderer, spaceRunStart, spaceRunStart + 1);
+                } else {
+                    emitCharacter(space, m_node, 0, runStart, runStart);
+                }
+                return;
+            }
+            unsigned textBoxEnd = textBoxStart + m_textBox->len();
+            unsigned runEnd = std::min(textBoxEnd, end);
+
+            // Determine what the next text box will be, but don't advance yet
+            InlineTextBox* nextTextBox = nullptr;
+            if (renderer->containsReversedText()) {
+                if (m_sortedTextBoxesPosition + 1 < m_sortedTextBoxes.size())
+                    nextTextBox = m_sortedTextBoxes[m_sortedTextBoxesPosition + 1];
             } else {
-                emitCharacter(space, m_node, 0, runStart, runStart);
+                nextTextBox = m_textBox->nextTextBox();
             }
-            return;
-        }
-        unsigned textBoxEnd = textBoxStart + m_textBox->len();
-        unsigned runEnd = std::min(textBoxEnd, end);
+            ASSERT(!nextTextBox || nextTextBox->renderer() == renderer);
 
-        // Determine what the next text box will be, but don't advance yet
-        InlineTextBox* nextTextBox = nullptr;
-        if (renderer->containsReversedText()) {
-            if (m_sortedTextBoxesPosition + 1 < m_sortedTextBoxes.size())
-                nextTextBox = m_sortedTextBoxes[m_sortedTextBoxesPosition + 1];
-        } else {
-            nextTextBox = m_textBox->nextTextBox();
-        }
-        ASSERT(!nextTextBox || nextTextBox->renderer() == renderer);
+            if (runStart < runEnd) {
+                // Handle either a single newline character (which becomes a space),
+                // or a run of characters that does not include a newline.
+                // This effectively translates newlines to spaces without copying the text.
+                if (str[runStart] == '\n') {
+                    emitCharacter(space, m_node, 0, runStart, runStart + 1);
+                    m_offset = runStart + 1;
+                } else {
+                    size_t subrunEnd = str.find('\n', runStart);
+                    if (subrunEnd == kNotFound || subrunEnd > runEnd)
+                        subrunEnd = runEnd;
 
-        if (runStart < runEnd) {
-            // Handle either a single newline character (which becomes a space),
-            // or a run of characters that does not include a newline.
-            // This effectively translates newlines to spaces without copying the text.
-            if (str[runStart] == '\n') {
-                emitCharacter(space, m_node, 0, runStart, runStart + 1);
-                m_offset = runStart + 1;
-            } else {
-                size_t subrunEnd = str.find('\n', runStart);
-                if (subrunEnd == kNotFound || subrunEnd > runEnd)
-                    subrunEnd = runEnd;
+                    m_offset = subrunEnd;
+                    emitText(m_node, renderer, runStart, subrunEnd);
+                }
 
-                m_offset = subrunEnd;
-                emitText(m_node, renderer, runStart, subrunEnd);
+                // If we are doing a subrun that doesn't go to the end of the text box,
+                // come back again to finish handling this text box; don't advance to the next one.
+                if (static_cast<unsigned>(m_positionEndOffset) < textBoxEnd)
+                    return;
+
+                // Advance and return
+                unsigned nextRunStart = nextTextBox ? nextTextBox->start() : str.length();
+                if (nextRunStart > runEnd)
+                    m_lastTextNodeEndedWithCollapsedSpace = true; // collapsed space between runs or at the end
+
+                m_textBox = nextTextBox;
+                if (renderer->containsReversedText())
+                    ++m_sortedTextBoxesPosition;
+                return;
             }
-
-            // If we are doing a subrun that doesn't go to the end of the text box,
-            // come back again to finish handling this text box; don't advance to the next one.
-            if (static_cast<unsigned>(m_positionEndOffset) < textBoxEnd)
-                return;
-
-            // Advance and return
-            unsigned nextRunStart = nextTextBox ? nextTextBox->start() : str.length();
-            if (nextRunStart > runEnd)
-                m_lastTextNodeEndedWithCollapsedSpace = true; // collapsed space between runs or at the end
+            // Advance and continue
             m_textBox = nextTextBox;
             if (renderer->containsReversedText())
                 ++m_sortedTextBoxesPosition;
-            return;
         }
-        // Advance and continue
-        m_textBox = nextTextBox;
-        if (renderer->containsReversedText())
-            ++m_sortedTextBoxesPosition;
     }
+
     if (!m_textBox && m_remainingTextBox) {
         m_textBox = m_remainingTextBox;
         m_remainingTextBox = 0;
         m_firstLetterText = nullptr;
         m_offset = 0;
         handleTextBox();
     }
 }
 
-static inline RenderText* firstRenderTextInFirstLetter(RenderBoxModelObject* firstLetter)
-{
-    if (!firstLetter)
-        return 0;
-
-    // FIXME: Should this check descendent objects?
-    for (RenderObject* current = firstLetter->slowFirstChild(); current; current = current->nextSibling()) {
-        if (current->isText())
-            return toRenderText(current);
-    }
-    return 0;
-}
-
 void TextIterator::handleTextNodeFirstLetter(RenderTextFragment* renderer)
 {
-    if (renderer->firstLetter()) {
-        RenderBoxModelObject* r = renderer->firstLetter();
-        if (r->style()->visibility() != VISIBLE && !m_ignoresStyleVisibility)
-            return;
-        if (RenderText* firstLetter = firstRenderTextInFirstLetter(r)) {
-            m_handledFirstLetter = true;
-            m_remainingTextBox = m_textBox;
-            m_textBox = firstLetter->firstTextBox();
-            m_sortedTextBoxes.clear();
-            m_firstLetterText = firstLetter;
-        }
-    }
     m_handledFirstLetter = true;
+
+    if (!renderer->isRemainingTextRenderer())
+        return;
+
+    FirstLetterPseudoElement* firstLetterElement = renderer->firstLetterPseudoElement();
+    if (!firstLetterElement)
+        return;
+
+    RenderObject* pseudoRenderer = firstLetterElement->renderer();
+    if (pseudoRenderer->style()->visibility() != VISIBLE && !m_ignoresStyleVisibility)
+        return;
+
+    RenderObject* firstLetter = pseudoRenderer->slowFirstChild();
+    ASSERT(firstLetter);
+
+    m_remainingTextBox = m_textBox;
+    m_textBox = toRenderText(firstLetter)->firstTextBox();
+    m_sortedTextBoxes.clear();
+    m_firstLetterText = toRenderText(firstLetter);
 }
 
 bool TextIterator::handleReplacedElement()
 {
     if (m_fullyClippedStack.top())
         return false;
 
     RenderObject* renderer = m_node->renderer();
     if (renderer->style()->visibility() != VISIBLE && !m_ignoresStyleVisibility)
         return false;
@@ skipping 41 matching lines @@
     m_textLength = 0;
     m_lastCharacter = 0;
 
     return true;
 }
 
 bool TextIterator::hasVisibleTextNode(RenderText* renderer)
 {
     if (renderer->style()->visibility() == VISIBLE)
         return true;
-    if (renderer->isTextFragment()) {
-        RenderTextFragment* fragment = toRenderTextFragment(renderer);
-        if (fragment->firstLetter() && fragment->firstLetter()->style()->visibility() == VISIBLE)
-            return true;
-    }
+
+    if (!renderer->isTextFragment())
+        return false;
+
+    RenderTextFragment* fragment = toRenderTextFragment(renderer);
+    if (!fragment->isRemainingTextRenderer())
+        return false;
+
+    RenderObject* pseudoElementRenderer = fragment->firstLetterPseudoElement()->renderer();
+    if (pseudoElementRenderer && pseudoElementRenderer->style()->visibility() == VISIBLE)

[Julien - ping for review, 2014/11/12 21:58:51]

Way, way better written:

return pseudoElementRenderer && pseudoElementRenderer->style()->visibility() ==
VISIBLE;

[dsinclair, 2014/11/13 17:55:36]

Good catch, thanks.

+        return true;
+
     return false;
 }
 
 static bool shouldEmitTabBeforeNode(Node* node)
 {
     RenderObject* r = node->renderer();
 
     // Table cells are delimited by tabs.
     if (!r || !isTableCell(node))
         return false;
@@ skipping 653 matching lines @@
     }
 
     if (!m_shouldHandleFirstLetter && offsetAfterFirstLetter < m_offset) {
         m_shouldHandleFirstLetter = true;
         offsetInNode = offsetAfterFirstLetter;
         return renderer;
     }
 
     m_shouldHandleFirstLetter = false;
     offsetInNode = 0;
-    RenderText* firstLetterRenderer = firstRenderTextInFirstLetter(fragment->firstLetter());
+
+    ASSERT(fragment->isRemainingTextRenderer());
+    ASSERT(fragment->firstLetterPseudoElement());
+
+    RenderObject* pseudoElementRenderer = fragment->firstLetterPseudoElement()->renderer();
+    ASSERT(pseudoElementRenderer);
+    ASSERT(pseudoElementRenderer->slowFirstChild());
+    RenderText* firstLetterRenderer = toRenderText(pseudoElementRenderer->slowFirstChild());
 
     m_offset = firstLetterRenderer->caretMaxOffset();
     m_offset += collapsedSpaceLength(firstLetterRenderer, m_offset);
 
     return firstLetterRenderer;
 }
 
 bool SimplifiedBackwardsTextIterator::handleReplacedElement()
 {
     unsigned index = m_node->nodeIndex();
@@ skipping 879 matching lines @@
             resultEnd = collapseTo;
             return;
         }
     }
 
     CharacterIterator computeRangeIterator(inputStart, inputEnd, iteratorFlagsForFindPlainText);
     calculateCharacterSubrange(computeRangeIterator, matchStart, matchLength, resultStart, resultEnd);
 }
 
 }