Closed Bug 1216665 Opened 4 years ago Closed 4 years ago

Disable asyncInit in 42

Categories

(Core :: Plug-ins, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
Tracking Status
firefox42 --- fixed

People

(Reporter: aklotz, Assigned: aklotz)

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.
Attached patch PatchSplinter Review
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: 4 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.