Allow blank URL when creating popup windows (issue #1289)

This commit is contained in:
Marshall Greenblatt 2015-12-03 13:41:40 -05:00
parent 26e68c3d26
commit 6824387c8d
2 changed files with 2 additions and 2 deletions

View File

@ -139,7 +139,6 @@ void CefBrowserInfoManager::OnCreateWindow(
const ViewHostMsg_CreateWindow_Params& params) {
DCHECK_GT(params.opener_id, 0);
DCHECK_GT(params.opener_render_frame_id, 0);
DCHECK(!params.target_url.is_empty());
scoped_ptr<CefBrowserInfoManager::PendingPopup> pending_popup(
new CefBrowserInfoManager::PendingPopup);
@ -465,7 +464,6 @@ scoped_ptr<CefBrowserInfoManager::PendingPopup>
const GURL& target_url) {
DCHECK_GT(opener_process_id, 0);
DCHECK_GT(opener_view_id, 0);
DCHECK(!target_url.is_empty());
base::AutoLock lock_scope(pending_popup_lock_);

View File

@ -159,6 +159,8 @@ class CefBrowserInfoManager : public content::RenderProcessHostObserver {
} step;
// Initial state from ViewHostMsg_CreateWindow.
// |target_url| will be empty if a popup is created via window.open() and
// never navigated. For example: javascript:window.open();
int opener_process_id;
int opener_view_id;
int64 opener_frame_id;