Split extensions::MenuManager instance out from ExtensionService.

extensions/browser/process_manager.cc

Index: extensions/browser/process_manager.cc
diff --git a/extensions/browser/process_manager.cc b/extensions/browser/process_manager.cc
index 8c664ec519b51a8b4828c5100b010030459a3e88..3ae5be4094cbe04484100407beb629defd61a491 100644
--- a/extensions/browser/process_manager.cc
+++ b/extensions/browser/process_manager.cc
@@ -83,7 +83,7 @@ class IncognitoProcessManager : public ProcessManager {
  public:
   IncognitoProcessManager(BrowserContext* incognito_context,
                           BrowserContext* original_context);
-  virtual ~IncognitoProcessManager();
+  virtual ~IncognitoProcessManager() {}
   virtual ExtensionHost* CreateBackgroundHost(const Extension* extension,
                                               const GURL& url) OVERRIDE;
   virtual SiteInstance* GetSiteInstanceForURL(const GURL& url) OVERRIDE;
@@ -792,15 +792,6 @@ IncognitoProcessManager::IncognitoProcessManager(
                     content::Source<BrowserContext>(original_context));
 }
 
-IncognitoProcessManager::~IncognitoProcessManager() {
-  // TODO(yoz): This cleanup code belongs in the MenuManager.

[James Cook, 2013/11/13 17:04:31]

Thanks for cleaning this up!

-  // Remove "incognito" "split" mode context menu items.
-  ExtensionService* service = ExtensionSystem::GetForBrowserContext(
-      GetBrowserContext())->extension_service();
-  if (service)
-    service->menu_manager()->RemoveAllIncognitoContextItems();
-}
-
 ExtensionHost* IncognitoProcessManager::CreateBackgroundHost(
     const Extension* extension, const GURL& url) {
   if (IncognitoInfo::IsSplitMode(extension)) {