Merge pull request #7869 from electron/less-browserify-build-time

Only run browserify when sandbox files change

electron.gyp

@@ -436,10 +436,6 @@
           'action_name': 'atom_browserify',
           'inputs': [
             '<@(browserify_entries)',
-            # Any js file under `lib/` can be included in the preload bundle.
-            # Add all js sources as dependencies so any change to a js file will
-            # trigger a rebuild of the bundle(and consequently of js2c).
-            '<@(js_sources)',
           ],
           'outputs': [
             '<(js2c_input_dir)/preload_bundle.js',
@@ -447,9 +443,10 @@
           'action': [
             'npm',
             'run',
+            '--silent',
             'browserify',
             '--',
-            '<@(browserify_entries)',
+            'lib/sandboxed_renderer/init.js',
             '-o',
             '<@(_outputs)',
           ],

filenames.gypi

@@ -73,6 +73,7 @@
       'lib/renderer/extensions/web-navigation.js',
     ],
     'browserify_entries': [
+      'lib/renderer/api/ipc-renderer-setup.js',
       'lib/sandboxed_renderer/init.js',
     ],
     'js2c_sources': [

lib/renderer/api/ipc-renderer-setup.js

@@ -1,3 +1,6 @@
+// Any requires added here need to be added to the browserify_entries array
+// in filenames.gypi so they get built into the preload_bundle.js bundle
+
 module.exports = function (ipcRenderer, binding) {
   ipcRenderer.send = function (...args) {
     return binding.send('ipc-message', args)

lib/sandboxed_renderer/init.js

@@ -1,3 +1,6 @@
+// Any requires added here need to be added to the browserify_entries array
+// in filenames.gypi so they get built into the preload_bundle.js bundle
+
 /* eslint no-eval: "off" */
 /* global binding, preloadPath, process, Buffer */
 const events = require('events')