js/src/jscompartment.cpp:276 JSCompartment calls str->zone(), which requires the zone to have exclusive access. This is in fact unnecessary.

Shihua, this will need a full try run also.

(In reply to ben turner [:bent] (use the needinfo? flag!) from comment #2) > Shihua, this will need a full try run also. Try: https://tbpl.mozilla.org/?tree=Try&rev=5165684548a3

This needs backporting to branches, no? Please request approvals.

Comment on attachment 8438796 [details] [diff] [review] Patch Approval Request Comment [Feature/regressing bug #]: 600402 [User impact if declined]: Have unexpected failures when dealing with strings on web workers [Describe test coverage new/current, TBPL]: https://tbpl.mozilla.org/?tree=Try&rev=5165684548a3 , passed [Risks and why]: Low [String/UUID change made/needed]: None

Looking at bug 600402, I can see that the bug existed for the last 3 years. We are late in the beta cycle, why do you think we should take it?

Note that this blocks bug 1027221 from being uplifted as well.

(In reply to Sylvestre Ledru [:sylvestre] from comment #8) > Looking at bug 600402, I can see that the bug existed for the last 3 years. > We are late in the beta cycle, why do you think we should take it? This bug, as well as bug 1027221, blocks worker features from shipping. It will be very helpful to get them landed in beta channel early. In addition, the stability is not expected to be impaired by these patches.

https://hg.mozilla.org/releases/mozilla-aurora/rev/fad78d49fe07 https://hg.mozilla.org/releases/mozilla-beta/rev/09a209b681b9

Is this something that impacts b2g v1.4 (b2g30) as well? If so, should this be blocking?

Tagging [qa-]. Please advise if this needs QA attention before we release.

(In reply to Ryan VanderMeulen [:RyanVM UTC-4] from comment #12) > Is this something that impacts b2g v1.4 (b2g30) as well? If so, should this > be blocking? Impact, but nonblocking.