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

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

Issue 1095913002: Fix window.open issue in Document mode on Android. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Add GuestViewBase delegate method Created 5 years, 8 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 59791b51e94f69bfbe162b375242ea576cb65ebc..6228fbf1316f1e4295bfe077168b7be7b1e8a708 100644
--- a/content/browser/web_contents/web_contents_impl.cc
+++ b/content/browser/web_contents/web_contents_impl.cc
@@ -1702,6 +1702,9 @@ void WebContentsImpl::ShowCreatedWindow(int route_id,
WebContentsImpl* contents = GetCreatedWindow(route_id);
if (contents) {
WebContentsDelegate* delegate = GetDelegate();
+ if (!delegate || delegate->ShouldResumeRequestsForCreatedWindow())
+ contents->ResumeLoadingCreatedWebContents();
+
if (delegate) {
delegate->AddNewContents(
this, contents, disposition, initial_rect, user_gesture, NULL);
@@ -1789,11 +1792,6 @@ WebContentsImpl* WebContentsImpl::GetCreatedWindow(int route_id) {
!new_contents->GetRenderViewHost()->GetView())
return NULL;
- // Resume blocked requests for both the RenderViewHost and RenderFrameHost.
- // TODO(brettw): It seems bogus to reach into here and initialize the host.
- new_contents->GetRenderViewHost()->Init();
- new_contents->GetMainFrame()->Init();
-
return new_contents;
}
@@ -2540,6 +2538,13 @@ void WebContentsImpl::ExitFullscreen() {
ExitFullscreenMode();
}
+void WebContentsImpl::ResumeLoadingCreatedWebContents() {
+ // Resume blocked requests for both the RenderViewHost and RenderFrameHost.
+ // TODO(brettw): It seems bogus to reach into here and initialize the host.
+ GetRenderViewHost()->Init();
+ GetMainFrame()->Init();
+}
+
bool WebContentsImpl::FocusLocationBarByDefault() {
NavigationEntry* entry = controller_.GetVisibleEntry();
if (entry && entry->GetURL() == GURL(url::kAboutBlankURL))

Powered by Google App Engine
This is Rietveld 408576698