Separate out logic to handle different category/group of context menu items from RVContextMenu class

chrome/browser/tab_contents/render_view_context_menu.h

 // Copyright (c) 2012 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.
 
 #ifndef CHROME_BROWSER_TAB_CONTENTS_RENDER_VIEW_CONTEXT_MENU_H_
 #define CHROME_BROWSER_TAB_CONTENTS_RENDER_VIEW_CONTEXT_MENU_H_
 
 #include <map>
 #include <string>
 
 #include "base/memory/scoped_ptr.h"
 #include "base/observer_list.h"
 #include "base/strings/string16.h"
 #include "chrome/browser/custom_handlers/protocol_handler_registry.h"
 #include "chrome/browser/extensions/context_menu_matcher.h"
 #include "chrome/browser/extensions/menu_manager.h"
+#include "chrome/browser/tab_contents/menu_category_base.h"
 #include "chrome/browser/tab_contents/render_view_context_menu_observer.h"
 #include "content/public/common/context_menu_params.h"
 #include "content/public/common/page_transition_types.h"
 #include "ui/base/models/simple_menu_model.h"
 #include "ui/base/window_open_disposition.h"
 
 class PrintPreviewContextMenuObserver;
 class Profile;
 class SpellingMenuObserver;
 class SpellCheckerSubMenuObserver;
@@ skipping 115 matching lines @@
 
   const ui::MenuModel& menu_model() const { return menu_model_; }
 
   // SimpleMenuModel::Delegate implementation.
   virtual bool IsCommandIdChecked(int command_id) const OVERRIDE;
   virtual bool IsCommandIdEnabled(int command_id) const OVERRIDE;
   virtual void ExecuteCommand(int command_id, int event_flags) OVERRIDE;
   virtual void MenuWillShow(ui::SimpleMenuModel* source) OVERRIDE;
   virtual void MenuClosed(ui::SimpleMenuModel* source) OVERRIDE;
 
-  // RenderViewContextMenuDelegate implementation.
+  // RenderViewContextMenuProxy implementation.
   virtual void AddMenuItem(int command_id,
                            const base::string16& title) OVERRIDE;
   virtual void AddCheckItem(int command_id,
                             const base::string16& title) OVERRIDE;
   virtual void AddSeparator() OVERRIDE;
   virtual void AddSubMenu(int command_id,
                           const base::string16& label,
                           ui::MenuModel* model) OVERRIDE;
   virtual void UpdateMenuItem(int command_id,
                               bool enabled,
@@ skipping 33 matching lines @@
   static bool ExtensionContextAndPatternMatch(
       const content::ContextMenuParams& params,
       extensions::MenuItem::ContextList contexts,
       const extensions::URLPatternSet& target_url_patterns);
   static bool MenuItemMatchesParams(
       const content::ContextMenuParams& params,
       const extensions::MenuItem* item);
 
   // Gets the extension (if any) associated with the WebContents that we're in.
   const extensions::Extension* GetExtension() const;
-  void AppendAppModeItems();
-  void AppendPlatformAppItems();
-  void AppendPopupExtensionItems();
-  void AppendPanelItems();
   bool AppendCustomItems();

[Avi (use Gerrit), 2014/02/22 01:37:45]

blank line above this

[lazyboy, 2014/02/24 23:45:00]

Done.

   void AppendDeveloperItems();
+  void AppendDevtoolsForUnpackedExtensions();
   void AppendLinkItems();
   void AppendImageItems();
   void AppendAudioItems();
   void AppendVideoItems();
   void AppendMediaItems();
   void AppendPluginItems();
   void AppendPageItems();
   void AppendFrameItems();
   void AppendCopyItem();
   void AppendPrintItem();
   void AppendEditableItems();
   void AppendSearchProvider();
   void AppendAllExtensionItems();
+  void AppendCurrentExtensionItems();
+  void AppendPrintPreviewItems();
+  void AppendSearchWebForImageItems();
   void AppendSpellingSuggestionsSubMenu();
   void AppendSpellcheckOptionsSubMenu();
   void AppendSpeechInputOptionsSubMenu();
   void AppendProtocolHandlerSubMenu();
 
   // Opens the specified URL string in a new tab.  The |frame_id| specifies the
   // frame in which the context menu was displayed, or 0 if the menu action is
   // independent of that frame (e.g. protocol handler settings).
   void OpenURL(const GURL& url, const GURL& referrer, int64 frame_id,
                WindowOpenDisposition disposition,
@@ skipping 45 matching lines @@
   scoped_ptr<PrintPreviewContextMenuObserver> print_preview_menu_observer_;
 #endif
 
   // Our observers.
   mutable ObserverList<RenderViewContextMenuObserver> observers_;
 
   // Whether a command has been executed. Used to track whether menu observers
   // should be notified of menu closing without execution.
   bool command_executed_;
 
-  // Whether or not the menu was triggered for a browser plugin guest.
-  // Guests are rendered inside chrome apps, but have most of the actions
-  // that a regular web page has.
-  // Currently actions/items that are suppressed from guests are: searching,
-  // printing, speech and instant.
-  bool is_guest_;
+  scoped_ptr<MenuCategoryBase> menu_category_;
 
   DISALLOW_COPY_AND_ASSIGN(RenderViewContextMenu);
 };
 
 #endif  // CHROME_BROWSER_TAB_CONTENTS_RENDER_VIEW_CONTEXT_MENU_H_