chore: address TODO for app.makeSingleInstance / releaseSingleInstance removal

lib/browser/api/app.js

@@ -107,21 +107,6 @@ for (let name of events) {
   })
 }
 
-// TODO(MarshallOfSound): Remove in 4.0
-app.releaseSingleInstance = () => {
-  deprecate.warn('app.releaseSingleInstance(cb)', 'app.releaseSingleInstanceLock()')
-  app.releaseSingleInstanceLock()
-}
-
-// TODO(MarshallOfSound): Remove in 4.0
-app.makeSingleInstance = (oldStyleFn) => {
-  deprecate.warn('app.makeSingleInstance(cb)', 'app.requestSingleInstanceLock() and app.on(\'second-instance\', cb)')
-  if (oldStyleFn && typeof oldStyleFn === 'function') {
-    app.on('second-instance', (event, ...args) => oldStyleFn(...args))
-  }
-  return !app.requestSingleInstanceLock()
-}
-
 // Wrappers for native classes.
 const {DownloadItem} = process.atomBinding('download_item')
 Object.setPrototypeOf(DownloadItem.prototype, EventEmitter.prototype)

spec/api-app-spec.js

@@ -202,28 +202,6 @@ describe('app module', () => {
     })
   })
 
-  // TODO(MarshallOfSound) - Remove in 4.0.0
-  describe('app.makeSingleInstance', () => {
-    it('prevents the second launch of app', function (done) {
-      this.timeout(120000)
-      const appPath = path.join(__dirname, 'fixtures', 'api', 'singleton-old')
-      // First launch should exit with 0.
-      const first = ChildProcess.spawn(remote.process.execPath, [appPath])
-      first.once('exit', code => {
-        expect(code).to.equal(0)
-      })
-      // Start second app when received output.
-      first.stdout.once('data', () => {
-        // Second launch should exit with 1.
-        const second = ChildProcess.spawn(remote.process.execPath, [appPath])
-        second.once('exit', code => {
-          expect(code).to.equal(1)
-          done()
-        })
-      })
-    })
-  })
-
   describe('app.requestSingleInstanceLock', () => {
     it('prevents the second launch of app', function (done) {
       this.timeout(120000)