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

content/browser/frame_host/render_widget_host_view_child_frame.cc

 // Copyright 2014 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.
 
 #include "content/browser/frame_host/render_widget_host_view_child_frame.h"
 
+#include "base/command_line.h"
 #include "cc/surfaces/surface.h"
 #include "cc/surfaces/surface_factory.h"
 #include "cc/surfaces/surface_manager.h"
 #include "cc/surfaces/surface_sequence.h"
 #include "content/browser/accessibility/browser_accessibility_manager.h"
+#include "content/browser/browser_plugin/browser_plugin_guest.h"
 #include "content/browser/compositor/surface_utils.h"
 #include "content/browser/frame_host/cross_process_frame_connector.h"
 #include "content/browser/gpu/compositor_util.h"
+#include "content/browser/renderer_host/render_view_host_impl.h"
 #include "content/browser/renderer_host/render_widget_host_impl.h"
 #include "content/common/gpu/gpu_messages.h"
 #include "content/common/view_messages.h"
 #include "content/public/browser/render_process_host.h"
+#include "content/public/common/content_switches.h"
 
 namespace content {
 
 RenderWidgetHostViewChildFrame::RenderWidgetHostViewChildFrame(
     RenderWidgetHost* widget_host)
     : host_(RenderWidgetHostImpl::From(widget_host)),
       use_surfaces_(UseSurfacesEnabled()),
       next_surface_sequence_(1u),
       last_output_surface_id_(0),
       current_surface_scale_factor_(1.f),
@@ skipping 115 matching lines @@
 }
 
 void RenderWidgetHostViewChildFrame::MovePluginWindows(
     const std::vector<WebPluginGeometry>& moves) {
 }
 
 void RenderWidgetHostViewChildFrame::UpdateCursor(const WebCursor& cursor) {
 }
 
 void RenderWidgetHostViewChildFrame::SetIsLoading(bool is_loading) {
+  // Inner WebContents reaches this, so skip the NOTREACHED() check below.

[nasko, 2015/06/15 23:00:13]

Missing TODO on how to fix this?

[lazyboy, 2015/06/16 17:59:51]

Hmmm, I don't think this needs to be fixed.
Subframe's RWHVChildFrame wouldn't reach this but inner WebContents's
one would because we'd see WebContentsImpl::DidStart/StopLoading().
Updated the comment so it doesn't sound like it is a TODO.

+  if (base::CommandLine::ForCurrentProcess()->HasSwitch(
+          switches::kSitePerProcess) &&
+      BrowserPluginGuest::IsGuest(
+          static_cast<RenderViewHostImpl*>(RenderViewHost::From(host_)))) {
+    return;
+  }
+
   NOTREACHED();
 }
 
 void RenderWidgetHostViewChildFrame::TextInputTypeChanged(
     ui::TextInputType type,
     ui::TextInputMode input_mode,
     bool can_compose_inline,
     int flags) {
   // TODO(kenrb): Implement.
 }
@@ skipping 248 matching lines @@
 }
 
 BrowserAccessibilityManager*
 RenderWidgetHostViewChildFrame::CreateBrowserAccessibilityManager(
     BrowserAccessibilityDelegate* delegate) {
   return BrowserAccessibilityManager::Create(
       BrowserAccessibilityManager::GetEmptyDocument(), delegate);
 }
 
 }  // namespace content