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

content/browser/web_contents/web_contents_impl.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 CONTENT_BROWSER_WEB_CONTENTS_WEB_CONTENTS_IMPL_H_
 #define CONTENT_BROWSER_WEB_CONTENTS_WEB_CONTENTS_IMPL_H_
 
 #include <map>
 #include <set>
 #include <string>
@@ skipping 200 matching lines @@
   // the accessibility mode.
   typedef base::Callback<void(const ui::AXTreeUpdate&)>
       AXTreeSnapshotCallback;
   void RequestAXTreeSnapshot(AXTreeSnapshotCallback callback);
 
   // Clear the navigation transition data when the user navigates back to Chrome
   // from a native app.
   void ClearNavigationTransitionData();
 
   // WebContents ------------------------------------------------------
+  int AttachLocalFrameToGuest(WebContents* embedder_web_contents,

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

Move lower, as mentioned in web_contents.h.

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

Done.

+                              int embedder_frame_process_id,
+                              int embedder_frame_routing_id) override;
   WebContentsDelegate* GetDelegate() override;
   void SetDelegate(WebContentsDelegate* delegate) override;
   NavigationControllerImpl& GetController() override;
   const NavigationControllerImpl& GetController() const override;
   BrowserContext* GetBrowserContext() const override;
   const GURL& GetURL() const override;
   const GURL& GetVisibleURL() const override;
   const GURL& GetLastCommittedURL() const override;
   RenderProcessHost* GetRenderProcessHost() const override;
   RenderFrameHostImpl* GetMainFrame() override;
@@ skipping 359 matching lines @@
   void NotifyMainFrameSwappedFromRenderManager(
       RenderViewHost* old_host,
       RenderViewHost* new_host) override;
   int CreateOpenerRenderViewsForRenderManager(SiteInstance* instance) override;
   NavigationControllerImpl& GetControllerForRenderManager() override;
   scoped_ptr<WebUIImpl> CreateWebUIForRenderManager(const GURL& url) override;
   NavigationEntry* GetLastCommittedNavigationEntryForRenderManager() override;
   bool FocusLocationBarByDefault() override;
   void SetFocusToLocationBar(bool select_all) override;
   bool IsHidden() override;
+  int64 GetEmbedderFrameTreeNodeID() override;
 
   // NotificationObserver ------------------------------------------------------
 
   void Observe(int type,
                const NotificationSource& source,
                const NotificationDetails& details) override;
 
   // NavigationControllerDelegate ----------------------------------------------
 
   WebContents* GetWebContents() override;
@@ skipping 115 matching lines @@
                            CrossSiteIframeAccessibility);
 
   // So InterstitialPageImpl can access SetIsLoading.
   friend class InterstitialPageImpl;
 
   // TODO(brettw) TestWebContents shouldn't exist!
   friend class TestWebContents;
 
   class DestructionObserver;
 
+  // Used to create tree of WebContents.

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

Please clarify what this is used for, since it's not obvious when reading
WebContentsImpl code.  It's especially important to distinguish it from
FrameTree and FrameTreeNodes.

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

Done.

+  class WebContentsNode {

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

Perhaps this should just be a struct?

WebContentsTreeNode may also make the uses clearer.

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

Done.
Done.

+   public:
+    WebContentsNode();
+    ~WebContentsNode();
+
+    void set_parent_web_contents(WebContentsImpl* parent_web_contents) {
+      parent_web_contents_ = parent_web_contents;
+    }
+    WebContentsImpl* parent_web_contents() {
+      return parent_web_contents_;
+    }
+
+   private:
+    WebContentsImpl* parent_web_contents_;
+  };
+
   // See WebContents::Create for a description of these parameters.
   WebContentsImpl(BrowserContext* browser_context,
                   WebContentsImpl* opener);
 
   // Add and remove observers for page navigation notifications. The order in
   // which notifications are sent to observers is undefined. Clients must be
   // sure to remove the observer before they go away.
   void AddObserver(WebContentsObserver* observer);
   void RemoveObserver(WebContentsObserver* observer);
 
@@ skipping 507 matching lines @@
 
   // The accessibility mode for all frames. This is queried when each frame
   // is created, and broadcast to all frames when it changes.
   AccessibilityMode accessibility_mode_;
 
   // Created on-demand to mute all audio output from this WebContents.
   scoped_ptr<WebContentsAudioMuter> audio_muter_;
 
   bool virtual_keyboard_requested_;
 
+  WebContentsNode node_;

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

This needs a comment and should probably be up higher, perhaps above
frame_tree_.

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

Done.

+
 #if defined(ENABLE_BROWSER_CDMS)
   // Manages all the media player and CDM managers and forwards IPCs to them.
   scoped_ptr<MediaWebContentsObserver> media_web_contents_observer_;
 #endif
 
   base::WeakPtrFactory<WebContentsImpl> loading_weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(WebContentsImpl);
 };
 
@@ skipping 10 matching lines @@
   // Adds/removes a callback called on creation of each new WebContents.
   static void AddCreatedCallbackForTesting(const CreatedCallback& callback);
   static void RemoveCreatedCallbackForTesting(const CreatedCallback& callback);
 
   DISALLOW_COPY_AND_ASSIGN(FriendZone);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_WEB_CONTENTS_WEB_CONTENTS_IMPL_H_