Fix the context menu positioning for ExtensionOptionsGuest

chrome/browser/guest_view/extension_options/extension_options_guest.cc

Index: chrome/browser/guest_view/extension_options/extension_options_guest.cc
diff --git a/chrome/browser/guest_view/extension_options/extension_options_guest.cc b/chrome/browser/guest_view/extension_options/extension_options_guest.cc
index 1e1866b25c8d9db082d1241109a10abe1eaadb75..90d79e3b5e65dba7ab32e64834bd88ad6b334bfb 100644
--- a/chrome/browser/guest_view/extension_options/extension_options_guest.cc
+++ b/chrome/browser/guest_view/extension_options/extension_options_guest.cc
@@ -9,6 +9,7 @@
 #include "chrome/browser/extensions/extension_tab_util.h"
 #include "chrome/browser/guest_view/extension_options/extension_options_constants.h"
 #include "chrome/browser/profiles/profile.h"
+#include "chrome/browser/renderer_context_menu/render_view_context_menu.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/browser_finder.h"
 #include "chrome/browser/ui/browser_window.h"
@@ -16,6 +17,7 @@
 #include "chrome/common/extensions/api/extension_options_internal.h"
 #include "chrome/common/extensions/manifest_url_handler.h"
 #include "components/crx_file/id_util.h"
+#include "components/renderer_context_menu/context_menu_delegate.h"
 #include "content/public/browser/render_process_host.h"
 #include "content/public/browser/site_instance.h"
 #include "content/public/browser/web_contents.h"
@@ -193,6 +195,18 @@ void ExtensionOptionsGuest::CloseContents(content::WebContents* source) {
       make_scoped_ptr(new base::DictionaryValue())));
 }
 
+bool ExtensionOptionsGuest::HandleContextMenu(
+    const content::ContextMenuParams& params) {

[Fady Samuel, 2014/08/27 22:19:10]

Could you move this code to GuestViewBase, and delete it here and in
AppViewGuest? WebViewGuest does its own implementation.

[Fady Samuel, 2014/08/27 22:19:54]

Actually, nm, I forgot GuestViewBase moved to extensions :P That doesn't work.
LGTM.

+  ContextMenuDelegate* menu_delegate =
+      ContextMenuDelegate::FromWebContents(guest_web_contents());
+  DCHECK(menu_delegate);
+
+  scoped_ptr<RenderViewContextMenu> menu =
+      menu_delegate->BuildMenu(guest_web_contents(), params);
+  menu_delegate->ShowMenu(menu.Pass());
+  return true;
+}
+
 bool ExtensionOptionsGuest::ShouldCreateWebContents(
     content::WebContents* web_contents,
     int route_id,