From 8f63fdb278ccefa4847748637ec69ee04c887ba7 Mon Sep 17 00:00:00 2001 From: Kevin Sawicki Date: Tue, 10 Jan 2017 15:40:07 -0800 Subject: [PATCH] Destroy web contents when preventing default --- atom/browser/api/atom_api_web_contents.cc | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/atom/browser/api/atom_api_web_contents.cc b/atom/browser/api/atom_api_web_contents.cc index a3753a6d8f05..af64d9017c4a 100644 --- a/atom/browser/api/atom_api_web_contents.cc +++ b/atom/browser/api/atom_api_web_contents.cc @@ -419,9 +419,11 @@ void WebContents::AddNewContents(content::WebContents* source, v8::Locker locker(isolate()); v8::HandleScope handle_scope(isolate()); auto api_web_contents = CreateFrom(isolate(), new_contents); - Emit("-add-new-contents", api_web_contents, disposition, user_gesture, + if (Emit("-add-new-contents", api_web_contents, disposition, user_gesture, initial_rect.x(), initial_rect.y(), initial_rect.width(), - initial_rect.height()); + initial_rect.height())) { + api_web_contents->DestroyWebContents(); + } } content::WebContents* WebContents::OpenURLFromTab(