Hide default context menu for platform apps.

chrome/browser/renderer_host/chrome_render_view_host_observer.cc

Index: chrome/browser/renderer_host/chrome_render_view_host_observer.cc
diff --git a/chrome/browser/renderer_host/chrome_render_view_host_observer.cc b/chrome/browser/renderer_host/chrome_render_view_host_observer.cc
index 6ce517d6fa845629f7ea38b6056b13b63b7cbb27..e11ed2fcf48abad136c5d5f634a5c3c942ff6ba6 100644
--- a/chrome/browser/renderer_host/chrome_render_view_host_observer.cc
+++ b/chrome/browser/renderer_host/chrome_render_view_host_observer.cc
@@ -19,6 +19,7 @@
 #include "content/browser/renderer_host/render_view_host_delegate.h"
 #include "content/browser/renderer_host/render_view_host.h"
 #include "content/browser/site_instance.h"
+#include "content/common/view_messages.h"
 #include "content/public/browser/notification_service.h"
 
 ChromeRenderViewHostObserver::ChromeRenderViewHostObserver(
@@ -64,6 +65,13 @@ bool ChromeRenderViewHostObserver::OnMessageReceived(
                         OnFocusedEditableNodeTouched)
     IPC_MESSAGE_UNHANDLED(handled = false)
   IPC_END_MESSAGE_MAP()
+
+  if (message.type() == ViewHostMsg_ContextMenu::ID) {

[Mihai Parparita -not on Chrome, 2011/12/08 03:34:52]

Blocking this at the IPC level doesn't seem quite right. I think there are two
ways we can approach context menus in platform apps:

1. Remove context menu support entirely, since it's up to apps to decide how
they want to handle right-clicks. In that case, we would ideally want to prevent
the IPC being sent in the first place. This might require making WebKit changes
(perhaps in
https://www.google.com/codesearch#OAMlx_jo-ck/src/third_party/WebKit/Source/W...
to allow the ContextMenuClient (of which Chrome has an implementation) to say
that that context menus shouldn't be shown at all).

2. Allow platform apps to customize the content of context menus (they're not
shown by default, but they can request items to be shown in it). In that case
we'd want to modify the behavior of RenderViewContextMenu::InitMenu
(https://www.google.com/codesearch#OAMlx_jo-ck/src/chrome/browser/tab_contents...),
so that for platform apps none of the other context menu items are added, but
the ones specified by the app are added.

+    const Extension* extension = GetExtension();
+    if (extension && extension->is_platform_app())
+      handled = true;
+  }
+
   return handled;
 }