Browser Plugin: Disable context menu on guests

content/renderer/render_view_impl.cc

Index: content/renderer/render_view_impl.cc
diff --git a/content/renderer/render_view_impl.cc b/content/renderer/render_view_impl.cc
index aeecc2258e9a64a50555f61a2f2d2d16f1ff1dfb..ef17453e6f08224d8766a8fa998e92cd874d9b46 100644
--- a/content/renderer/render_view_impl.cc
+++ b/content/renderer/render_view_impl.cc
@@ -2002,6 +2002,11 @@ bool RenderViewImpl::runModalBeforeUnloadDialog(
 
 void RenderViewImpl::showContextMenu(
     WebFrame* frame, const WebContextMenuData& data) {
+  // TODO(fsamuel): In the future, we might want the embedder to be able to
+  // decide whether the guest can show a context menu or not.

[jam, 2012/06/22 20:43:05]

nit: put the bug number here

+  if (GetGuestToEmbedderChannel())
+    return;

[Charlie Reis, 2012/06/22 19:55:23]

I'm not 100% sure that the renderer isn't expecting some kind of reply from
this, but I think it's ok.

[Fady Samuel, 2012/06/22 20:03:44]

OK. This seems to work OK. A good fix with an API will take longer but this way
avoids the crash and invalid options like "Open Link in New Window" until we
have an API.

+
   content::ContextMenuParams params(data);
 
   // frame is NULL if invoked by BlockedPlugin.