Closed Bug 1216665 Opened 7 years ago Closed 7 years ago

Disable asyncInit in 42

Categories

(Core Graveyard :: Plug-ins, defect)

defect
Not set
normal

Tracking

(firefox42 fixed)

RESOLVED FIXED
Tracking Status
firefox42 --- fixed

People

(Reporter: bugzilla, Assigned: bugzilla)

Details

Attachments

(1 file)

Given the Window neutering churn in 42, I am hesitant to ship 42 and asyncInit in one go. I think that it is safer to release with the updated neutering code in 42 and follow that up with full asyncInit in 43.
Comment on attachment 8676392 [details] [diff] [review]
Patch

doing r=me given it's just a pref change

Approval Request Comment
[Feature/regressing bug #]: plugin async init
[User impact if declined]: unknown side effects of neutering changes + async init
[Describe test coverage new/current, TreeHerder]: Plugin test suite
[Risks and why]: Low, was already disabled on Nightly and Aurora, and disabled in 41 release
[String/UUID change made/needed]: None
Attachment #8676392 - Flags: review+
Attachment #8676392 - Flags: approval-mozilla-beta?
Comment on attachment 8676392 [details] [diff] [review]
Patch

Sure, should be in 42 beta 9!
Thanks again for all your help and work on this feature!
Attachment #8676392 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
https://hg.mozilla.org/releases/mozilla-beta/rev/ad9b70fef588
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.