diff --git a/atom/browser/api/atom_api_web_contents.cc b/atom/browser/api/atom_api_web_contents.cc index 09c220c6e1..a45e166c92 100644 --- a/atom/browser/api/atom_api_web_contents.cc +++ b/atom/browser/api/atom_api_web_contents.cc @@ -439,6 +439,11 @@ content::WebContents* WebContents::OpenURLFromTab( if (Emit("will-navigate", params.url)) return nullptr; + // Don't load the URL if the web contents was marked as destroyed from a + // will-navigate event listener + if (IsDestroyed()) + return nullptr; + return CommonWebContentsDelegate::OpenURLFromTab(source, params); } diff --git a/atom/browser/api/trackable_object.h b/atom/browser/api/trackable_object.h index 3f04783bcd..7ec5445788 100644 --- a/atom/browser/api/trackable_object.h +++ b/atom/browser/api/trackable_object.h @@ -65,6 +65,12 @@ class TrackableObject : public TrackableObjectBase, Wrappable::GetWrapper()->SetAlignedPointerInInternalField(0, nullptr); } + bool IsDestroyed() { + v8::Local wrapper = Wrappable::GetWrapper(); + return wrapper->InternalFieldCount() == 0 || + wrapper->GetAlignedPointerFromInternalField(0) == nullptr; + } + // Finds out the TrackableObject from its ID in weak map. static T* FromWeakMapID(v8::Isolate* isolate, int32_t id) { if (!weak_map_) diff --git a/spec/api-browser-window-spec.js b/spec/api-browser-window-spec.js index b718a6aa49..eabd9209e8 100644 --- a/spec/api-browser-window-spec.js +++ b/spec/api-browser-window-spec.js @@ -267,6 +267,16 @@ describe('BrowserWindow module', function () { }) }) + describe('will-navigate event', function () { + it('allows the window to be closed from the event listener', (done) => { + ipcRenderer.send('close-on-will-navigate', w.id) + ipcRenderer.once('closed-on-will-navigate', () => { + done() + }) + w.loadURL('file://' + fixtures + '/pages/will-navigate.html') + }) + }) + describe('BrowserWindow.show()', function () { if (isCI) { return diff --git a/spec/static/main.js b/spec/static/main.js index 19b0739930..259350f959 100644 --- a/spec/static/main.js +++ b/spec/static/main.js @@ -221,3 +221,12 @@ ipcMain.on('set-client-certificate-option', function (event, skip) { }) event.returnValue = 'done' }) + +ipcMain.on('close-on-will-navigate', (event, id) => { + const contents = event.sender + const window = BrowserWindow.fromId(id) + window.webContents.once('will-navigate', (event, input) => { + window.close() + contents.send('closed-on-will-navigate') + }) +})