Fix AXLayoutObject not to add duplicated children for CSS first-letter

third_party/WebKit/Source/modules/accessibility/AXLayoutObject.cpp

Index: third_party/WebKit/Source/modules/accessibility/AXLayoutObject.cpp
diff --git a/third_party/WebKit/Source/modules/accessibility/AXLayoutObject.cpp b/third_party/WebKit/Source/modules/accessibility/AXLayoutObject.cpp
index ce8c01c0b09f55b241e3c7c67d3c3a8126570dfe..988a5422ddd582024a124496b253e858650d39bf 100644
--- a/third_party/WebKit/Source/modules/accessibility/AXLayoutObject.cpp
+++ b/third_party/WebKit/Source/modules/accessibility/AXLayoutObject.cpp
@@ -62,6 +62,7 @@
 #include "core/layout/LayoutListMarker.h"
 #include "core/layout/LayoutMenuList.h"
 #include "core/layout/LayoutTextControl.h"
+#include "core/layout/LayoutTextFragment.h"
 #include "core/layout/LayoutView.h"
 #include "core/layout/api/LayoutAPIShim.h"
 #include "core/layout/api/LayoutViewItem.h"
@@ -121,6 +122,11 @@ static inline LayoutObject* firstChildConsideringContinuation(
     LayoutObject* layoutObject) {
   LayoutObject* firstChild = layoutObject->slowFirstChild();
 
+  if (firstChild && firstChild->isText() &&
+      toLayoutText(firstChild)->isTextFragment() &&
+      toLayoutTextFragment(firstChild)->firstLetterPseudoElement())
+    firstChild = nullptr;

[eae, 2016/11/24 21:50:13]

Shouldn't this be return nullptr to ensure that it's not overriden by the next
statement?

[kojii, 2016/11/25 06:38:35]

Done, thank you.

+
   if (!firstChild && isInlineWithContinuation(layoutObject))
     firstChild = firstChildInContinuation(toLayoutInline(*layoutObject));