From 559b2157d44449284198c69ff127008b82c2a83b Mon Sep 17 00:00:00 2001
From: Scott Nonnenberg <scott@signal.org>
Date: Fri, 18 May 2018 18:21:02 -0700
Subject: [PATCH] Disable window.open(); has extra Electron functionalnity

(that we never use)

https://electronjs.org/docs/api/browser-window-proxy
---
 preload.js | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/preload.js b/preload.js
index b884864db..ec425fb06 100644
--- a/preload.js
+++ b/preload.js
@@ -19,6 +19,9 @@ window.config.localeMessages = ipc.sendSync('locale-data');
 
 window.setBadgeCount = count => ipc.send('set-badge-count', count);
 
+// We never do this in our code, so we'll prevent it everywhere
+window.open = () => null;
+
 window.drawAttention = () => {
   console.log('draw attention');
   ipc.send('draw-attention');