Remove EPM:all_hosts_ and use all_extension_views_ instead.

content/browser/renderer_host/render_widget_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_widget_host_impl.h"
 
 #include <utility>
 
 #include "base/auto_reset.h"
 #include "base/bind.h"
@@ skipping 225 matching lines @@
 void RenderWidgetHostImpl::Shutdown() {
   if (process_->HasConnection()) {
     // Tell the renderer object to close.
     bool rv = Send(new ViewMsg_Close(routing_id_));
     DCHECK(rv);
   }
 
   Destroy();
 }
 
+bool RenderWidgetHostImpl::IsLoading() const {

[Aaron Boodman, 2012/04/20 06:34:56]

Style should be: is_loading()

[benwells, 2012/04/23 05:00:26]

I think its like this as its a private impl of an abstract class, see GetView
and GetRoutingId above.

[Matt Perry, 2012/04/23 21:53:04]

Yeah, is_loading() style is only used for cheap non-virtual inline methods.

+  return is_loading_;
+}
+
 bool RenderWidgetHostImpl::IsRenderView() const {
   return false;
 }
 
 bool RenderWidgetHostImpl::OnMessageReceived(const IPC::Message &msg) {
   bool handled = true;
   bool msg_is_ok = true;
   IPC_BEGIN_MESSAGE_MAP_EX(RenderWidgetHostImpl, msg, msg_is_ok)
     IPC_MESSAGE_HANDLER(ViewHostMsg_RenderViewReady, OnMsgRenderViewReady)
     IPC_MESSAGE_HANDLER(ViewHostMsg_RenderViewGone, OnMsgRenderViewGone)
@@ skipping 1470 matching lines @@
   // indicate that no callback is in progress (i.e. without this line
   // DelayedAutoResized will not get called again).
   new_auto_size_.SetSize(0, 0);
   if (!should_auto_resize_)
     return;
 
   OnRenderAutoResized(new_size);
 }
 
 }  // namespace content