Make <webview> use out-of-process iframe architecture.

content/public/browser/web_contents.h

Index: content/public/browser/web_contents.h
diff --git a/content/public/browser/web_contents.h b/content/public/browser/web_contents.h
index 43804129322c4ed56ffc7b451c6ae1e5b84bfc2d..08920a59de204c700fcead3b15b67eb1eb251ab9 100644
--- a/content/public/browser/web_contents.h
+++ b/content/public/browser/web_contents.h
@@ -156,6 +156,14 @@ class WebContents : public PageNavigator,
 
   ~WebContents() override {}
 
+  // Attaches or associates this WebContents to a frame in

[Charlie Reis, 2015/04/30 23:06:47]

this guest WebContents to its embedder frame in...

[lazyboy, 2015/05/05 07:28:15]

Done.

+  // |embedder_web_contents|.
+  // The frame can identified by
+  // {embedder_frame_process_id, embedder_frame_routing_id} pair.

[Charlie Reis, 2015/04/30 23:06:47]

Also say what this returns.

[lazyboy, 2015/05/05 07:28:15]

Done.

+  virtual int AttachLocalFrameToGuest(WebContents* embedder_web_contents,

[Charlie Reis, 2015/04/30 23:06:47]

This name doesn't make sense to me, if we're calling it on the guest.  What
about AttachToEmbedderFrame?

Also, this belongs lower in the class.  Maybe in "Internal state?"

[lazyboy, 2015/05/05 07:28:15]

Ok.
Done.

+                                      int embedder_frame_process_id,
+                                      int embedder_frame_routing_id) = 0;

[Charlie Reis, 2015/04/30 23:06:47]

I think it might be clearer to pass in the the RenderFrameHost instead of the ID
pair.  The caller should be able to get it.

[lazyboy, 2015/05/05 07:28:15]

Done.

+
   // Intrinsic tab state -------------------------------------------------------
 
   // Gets/Sets the delegate.