Make TextIterator not stop at IMG element for finding text separating inline image

Source/core/editing/TextIterator.h

Index: Source/core/editing/TextIterator.h
diff --git a/Source/core/editing/TextIterator.h b/Source/core/editing/TextIterator.h
index cbe398e7ffbd571bdb8c98b7a93d73fa447ff74a..ee525ebd6acbe9eda2fa9b1735f757039e766a2f 100644
--- a/Source/core/editing/TextIterator.h
+++ b/Source/core/editing/TextIterator.h
@@ -46,7 +46,8 @@ enum TextIteratorBehavior {
     TextIteratorStopsOnFormControls = 1 << 4,
     TextIteratorEmitsImageAltText = 1 << 5,
     TextIteratorEntersAuthorShadowRoots = 1 << 6,
-    TextIteratorEmitsObjectReplacementCharacter = 1 << 7
+    TextIteratorEmitsObjectReplacementCharacter = 1 << 7,
+    TextIteratorNotBreakAtZeroLengthBoundary = 1 << 8

[Yuta Kitamura, 2014/10/07 09:51:30]

The name "NotBreakAtZeroLengthBoundary" does not make sense at all
if one hasn't read the implementation.

The flag should be named after something based on the behavior,
not the implementation. Should be something like
"DoesNotBreakAtReplacedElement".

As a more minor nitpick, you should use the third-person verb form
("breaks" instead of "break", "does not break" instead of "not break"
or "do not break") to align with other flags.

[c.shu, 2014/10/07 14:30:26]

Done.

 };
 typedef unsigned TextIteratorBehaviorFlags;
 
@@ -119,6 +120,8 @@ public:
     Position startPosition() const;
     Position endPosition() const;
 
+    bool breakAtZeroLengthBoundary() { return m_breakAtZeroLengthBoundary; }
+
     // Computes the length of the given range using a text iterator. The default
     // iteration behavior is to always emit object replacement characters for
     // replaced elements. When |forSelectionPreservation| is set to true, it
@@ -228,6 +231,8 @@ private:
     bool m_entersAuthorShadowRoots;
 
     bool m_emitsObjectReplacementCharacter;
+
+    bool m_breakAtZeroLengthBoundary;
 };
 
 // Iterates through the DOM range, returning all the text, and 0-length boundaries