Add the screen offset to the location of the flash context menu.

content/renderer/pepper/pepper_plugin_delegate_impl.cc

Index: content/renderer/pepper/pepper_plugin_delegate_impl.cc
diff --git a/content/renderer/pepper/pepper_plugin_delegate_impl.cc b/content/renderer/pepper/pepper_plugin_delegate_impl.cc
index c4d8891516f00b2b48d2c049e4dde92f9ac3c0ce..e690fb4e8e65eb9146748b80ecea1019125af30c 100644
--- a/content/renderer/pepper/pepper_plugin_delegate_impl.cc
+++ b/content/renderer/pepper/pepper_plugin_delegate_impl.cc
@@ -1279,9 +1279,10 @@ int32_t PepperPluginDelegateImpl::ShowContextMenu(
 
   // Transform the position to be in render view's coordinates.
   if (instance->view_data().is_fullscreen || instance->flash_fullscreen()) {
-    WebKit::WebRect rect = render_view_->windowRect();
-    params.x -= rect.x;
-    params.y -= rect.y;
+    WebKit::WebRect window_rect = render_view_->windowRect();

[viettrungluu, 2012/08/05 05:55:16]

Are you sure this doesn't break multi-monitor fullscreen for Pepper, non-Flash
fullscreen?

This change feels rather wrong. At the same time, the existing coordinate
transformation feels rather wrong: The render view is interpreting the
coordinates relative to the normal, non-fullscreen window instead of the render
widget's (which is actually in |params| -- so the render view actually knows
which render widget is being targeted).

That said, I'm too lazy to go read code on the renderer host side, and if this
doesn't break anything, it'll do for now.

[raymes, 2012/08/05 06:19:57]

AFAICT this isn't currently used by anything other than flash (the only use I
see is from
http://code.google.com/searchframe#OAMlx_jo-ck/src/webkit/plugins/ppapi/ppb_f...
).  If there is some other non-flash usage, I'm happy to test. I'm not sure
whether it /would/ break non-flash plugins if it were used. 
I agree, but I wasn't sure if this was intended/expected behavior since the
existing code intentionally did the transformation to the windows coordinate
space. I can file a bug to investigate.

+    WebKit::WebRect screen_rect = render_view_->screenInfo().rect;
+    params.x = params.x - window_rect.x + screen_rect.x;
+    params.y = params.y - window_rect.y + screen_rect.y;
   } else {
     params.x += instance->view_data().rect.point.x;
     params.y += instance->view_data().rect.point.y;