ContentSettings API should not interact with <webview>

content/browser/renderer_host/render_view_host_impl.cc

 // 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.
 
 #include "content/browser/renderer_host/render_view_host_impl.h"
 
 #include <set>
 #include <string>
 #include <utility>
 #include <vector>
@@ skipping 1358 matching lines @@
     // The ack for the mouse down won't come until the dialog closes, so fake it
     // so that we don't get a timeout.
     opener->decrement_in_flight_event_count();
   }
 
   // TODO(darin): Bug 1107929: Need to inform our delegate to show this view in
   // an app-modal fashion.
 }
 
 void RenderViewHostImpl::OnRenderViewReady() {
+  GetContentClient()->browser()->RenderViewReady(this);

[jam, 2013/11/25 22:42:35]

this duplicates WebContentsObserver::RenderViewReady, please use that instead of
adding another method to ContentBrowserClient. The latter is a last-resort place
to put embedder callbacks when there isn't context 

[jam, 2013/11/25 22:51:53]

to expand, you can put this on TabSpecificContentSettings::RenderViewReady

   render_view_termination_status_ = base::TERMINATION_STATUS_STILL_RUNNING;
   SendScreenRects();
   WasResized();
   delegate_->RenderViewReady(this);
 }
 
 void RenderViewHostImpl::OnRenderProcessGone(int status, int exit_code) {
   // Keep the termination status so we can get at it later when we
   // need to know why it died.
   render_view_termination_status_ =
@@ skipping 896 matching lines @@
 void RenderViewHostImpl::AttachToFrameTree() {
   FrameTree* frame_tree = delegate_->GetFrameTree();
 
   frame_tree->SwapMainFrame(main_render_frame_host_.get());
   if (main_frame_id() != FrameTreeNode::kInvalidFrameId) {
     frame_tree->OnFirstNavigationAfterSwap(main_frame_id());
   }
 }
 
 }  // namespace content