mirrors/electron
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages 1 Wiki Activity Actions

re-add warning removed in https://github.com/electron/electron/pull/11013

Browse source
This commit is contained in:
Shelley Vohr 2017-12-02 09:45:35 -05:00
parent 5fa29fcf58
commit 24f60dcd1b
No known key found for this signature in database
GPG key ID: F13993A75599653C
1 changed files with 5 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

14
lib/common/api/crash-reporter.js
View file

@ -4,7 +4,7 @@ const {spawn} = require('child_process')
const os = require('os')
const path = require('path')
const electron = require('electron')
const {app} = process.type === 'browser' ? electron : electron.remote
const {app, deprecate} = process.type === 'browser' ? electron : electron.remote
const binding = process.atomBinding('crash_reporter')
class CrashReporter {
@ -106,14 +106,10 @@ class CrashReporter {
// TODO(2.0) Remove
setExtraParameter (key, value) {
// TODO(alexeykuzmin): Warning disabled since it caused
// a couple of Crash Reported tests to time out on Mac. Add it back.
// https://github.com/electron/electron/issues/11012
// if (!process.noDeprecations) {
// deprecate.warn('crashReporter.setExtraParameter',
// 'crashReporter.addExtraParameter or crashReporter.removeExtraParameter')
// }
if (!process.noDeprecations) {
deprecate.warn('crashReporter.setExtraParameter',
'crashReporter.addExtraParameter or crashReporter.removeExtraParameter')
}
binding.setExtraParameter(key, value)
}