diff options
author | yzshen@chromium.org <yzshen@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-04-11 19:16:02 +0000 |
---|---|---|
committer | yzshen@chromium.org <yzshen@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-04-11 19:16:02 +0000 |
commit | ea192e83450c3391f03c59219dfc9797e912a0d1 (patch) | |
tree | 1a3b36e00cce783a135c2f0edc463eb58ce3e6bd /webkit/plugins/ppapi/ppb_flash_menu_impl.cc | |
parent | dc1e688dc600bd425ad6be0a923c0a6c1d53b6c9 (diff) | |
download | chromium_src-ea192e83450c3391f03c59219dfc9797e912a0d1.zip chromium_src-ea192e83450c3391f03c59219dfc9797e912a0d1.tar.gz chromium_src-ea192e83450c3391f03c59219dfc9797e912a0d1.tar.bz2 |
Fix the issue that context menu doesn't show on fullscreen Pepper Flash.
- generate WebInputEvent::ContextMenu events.
- calculate the context menu position correctly.
- monitor mouse down events on fullscreen render widget, so that we can pass correct timestamp to gtk_menu_popup.
BUG=None.
TEST=Open a fullscreen Youtube video, and right click on it to see whether context menu shows up or not.
Review URL: http://codereview.chromium.org/6760019
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@81135 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'webkit/plugins/ppapi/ppb_flash_menu_impl.cc')
-rw-r--r-- | webkit/plugins/ppapi/ppb_flash_menu_impl.cc | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/webkit/plugins/ppapi/ppb_flash_menu_impl.cc b/webkit/plugins/ppapi/ppb_flash_menu_impl.cc index 52f81b6..d147996 100644 --- a/webkit/plugins/ppapi/ppb_flash_menu_impl.cc +++ b/webkit/plugins/ppapi/ppb_flash_menu_impl.cc @@ -176,8 +176,7 @@ int32_t PPB_Flash_Menu_Impl::Show(const PP_Point* location, } int32_t rv = instance()->delegate()->ShowContextMenu( - this, gfx::Point(instance()->position().x() + location->x, - instance()->position().y() + location->y)); + instance(), this, gfx::Point(location->x, location->y)); if (rv == PP_ERROR_WOULDBLOCK) { // Record callback and output buffers. callback_ = new TrackedCompletionCallback( |