Fix Flash fullscreen context menu target and position.

content/renderer/pepper/renderer_ppapi_host_impl.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/renderer/pepper/renderer_ppapi_host_impl.h"
 
 #include "base/bind.h"
 #include "base/files/file_path.h"
 #include "base/logging.h"
 #include "base/message_loop/message_loop.h"
@@ skipping 185 matching lines @@
     FullscreenContainer* container = plugin_instance->fullscreen_container();
     return static_cast<RenderWidgetFullscreenPepper*>(container)->routing_id();
   }
   return GetRenderViewForInstance(instance)->GetRoutingID();
 }
 
 gfx::Point RendererPpapiHostImpl::PluginPointToRenderFrame(
     PP_Instance instance,
     const gfx::Point& pt) const {
   PepperPluginInstanceImpl* plugin_instance = GetAndValidateInstance(instance);
-  if (!plugin_instance)
+  if (!plugin_instance || plugin_instance->flash_fullscreen()) {
+    // Flash fullscreen is special in that it renders into its own separate,
+    // dedicated window.  So, do not offset the point.
     return pt;
-
-  RenderFrameImpl* render_frame = static_cast<RenderFrameImpl*>(
-      GetRenderFrameForInstance(instance));
-  if (plugin_instance->view_data().is_fullscreen ||
-      plugin_instance->flash_fullscreen()) {
-    blink::WebRect window_rect = render_frame->GetRenderWidget()->windowRect();
-    blink::WebRect screen_rect =
-        render_frame->GetRenderWidget()->screenInfo().rect;
-    return gfx::Point(pt.x() - window_rect.x + screen_rect.x,
-                      pt.y() - window_rect.y + screen_rect.y);
   }
   return gfx::Point(pt.x() + plugin_instance->view_data().rect.point.x,
                     pt.y() + plugin_instance->view_data().rect.point.y);
 }
 
 IPC::PlatformFileForTransit RendererPpapiHostImpl::ShareHandleWithRemote(
     base::PlatformFile handle,
     bool should_close_source) {
   if (!dispatcher_) {
     DCHECK(is_running_in_process_);
@@ skipping 42 matching lines @@
   PepperPluginInstanceImpl* instance =
       HostGlobals::Get()->GetInstance(pp_instance);
   if (!instance)
     return NULL;
   if (!instance->IsValidInstanceOf(module_))
     return NULL;
   return instance;
 }
 
 }  // namespace content