1395262 - replace e => console.error(e) with console.error

Status: RESOLVED FIXED

(DevTools :: Framework, enhancement)

Description

[Tom Tromey :tromey]

We decided to do the reverse of bug 1234620: replace
e => console.error(e) with plain console.error.

Normally this wouldn't be worth doing, but a quick inspection showed
some invalid code like  .catch(e => console.error), which isn't useful at all.
Also, it's trivial to fix the normal uses of this pattern.

Comment 1

[Tom Tromey :tromey]

Attachment: Bug 1395262 - use plain console.error in devtools promise catches;

Review commit: https://reviewboard.mozilla.org/r/174468/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/174468/

Comment 2

[Brian Grinstead [:bgrins]]

Comment on attachment 8902798 [details]
Bug 1395262 - use plain console.error in devtools promise catches;

https://reviewboard.mozilla.org/r/174468/#review179634

Thank you! Worth a try push before landing, but this is nice

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/18d3376156f0
use plain console.error in devtools promise catches; r=bgrins

Comment 4

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/18d3376156f0