Remove BrowserPlugin's -internal-attach method

content/public/browser/browser_plugin_guest_manager.h

Index: content/public/browser/browser_plugin_guest_manager.h
diff --git a/content/public/browser/browser_plugin_guest_manager.h b/content/public/browser/browser_plugin_guest_manager.h
index 9839dfb91c1aa8c182853a728f954d5e51ec8829..0a901329d9fea2c7b30f7a1722f3b7307e53f48b 100644
--- a/content/public/browser/browser_plugin_guest_manager.h
+++ b/content/public/browser/browser_plugin_guest_manager.h
@@ -39,6 +39,13 @@ class CONTENT_EXPORT BrowserPluginGuestManager {
       int embedder_render_process_id,
       const GuestByInstanceIDCallback& callback) {}
 
+  // Translates a content/ generated instance ID to chrome/ instance ID for a
+  // Browser Plugin.
+  // TODO(lazyboy): This is temporary, once we have clear separation of
+  // content/ vs chrome/ IDs, remove this.

[jam, 2014/08/15 17:41:04]

this is breaking http://www.chromium.org/developers/content-module/content-api

"methods in the API should be there because either content is calling out to its
embedder, or the embedder is calling to content. There shouldn't be any methods
which are used to call from the embedder to the embedder."

[Fady Samuel, 2014/08/16 23:24:30]

Removed. 

[Fady Samuel, 2014/08/16 23:24:30]

Removed.

+  virtual int GetGuestInstanceIDForPluginID(WebContents* embedder_web_contents,
+                                            int element_instance_id);
+
   // Iterates over all WebContents belonging to a given |embedder_web_contents|,
   // calling |callback| for each. If one of the callbacks returns true, then
   // the iteration exits early.