Fixing the Position of Context Menu for BrowserPlugin (<webview>)

content/browser/browser_plugin/browser_plugin_guest.h

Index: content/browser/browser_plugin/browser_plugin_guest.h
diff --git a/content/browser/browser_plugin/browser_plugin_guest.h b/content/browser/browser_plugin/browser_plugin_guest.h
index 868fcc8a35ab07c5920b2e70f48a9c25ceca03b3..ec75b63fa8ccbb26e4f5f41312faab36637951a3 100644
--- a/content/browser/browser_plugin/browser_plugin_guest.h
+++ b/content/browser/browser_plugin/browser_plugin_guest.h
@@ -187,6 +187,8 @@ class CONTENT_EXPORT BrowserPluginGuest : public GuestHost,
 
   gfx::Point GetScreenCoordinates(const gfx::Point& relative_position) const;
 
+  void SetContextMenuPosition(const gfx::Point& position);
+
   // Helper to send messages to embedder. If this guest is not yet attached,
   // then IPCs will be queued until attachment.
   void SendMessageToEmbedder(IPC::Message* msg);
@@ -440,6 +442,9 @@ class CONTENT_EXPORT BrowserPluginGuest : public GuestHost,
   // Indicates the URL dragged into the guest if any.
   GURL dragged_url_;
 
+  // An offset used to find the position of the context menu
+  gfx::Point context_menu_position_;

[wjmaclean, 2015/08/17 13:05:23]

Why do we have this? It doesn't seem to be used (or did I miss something?)

[Ehsaan, 2015/08/17 14:09:12]

My mistake! I was initially setting this value here, but then moved it to the
ChromeWebViewGuestDelegate. I will remove this.

+
   // Guests generate frames and send a CompositorFrameSwapped (CFS) message
   // indicating the next frame is ready to be positioned and composited.
   // Subsequent frames are not generated until the IPC is ACKed. We would like