fix: do not expose Node's external strings (#13452)

spec/chromium-spec.js

@@ -70,6 +70,15 @@ describe('chromium feature', () => {
     })
   })
 
+  describe('accessing key names also used as Node.js module names', () => {
+    it('does not crash', (done) => {
+      w = new BrowserWindow({show: false})
+      w.webContents.once('did-finish-load', () => { done() })
+      w.webContents.once('crashed', () => done(new Error('WebContents crashed.')))
+      w.loadURL(`file://${fixtures}/pages/external-string.html`)
+    })
+  })
+
   describe('navigator.webkitGetUserMedia', () => {
     it('calls its callbacks', (done) => {
       navigator.webkitGetUserMedia({

spec/fixtures/pages/external-string.html

@@ -0,0 +1 @@
+<script>window.stream</script>

vendor/node

@@ -1 +1 @@
-Subproject commit 18aeda2077e7fe1ef9d7557692dda2e4d6a61860
+Subproject commit e61694e449e62e36dfe243d266e6313a0a043d5c