Commit a53f7dc4 authored by Fredrik Söderquist's avatar Fredrik Söderquist Committed by Commit Bot

Remove unused LayoutObject::AbsoluteElementBoundingBoxRect

Looks like the last caller was removed in
https://codereview.chromium.org/2287433003 .

Change-Id: I2664063bab31fe4983be65cb5c21039a90b73920
Reviewed-on: https://chromium-review.googlesource.com/1104342Reviewed-by: default avatarXianzhu Wang <wangxianzhu@chromium.org>
Commit-Queue: Xianzhu Wang <wangxianzhu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#568019}
parent 1f418de3
...@@ -1298,15 +1298,6 @@ bool LayoutObject::GetLowerRightCorner(ExpandScrollMargin expand, ...@@ -1298,15 +1298,6 @@ bool LayoutObject::GetLowerRightCorner(ExpandScrollMargin expand,
return true; return true;
} }
IntRect LayoutObject::AbsoluteElementBoundingBoxRect() const {
Vector<LayoutRect> rects;
const LayoutBoxModelObject& container = EnclosingLayer()->GetLayoutObject();
AddElementVisualOverflowRects(
rects, LayoutPoint(LocalToAncestorPoint(FloatPoint(), &container)));
return container.LocalToAbsoluteQuad(FloatQuad(FloatRect(UnionRect(rects))))
.EnclosingBoundingBox();
}
FloatRect LayoutObject::AbsoluteBoundingBoxRectForRange( FloatRect LayoutObject::AbsoluteBoundingBoxRectForRange(
const EphemeralRange& range) { const EphemeralRange& range) {
if (range.IsNull() || !range.StartPosition().ComputeContainerNode()) if (range.IsNull() || !range.StartPosition().ComputeContainerNode())
......
...@@ -1707,11 +1707,6 @@ class CORE_EXPORT LayoutObject : public ImageResourceObserver, ...@@ -1707,11 +1707,6 @@ class CORE_EXPORT LayoutObject : public ImageResourceObserver,
AddOutlineRects(rects, additional_offset, kIncludeBlockVisualOverflow); AddOutlineRects(rects, additional_offset, kIncludeBlockVisualOverflow);
} }
// Returns the rect enclosing united visual overflow of the DOM subtree under
// this object. It includes continuations which may be not in the layout
// subtree of this object.
virtual IntRect AbsoluteElementBoundingBoxRect() const;
// Compute a list of hit-test rectangles per layer rooted at this // Compute a list of hit-test rectangles per layer rooted at this
// layoutObject with at most the given touch action. // layoutObject with at most the given touch action.
virtual void ComputeLayerHitTestRects(LayerHitTestRects&, TouchAction) const; virtual void ComputeLayerHitTestRects(LayerHitTestRects&, TouchAction) const;
......
...@@ -148,12 +148,4 @@ bool LayoutSVGModelObject::NodeAtPoint(HitTestResult&, ...@@ -148,12 +148,4 @@ bool LayoutSVGModelObject::NodeAtPoint(HitTestResult&,
return false; return false;
} }
// The SVG addOutlineRects() method adds rects in local coordinates so the
// default absoluteElementBoundingBoxRect() returns incorrect values for SVG
// objects. Overriding this method provides access to the absolute bounds.
IntRect LayoutSVGModelObject::AbsoluteElementBoundingBoxRect() const {
return LocalToAbsoluteQuad(FloatQuad(VisualRectInLocalSVGCoordinates()))
.EnclosingBoundingBox();
}
} // namespace blink } // namespace blink
...@@ -100,7 +100,6 @@ class LayoutSVGModelObject : public LayoutObject { ...@@ -100,7 +100,6 @@ class LayoutSVGModelObject : public LayoutObject {
const HitTestLocation& location_in_container, const HitTestLocation& location_in_container,
const LayoutPoint& accumulated_offset, const LayoutPoint& accumulated_offset,
HitTestAction) final; HitTestAction) final;
IntRect AbsoluteElementBoundingBoxRect() const final;
protected: protected:
FloatRect local_visual_rect_; FloatRect local_visual_rect_;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment