1426520 - Run webpack with --display-optimization-bailout

Status: RESOLVED FIXED

(Firefox :: New Tab Page, enhancement, P2)

Description

[Ed Lee :Mardak]

Followup comments from dmose from https://github.com/mozilla/activity-stream/pull/3910

reading https://medium.com/webpack/webpack-freelancing-log-book-week-5-7-4764be3266f5 has a bunch of interseting info, including one pattern that will generate bad code unless we disable part of the optimization.  that's in the "other changes required" section.
"Don’t use this in exported functions, if you don’t .call them explicitly."
hmmm, maybe that's outdated, and that the optimization bailout code will fire instead of generating bad code: https://webpack.js.org/plugins/module-concatenation-plugin/#optimization-bailouts
also, eyeballing the generated code, some concatention is clearly going on, so that's a good thing

Comment 1

[[github robot]]

Commit pushed to master at https://github.com/mozilla/activity-stream

https://github.com/mozilla/activity-stream/commit/1374b5f26ff1e5931bd9d1e4ffc2ec293e2b37f4
feat(webpack): Add --display-optimization-bailout and use jsm-to-esmodules (#3932)

Fix Bug 1426520 - Run webpack with --display-optimization-bailout

Comment 2

[Ed Lee :Mardak]

https://hg.mozilla.org/mozilla-central/rev/0d163e2c4792