Convert first letter into a pseudo element.

Source/core/inspector/InspectorCSSAgent.cpp

Index: Source/core/inspector/InspectorCSSAgent.cpp
diff --git a/Source/core/inspector/InspectorCSSAgent.cpp b/Source/core/inspector/InspectorCSSAgent.cpp
index 9db044599bc210e0499c2162f0df2120cccefeb6..05acff7c93ac035c3b47561f71869278135a744c 100644
--- a/Source/core/inspector/InspectorCSSAgent.cpp
+++ b/Source/core/inspector/InspectorCSSAgent.cpp
@@ -789,16 +789,24 @@ void InspectorCSSAgent::getPlatformFontsForNode(ErrorString* errorString, int no
     for (size_t i = 0; i < textNodes.size(); ++i) {
         RenderText* renderer = textNodes[i]->renderer();
         collectPlatformFontsForRenderer(renderer, &fontStats);
-        if (renderer->isTextFragment()) {
-            RenderTextFragment* textFragment = toRenderTextFragment(renderer);
-            if (textFragment->firstLetter()) {
-                RenderBoxModelObject* firstLetter = textFragment->firstLetter();
-                for (RenderObject* current = firstLetter->slowFirstChild(); current; current = current->nextSibling()) {
-                    if (current->isText())
-                        collectPlatformFontsForRenderer(toRenderText(current), &fontStats);
-                }
-            }
-        }
+
+        if (!renderer->isTextFragment())
+            continue;
+
+        // If we're the remaining text from a first-letter then our previous
+        // sibling has to be the first-letter renderer.
+        RenderObject* previous = renderer->previousSibling();
+        if (!previous)
+            continue;
+
+        if (!previous->isPseudoElement() || !previous->node()->isFirstLetterPseudoElement())
+            continue;
+
+        // The first-letter pseudoElement only has one child, which is the
+        // first-letter renderer.
+        RenderObject* firstLetterRenderer = previous->slowFirstChild();
+        ASSERT(firstLetterRenderer && firstLetterRenderer->isText());

[Julien - ping for review, 2014/10/09 18:24:48]

toRenderText will also check for isText but in release too so this checks seems
redundant.

Also when you have some logical ors in ASSERT, it's a sign that it should be
split up in several ASSERTs.

[dsinclair, 2014/10/09 21:14:22]

Done (removed the assert).

So, in general, is:

ASSERT(firstLetterRenderer)
ASSERT(firstLetterRenderer->isText())

preferable to the &&'d version?

[Julien - ping for review, 2014/10/10 14:47:07]

From my perspective yes because if ASSERT(firstLetterRenderer &&
firstLetterRenderer->isText()) fails you don't know which clause caused it and
have to run through a debugger to know.

[dsinclair, 2014/10/10 18:06:53]

Acknowledged.

+        collectPlatformFontsForRenderer(toRenderText(firstLetterRenderer), &fontStats);
     }
 
     platformFonts = TypeBuilder::Array<TypeBuilder::CSS::PlatformFontUsage>::create();