Bug 22968 – Context menu rendering error when full page zoomed

• Component: WebCore Misc.
• Status: RESOLVED FIXED

[Josh Roesslein]

The right click context menu is not rendered in the right position when full page zoom is in use. Also the incorrect context menu type (link, image, etc) is incorrect. 

Cause of the issue is due to lack of transformation to account for the page zoom. By transforming the context menu's point, this issue is resolved.

This is an upstream issue from Chromium --> http://code.google.com/p/chromium/issues/detail?id=3474

I have been working on resolving this issue in chrome, but have determined that this should be an upstreamed patch to webkit.

Patch --> http://codereview.chromium.org/16204/show

Currently I have fixed the issue in our glue layer between webkit and chrome.
I feel this should be moved into the ContextMenuController::handleContextMenuEvent method instead.

[Josh Roesslein]

Created attachment 26214 [details]
patch to fix issue

[Oliver Hunt]

Comment on attachment 26214 [details]
patch to fix issue

This is basically okay, i'd just like an updated version of this patch with the 'if (frame->pageZoomFactor() != 1.0f)' branch removed.

[Josh Roesslein]

Created attachment 26309 [details]
Patch #2

revised orginal patch to remove the if zoomed test

[Oliver Hunt]

Committing to http://svn.webkit.org/repository/webkit/trunk ...
	M	WebCore/ChangeLog
	M	WebCore/page/ContextMenuController.cpp
Committed r39506