diff --git a/shell/browser/api/electron_api_web_contents.cc b/shell/browser/api/electron_api_web_contents.cc index e5f4c41dff..d8ab1e0844 100644 --- a/shell/browser/api/electron_api_web_contents.cc +++ b/shell/browser/api/electron_api_web_contents.cc @@ -1051,6 +1051,9 @@ void WebContents::InitWithWebContents( } WebContents::~WebContents() { + if (inspectable_web_contents_) + inspectable_web_contents_->GetView()->SetDelegate(nullptr); + if (owner_window_) { owner_window_->RemoveBackgroundThrottlingSource(this); } @@ -1065,8 +1068,6 @@ WebContents::~WebContents() { return; } - inspectable_web_contents_->GetView()->SetDelegate(nullptr); - // This event is only for internal use, which is emitted when WebContents is // being destroyed. Emit("will-destroy"); @@ -2202,8 +2203,8 @@ void WebContents::DevToolsOpened() { // Inherit owner window in devtools when it doesn't have one. auto* devtools = inspectable_web_contents_->GetDevToolsWebContents(); bool has_window = devtools->GetUserData(NativeWindowRelay::UserDataKey()); - if (owner_window_ && !has_window) { - DCHECK(!owner_window_.WasInvalidated()); + if (owner_window() && !has_window) { + CHECK(!owner_window_.WasInvalidated()); DCHECK_EQ(handle->owner_window(), nullptr); handle->SetOwnerWindow(devtools, owner_window()); }