Convert RenderViewContextMenu to MenuItemView.

chrome/common/automation_messages.h

 // Copyright (c) 2011 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.
 
 // Multiply-included message file, no traditional include guard.
 #include <string>
+#include <vector>
 
 #include "base/basictypes.h"
+#include "base/memory/ref_counted.h"
 #include "chrome/common/automation_constants.h"
 #include "chrome/common/content_settings.h"
 #include "content/common/common_param_traits.h"
 #include "content/common/page_type.h"
 #include "content/common/security_style.h"
 #include "content/common/webkit_param_traits.h"
 #include "ipc/ipc_message_macros.h"
 #include "ipc/ipc_message_utils.h"
 #include "net/base/host_port_pair.h"
 #include "net/base/upload_data.h"
@@ skipping 122 matching lines @@
   GURL src_url;
 
   // This is the URL of the top level page that the context menu was invoked
   // on.
   GURL page_url;
 
   // This is the URL of the subframe that the context menu was invoked on.
   GURL frame_url;
 };
 
+// This struct contains information from ui::MenuModel in Chrome to Chrome
+// Frame for the RenderViewContextMenu.  It is basically a container of
+// items that go in the menu.  An item may be a submenu, so the data
+// structure may be a tree.
+struct ContextMenuModel {

[iyengar, 2011/07/11 22:09:17]

We can probably reword the comment as below:-
"This struct passes information about the context menu in Chrome stored as a
ui::MenuModel to Chrome. 
". The other sentences are fine.

[rhashimoto, 2011/07/11 22:38:58]

Done.

+  ContextMenuModel();
+  ~ContextMenuModel();
+
+  // This struct describes one item in the menu.
+  struct Item {
+    Item();
+
+    // This is the type of the menu item, using values from the enum
+    // ui::MenuModel::ItemType (serialized as an int).
+    int type;
+
+    // This is the command id of the menu item, which will be passed by
+    // to Chrome if the item is selected.
+    int item_id;

[iyengar, 2011/07/11 22:09:17]

which will be passed by Chrome to ChromeFrame?

[rhashimoto, 2011/07/11 22:38:58]

Done.

+
+    // This the the menu label, if needed.
+    std::wstring label;
+
+    // These are states of the menu item.
+    bool checked;
+    bool enabled;
+
+    // This recursively describes the submenu if type is
+    // ui::MenuModel::TYPE_SUBMENU.
+    ContextMenuModel* submenu;
+  };
+
+  // This is the list of menu items.
+  std::vector<Item> items;
+};
+
 struct AttachExternalTabParams {
   AttachExternalTabParams();
   AttachExternalTabParams(uint64 cookie,
                           const GURL& url,
                           const gfx::Rect& dimensions,
                           int disposition,
                           bool user_gesture,
                           const std::string& profile_name);
   ~AttachExternalTabParams();
 
@@ skipping 181 matching lines @@
 
 // Traits for MiniContextMenuParams structure to pack/unpack.
 template <>
 struct ParamTraits<MiniContextMenuParams> {
   typedef MiniContextMenuParams param_type;
   static void Write(Message* m, const param_type& p);
   static bool Read(const Message* m, void** iter, param_type* p);
   static void Log(const param_type& p, std::string* l);
 };
 
+// Traits for ContextMenuModel structure to pack/unpack.
+template <>
+struct ParamTraits<ContextMenuModel> {
+  typedef ContextMenuModel param_type;
+  static void Write(Message* m, const param_type& p);
+  static bool Read(const Message* m, void** iter, param_type* p);
+  static void Log(const param_type& p, std::string* l);
+};
+
 template <>
 struct ParamTraits<AttachExternalTabParams> {
   typedef AttachExternalTabParams param_type;
   static void Write(Message* m, const param_type& p);
   static bool Read(const Message* m, void** iter, param_type* p);
   static void Log(const param_type& p, std::string* l);
 };
 
 }  // namespace IPC
 
 #endif  // CHROME_COMMON_AUTOMATION_MESSAGES_H__
 
 // Keep this internal message file unchanged to preserve line numbering
 // (and hence the dubious __LINE__-based message numberings) across versions.
 #include "chrome/common/automation_messages_internal.h"