third_party/WebKit/Source/web/WebPluginContainerImpl.cpp - Issue 1563303005: Traverse into subdocuments when mapping plugin rects from absolute to local space.

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Unified Diff: third_party/WebKit/Source/web/WebPluginContainerImpl.cpp

Issue 1563303005: Traverse into subdocuments when mapping plugin rects from absolute to local space. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@2564

Patch Set: Created 4 years, 11 months 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

Download patch

« no previous file with comments | « third_party/WebKit/Source/web/WebPluginContainerImpl.h ('k') | third_party/WebKit/Source/web/tests/WebPluginContainerTest.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

Index: third_party/WebKit/Source/web/WebPluginContainerImpl.cpp

diff --git a/third_party/WebKit/Source/web/WebPluginContainerImpl.cpp b/third_party/WebKit/Source/web/WebPluginContainerImpl.cpp

index e538e06542a233670651655abc3d820f9481fd8b..6ab87d38dd6b85bb5562f8cbc8c30f254da37d83 100644

--- a/third_party/WebKit/Source/web/WebPluginContainerImpl.cpp

+++ b/third_party/WebKit/Source/web/WebPluginContainerImpl.cpp

@@ -968,7 +968,7 @@ void WebPluginContainerImpl::computeClipRectsForPlugin(

LayoutBox* box = toLayoutBox(ownerElement->layoutObject());

- // Plugin frameRects are in absolute screen space.

+ // Plugin frameRects are in absolute space within their frame.

IntRect frameRectInOwnerElementSpace = box->absoluteToLocalQuad(FloatRect(frameRect()), UseTransforms).enclosingBoundingBox();

LayoutRect unclippedAbsoluteRect(frameRectInOwnerElementSpace);

@@ -986,8 +986,8 @@ void WebPluginContainerImpl::computeClipRectsForPlugin(

clippedLocalRect.intersect(rootView->frameView()->visibleContentRect());

// TODO(chrishtr): intentionally ignore transform, because the positioning of frameRect() does also. This is probably wrong.

- unclippedIntLocalRect = box->absoluteToLocalQuad(FloatRect(unclippedIntLocalRect)).enclosingBoundingBox();

- clippedLocalRect = box->absoluteToLocalQuad(FloatRect(clippedLocalRect)).enclosingBoundingBox();

+ unclippedIntLocalRect = box->absoluteToLocalQuad(FloatRect(unclippedIntLocalRect), TraverseDocumentBoundaries).enclosingBoundingBox();

+ clippedLocalRect = box->absoluteToLocalQuad(FloatRect(clippedLocalRect), TraverseDocumentBoundaries).enclosingBoundingBox();

}

void WebPluginContainerImpl::calculateGeometry(IntRect& windowRect, IntRect& clipRect, IntRect& unobscuredRect, Vector<IntRect>& cutOutRects)

@@ -1006,4 +1006,4 @@ void WebPluginContainerImpl::calculateGeometry(IntRect& windowRect, IntRect& cli

cutOutRects[i].move(-frameRect().x(), -frameRect().y());

}

-} // namespace blinkf

+} // namespace blink