Closed Bug 1312880 Opened 4 years ago Closed 4 years ago
Process Data for Remote Addons Child rather than a sync message
The sync message happens here: http://searchfox.org/mozilla-central/rev/e3e8571c5378ac92663d4f583ccc4ad0a3019716/toolkit/components/addoncompat/RemoteAddonsChild.jsm#52 We should be able to replace that using initialProcessData.
Comment on attachment 8805544 [details] Bug 1312880 - Remove sync message when starting up RemoteAddonsChild. https://reviewboard.mozilla.org/r/89314/#review88650
Attachment #8805544 - Flags: review?(wmccloskey) → review+
Pushed by email@example.com: https://hg.mozilla.org/integration/autoland/rev/8b1ef926af1c Remove sync message when starting up RemoteAddonsChild. r=billm
Comment on attachment 8805544 [details] Bug 1312880 - Remove sync message when starting up RemoteAddonsChild. Approval Request Comment [Feature/regressing bug #]: Add-on shims and e10s. [User impact if declined]: Slightly worse performance on tab switch / content process performance when a new tab or window opens. [Describe test coverage new/current, TreeHerder]: The add-on shim tests still work, and this has been on Nightly for a few days. The dashboard that chutten set up at https://chutten.github.io/bug1310250/ shows the win from this fix appearing in the Oct 30th Nightly. [Risks and why]: Very low risk. [String/UUID change made/needed]: None.
Hi Brad, Jim, at this point in RC 50 we are in Code Freeze mode and only taking fixes for release blocking issues. Is this something that would lead to a dot release once 50 goes live? If not, I'd prefer to take it in 50.1.0 (if deemed necessary) or let it ride the 51 release train. Do you anticipate this fix to disrupt the 50 crash rates? It has taken us a few weeks to get to a ship-worthy state from stability pov and I am wondering if this fix is safe from that aspect.
IMO we should take this for 51 but not 50. This should change timings and there are just too many timing interactions to be certain that this is risk free.
Comment on attachment 8805544 [details] Bug 1312880 - Remove sync message when starting up RemoteAddonsChild. Let's uplift to Aurora51. This is a no-go for 50.
You need to log in before you can comment on or make changes to this bug.