Do not de-select text across continuations

third_party/WebKit/Source/core/layout/LayoutInline.cpp

Index: third_party/WebKit/Source/core/layout/LayoutInline.cpp
diff --git a/third_party/WebKit/Source/core/layout/LayoutInline.cpp b/third_party/WebKit/Source/core/layout/LayoutInline.cpp
index 1c173c48b6254174163a90d83fbe0907a8019dfe..1ecbc13ea12dfc19d87f3fc42675036abb540b35 100644
--- a/third_party/WebKit/Source/core/layout/LayoutInline.cpp
+++ b/third_party/WebKit/Source/core/layout/LayoutInline.cpp
@@ -936,23 +936,20 @@ bool LayoutInline::hitTestCulledInline(
 
 PositionWithAffinity LayoutInline::positionForPoint(const LayoutPoint& point) {
   // FIXME: Does not deal with relative positioned inlines (should it?)
-  LayoutBlock* cb = containingBlock();
+
+  // If there are continuations, test them first because our containing block
+  // will not check them.
+  LayoutBoxModelObject* continuation = this->continuation();
+  while (continuation) {
+    if (continuation->isInline() || continuation->slowFirstChild())
+      return continuation->positionForPoint(point);
+    continuation = toLayoutBlockFlow(continuation)->inlineElementContinuation();
+  }
+
   if (firstLineBoxIncludingCulling()) {
     // This inline actually has a line box.  We must have clicked in the border/padding of one of these boxes.  We
     // should try to find a result by asking our containing block.
-    return cb->positionForPoint(point);
-  }
-
-  // Translate the coords from the pre-anonymous block to the post-anonymous block.

[pdr., 2016/10/06 04:52:32]

This code was added in 2007 in
https://chromium.googlesource.com/chromium/src/+/e2f61b1719b9254e7e3b6b542d41...
without much information or tests. I've added tests to cover what this might
have been intending. I did not find any existing layout test covers this
codepath.

-  LayoutPoint parentBlockPoint = cb->location() + point;
-  LayoutBoxModelObject* c = continuation();
-  while (c) {
-    LayoutBox* contBlock =
-        c->isInline() ? c->containingBlock() : toLayoutBlock(c);
-    if (c->isInline() || c->slowFirstChild())
-      return c->positionForPoint(parentBlockPoint -
-                                 contBlock->locationOffset());
-    c = toLayoutBlockFlow(c)->inlineElementContinuation();
+    return containingBlock()->positionForPoint(point);
   }
 
   return LayoutBoxModelObject::positionForPoint(point);