Use initialProcessData for RemoteAddonsChild rather than a sync message

RESOLVED FIXED in Firefox 51

Status

()

defect
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: billm, Assigned: mconley)

Tracking

unspecified
mozilla52
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox50 wontfix, firefox51 fixed, firefox52 fixed)

Details

Attachments

(1 attachment)

Assignee: nobody → mconley
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 mconley@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/8b1ef926af1c
Remove sync message when starting up RemoteAddonsChild. r=billm
https://hg.mozilla.org/mozilla-central/rev/8b1ef926af1c
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla52
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.
Attachment #8805544 - Flags: approval-mozilla-beta?
Attachment #8805544 - Flags: approval-mozilla-aurora?
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.
Flags: needinfo?(jmathies)
Flags: needinfo?(blassey.bugs)
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.
Flags: needinfo?(blassey.bugs)
Thanks Brad! We'll let this one ride the 51 release train.
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.
Attachment #8805544 - Flags: approval-mozilla-beta?
Attachment #8805544 - Flags: approval-mozilla-beta-
Attachment #8805544 - Flags: approval-mozilla-aurora?
Attachment #8805544 - Flags: approval-mozilla-aurora+
Flags: needinfo?(jmathies)
You need to log in before you can comment on or make changes to this bug.