third_party/WebKit/Source/core/layout/LayoutBoxModelObject.cpp - Issue 2414683002: Make offsetTop/Left handle a relative positioned inline offsetParent correctly.

Unified Diff: third_party/WebKit/Source/core/layout/LayoutBoxModelObject.cpp

Issue 2414683002: Make offsetTop/Left handle a relative positioned inline offsetParent correctly. (Closed)

Patch Set: Fixed broken rebase. Created 4 years, 1 month ago

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Jump to:

View side-by-side diff with in-line comments

« no previous file with comments | « third_party/WebKit/LayoutTests/fast/inline/inline-offsetLeft-relpos-expected.txt ('k') | third_party/WebKit/Source/core/layout/LayoutInline.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

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 4f9e99ebc6385f9f6ad27c6572ccc48714c81f17..9c12977b8ce22b218518f4af9b8857e9a743db76 100644

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

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

@@ -979,7 +979,7 @@ LayoutSize LayoutBoxModelObject::stickyPositionOffset() const {

LayoutPoint LayoutBoxModelObject::adjustedPositionRelativeTo(

const LayoutPoint& startPoint,

- const Element* element) const {

+ const Element* offsetParent) const {

// If the element is the HTML body element or doesn't have a parent

// return 0 and stop this algorithm.

if (isBody() || !parent())

@@ -987,13 +987,13 @@ LayoutPoint LayoutBoxModelObject::adjustedPositionRelativeTo(

LayoutPoint referencePoint = startPoint;

- // If the base element is null, return the distance between the canvas origin

- // and the left border edge of the element and stop this algorithm.

- if (!element)

+ // If the offsetParent is null, return the distance between the canvas origin

+ // and the left/top border edge of the element and stop this algorithm.

+ if (!offsetParent)

return referencePoint;

- if (const LayoutBoxModelObject* offsetParent =

- element->layoutBoxModelObject()) {

+ if (const LayoutBoxModelObject* offsetParentObject =

+ offsetParent->layoutBoxModelObject()) {

if (!isOutOfFlowPositioned()) {

if (isInFlowPositioned())

referencePoint.move(offsetForInFlowPosition());

@@ -1003,9 +1003,8 @@ LayoutPoint LayoutBoxModelObject::adjustedPositionRelativeTo(

// continuations: <body style="display:inline;" id="offsetParent">

// <div id="this">

// 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->node() != element;

+ current && current->node() != offsetParent;

current = current->container()) {

// FIXME: What are we supposed to do inside SVG content?

referencePoint.move(current->columnOffset(referencePoint));

@@ -1013,13 +1012,31 @@ LayoutPoint LayoutBoxModelObject::adjustedPositionRelativeTo(

referencePoint.moveBy(toLayoutBox(current)->topLeftLocation());

}

- if (offsetParent->isBox() && offsetParent->isBody() &&

- !offsetParent->isPositioned())

- referencePoint.moveBy(toLayoutBox(offsetParent)->topLeftLocation());

+ if (offsetParentObject->isBox() && offsetParentObject->isBody() &&

+ !offsetParentObject->isPositioned()) {

+ referencePoint.moveBy(

+ toLayoutBox(offsetParentObject)->topLeftLocation());

+ }

+ if (offsetParentObject->isLayoutInline()) {

+ const LayoutInline* inlineParent = toLayoutInline(offsetParentObject);

+ if (isBox() && style()->position() == AbsolutePosition &&

+ inlineParent->isInFlowPositioned()) {

+ // Offset for absolute elements with inline parent is a special

+ // case in the CSS spec

+ referencePoint +=

+ inlineParent->offsetForInFlowPositionedInline(*toLayoutBox(this));

+ }

+ referencePoint -= inlineParent->firstLineBoxTopLeft();

+ }

+ if (offsetParentObject->isBox() && !offsetParentObject->isBody()) {

+ referencePoint.move(-toLayoutBox(offsetParentObject)->borderLeft(),

+ -toLayoutBox(offsetParentObject)->borderTop());

}

- if (offsetParent->isBox() && !offsetParent->isBody())

- referencePoint.move(-toLayoutBox(offsetParent)->borderLeft(),

- -toLayoutBox(offsetParent)->borderTop());

}

return referencePoint;