Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1454)

Unified Diff: content/browser/web_contents/web_contents_impl.cc

Issue 10378089: Browser Plugin: Removed BrowserPluginWebContentsObserver (to be replaced by rewritten BrowserPlugi… (Closed) Base URL: http://git.chromium.org/git/chromium.git@trunk
Patch Set: Default to 0 Ids in IntersitialPageImpl Created 8 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/browser/web_contents/web_contents_impl.cc
diff --git a/content/browser/web_contents/web_contents_impl.cc b/content/browser/web_contents/web_contents_impl.cc
index 2aba4bcb91a3ec52f4d25e195000466ec695f8ed..10610dfb84c000b70c14d94faf13a517ac91c1a0 100644
--- a/content/browser/web_contents/web_contents_impl.cc
+++ b/content/browser/web_contents/web_contents_impl.cc
@@ -14,7 +14,6 @@
#include "base/string_util.h"
#include "base/time.h"
#include "base/utf_string_conversions.h"
-#include "content/browser/browser_plugin/browser_plugin_web_contents_observer.h"
#include "content/browser/child_process_security_policy_impl.h"
#include "content/browser/debugger/devtools_manager_impl.h"
#include "content/browser/dom_storage/session_storage_namespace_impl.h"
@@ -326,9 +325,6 @@ WebContentsImpl::WebContentsImpl(
java_bridge_dispatcher_host_manager_.reset(
new JavaBridgeDispatcherHostManager(this));
#endif
-
- browser_plugin_web_contents_observer_.reset(
- new content::BrowserPluginWebContentsObserver(this));
}
WebContentsImpl::~WebContentsImpl() {
@@ -669,10 +665,20 @@ content::RenderProcessHost* WebContentsImpl::GetRenderProcessHost() const {
return host ? host->GetProcess() : NULL;
}
+content::RenderProcessHost*
+ WebContentsImpl::GetPendingRenderProcessHost() const {
+ RenderViewHostImpl* host = render_manager_.pending_render_view_host();
+ return host ? host->GetProcess() : NULL;
+}
+
RenderViewHost* WebContentsImpl::GetRenderViewHost() const {
return render_manager_.current_host();
}
+RenderViewHost* WebContentsImpl::GetPendingRenderViewHost() const {
+ return render_manager_.pending_render_view_host();
+}
+
RenderWidgetHostView* WebContentsImpl::GetRenderWidgetHostView() const {
return render_manager_.GetRenderWidgetHostView();
}
@@ -2664,8 +2670,10 @@ bool WebContentsImpl::CreateRenderViewForRenderManager(
GetMaxPageIDForSiteInstance(render_view_host->GetSiteInstance());
if (!static_cast<RenderViewHostImpl*>(
- render_view_host)->CreateRenderView(string16(), opener_route_id,
- max_page_id)) {
+ render_view_host)->CreateRenderView(string16(),
+ opener_route_id,
+ max_page_id,
+ 0, 0)) {
return false;
}

Powered by Google App Engine
This is Rietveld 408576698