WebView context menu cleanup.

chrome/browser/extensions/menu_manager.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_EXTENSIONS_MENU_MANAGER_H_
 #define CHROME_BROWSER_EXTENSIONS_MENU_MANAGER_H_
 
 #include <map>
 #include <set>
 #include <string>
@@ skipping 26 matching lines @@
 class Extension;
 class ExtensionRegistry;
 class StateStore;
 
 // Represents a menu item added by an extension.
 class MenuItem {
  public:
   // A list of MenuItems.
   typedef std::vector<MenuItem*> List;
 
-  // Key used to identify which extension a menu item belongs to.
-  // A menu item can also belong to a <webview> inside an extension,
-  // only in that case |webview_instance_id| would be
-  // non-zero (i.e. != guestview::kInstanceIDNone).
+  // EnstensionKey is used to identify a menu item. If the menu item belongs to
+  // an extension, then |extension_id| will be non-zero. Alternatively, if the

[lazyboy, 2015/06/19 20:34:59]

|extension_id| will be non-empty you mean?

[paulmeyer, 2015/06/23 13:43:28]

Done.

+  // menu item belongs to a WebView, then |webview_embedder_process_id| and
+  // |webview_instance_id| will be non-zero.
+  // TODO(paulmeyer): This should be called something else, since it is not just
+  // associated with extensions.
   struct ExtensionKey {
     std::string extension_id;
+    int webview_embedder_process_id;
     int webview_instance_id;
 
     ExtensionKey();
-    ExtensionKey(const std::string& extension_id, int webview_instance_id);
     explicit ExtensionKey(const std::string& extension_id);
+    ExtensionKey(int webview_embedder_process_id, int webview_instance_id);
 
     bool operator==(const ExtensionKey& other) const;
     bool operator!=(const ExtensionKey& other) const;
     bool operator<(const ExtensionKey& other) const;
 
     bool empty() const;
   };
 
   // An Id uniquely identifies a context menu item registered by an extension.
   struct Id {
@@ skipping 333 matching lines @@
 
   // Owned by ExtensionSystem.
   StateStore* store_;
 
   DISALLOW_COPY_AND_ASSIGN(MenuManager);
 };
 
 }  // namespace extensions
 
 #endif  // CHROME_BROWSER_EXTENSIONS_MENU_MANAGER_H_