adjustedPositionRelativeTo() couldn't find offsetParent.

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

Index: third_party/WebKit/Source/core/layout/LayoutBoxModelObject.cpp
diff --git a/third_party/WebKit/Source/core/layout/LayoutBoxModelObject.cpp b/third_party/WebKit/Source/core/layout/LayoutBoxModelObject.cpp
index 653dffb1188e470df8c63354f5ee628f2b27cf94..28edd658b75c676e4818376b61b72510339785c5 100644
--- a/third_party/WebKit/Source/core/layout/LayoutBoxModelObject.cpp
+++ b/third_party/WebKit/Source/core/layout/LayoutBoxModelObject.cpp
@@ -1016,7 +1016,8 @@ LayoutPoint LayoutBoxModelObject::adjustedPositionRelativeTo(
       // This is why we have to do a nullptr check here.
       // offset(Left|Top) is generally broken when offsetParent is inline.
       for (const LayoutObject* current = container();
-           current && current != offsetParent; current = current->container()) {
+           current && current->node() != element;

[Gleb Lanbin, 2016/10/31 22:29:53]

I'm not very familiar with this code. I would suggest mstensho@ as a reviewer.
According to git blame he authored this code.

From my point of view I have 2 comments:

1) should we change the comment at line 1012 as well?
2) if we don't depend on offsetParent anymore then perhaps we can move this part
of code out of the "offsetParent = element->layoutBoxModelObject()" IF
statement?

[Karl Øygard, 2016/11/02 12:48:01]

Agreed, I've updated the comment with another problem that I know of that
isn't addressed by this fix.

I can't see an easy way of moving the code out without some code
duplication, so I've let it remain as is.

+           current = current->container()) {
         // FIXME: What are we supposed to do inside SVG content?
         referencePoint.move(current->columnOffset(referencePoint));
         if (current->isBox() && !current->isTableRow())