In order to calculate the exact position of the context menu on netscape plugins, the plugin instances need to know the size of the window frame in screen coordinates. This size is passed to NetscapePlugin::windowAndViewFramesChanged() from WebPageProxy::windowAndViewFramesChanged() and WebPageProxy::windowAndViewFramesChanged() gets the size from API::UIClient::windowFrame(), but the result is always 0 because there is no overridden implementation to return the actual size of the window frame.
Created attachment 255324 [details]
Screenshot of the issue
Created attachment 255325 [details]
Any good way to make a regression test? Anders, does this seem like the right fix?
Is there a regression test suite for testing MiniBrowser itself?
And I wanna know the patch is right.
(In reply to comment #4)
> Is there a regression test suite for testing MiniBrowser itself?
No. But despite the bug title, this is not a bug in MiniBrowser.
Need to double check what we do inside Safari and why there's even a UIDelegate client callback for this.
Comment on attachment 255325 [details]
This patch has been pending review since 2015 with no recent activity.
It seems unlikely that it would even still apply to trunk in its current form.
Clearing from the review queue.
Feel free to update and resubmit if the patch is still relevant.