[webkit-reviews] review denied: [Bug 132180] [GTK] Implement iframe shim technique for windowed plugins in WebKit2 : [Attachment 230170] Try to fix mac and EFL builds
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Fri Apr 25 07:37:52 PDT 2014
Martin Robinson <mrobinson at webkit.org> has denied Carlos Garcia Campos
<cgarcia at igalia.com>'s request for review:
Bug 132180: [GTK] Implement iframe shim technique for windowed plugins in
WebKit2
https://bugs.webkit.org/show_bug.cgi?id=132180
Attachment 230170: Try to fix mac and EFL builds
https://bugs.webkit.org/attachment.cgi?id=230170&action=review
------- Additional Comments from Martin Robinson <mrobinson at webkit.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=230170&action=review
GTK+ bits look good to me.
> Source/WebKit2/WebProcess/Plugins/PluginView.cpp:1027
> +static bool iframeIsAbovePlugin(const Vector<const RenderObject*>&
iframeZIndexStack, const Vector<const RenderObject*>& pluginZIndexStack)
> +{
> + for (size_t i = 0; i < iframeZIndexStack.size() && i <
pluginZIndexStack.size(); ++i) {
> + // The root is at the end of these stacks. We want to iterate
root-downwards so we index backwards from the end.
> + const RenderObject* iframeStackRootObject =
iframeZIndexStack[iframeZIndexStack.size() - 1 - i];
> + const RenderObject* pluginStackRootObject =
pluginZIndexStack[pluginZIndexStack.size() - 1 - i];
This requires preserving the original copyright, I think.
More information about the webkit-reviews
mailing list