Closed Bug 1428176 Opened 2 years ago Closed 2 years ago
_startup .js on Dev Edition builds
Per bug 1410072 comment 8 and 9, we need to enable recording stacks in mozJSComponentLoader.cpp for dev edition builds if we want to enable browser_startup.js there.
Changing the ifdef means this will be compiled in, but it'll cause little overhead unless the pref is enabled, only a Preferences::GetBool("browser.startup.record", false) call.
Attachment #8939966 - Flags: review?(continuation)
Ryan, could you please verify that this works as expected on a 'real' devedition build?
Attachment #8939966 - Flags: review?(continuation) → review+
Made a couple changes per IRC discussion with Andrew and Florian, but this version is confirmed working (with logs confirming the test operating as expected) :)
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/mozilla-inbound/rev/a6c8d268c35e Enable browser_startup.js on DevEdition builds. r=mccr8, r=florian
You need to log in before you can comment on or make changes to this bug.