Opened on 01/07/2016 at 07:49:21 PM
Closed on 01/08/2016 at 01:40:42 PM
#3490 closed change (fixed)
Make multiprocessCompatible the default for our extensions
Reported by: | trev | Assignee: | trev |
---|---|---|---|
Priority: | P3 | Milestone: | |
Module: | Automation | Keywords: | |
Cc: | Blocked By: | ||
Blocking: | Platform: | Unknown / Cross platform | |
Ready: | yes | Confidential: | no |
Tester: | Unknown | Verified working: | no |
Review URL(s): |
Description
Background
The last E10S issues in Adblock Plus are under review. Next step would be adding <em:multiprocessCompatible> to install.rdf to mark it as fully compatible with E10S and disable compatibility shims which are only getting in the way.
What to change
Make <em:multiprocessCompatible> the new default and introduce needMultiprocessShims to metadata.gecko for the extensions still relying on shims (like Element Hiding Helper). While this approach will make upgrading somewhat more complicated as opposed to keeping the shims enabled by default, I think that most of our extensions are already E10S-compatible (Diagnostics and Customizations don't touch content) and we won't have to revert the default later on.
Attachments (0)
Change History (4)
comment:1 Changed on 01/07/2016 at 08:02:26 PM by trev
- Owner set to trev
comment:2 Changed on 01/07/2016 at 08:02:36 PM by trev
- Review URL(s) modified (diff)
- Status changed from new to reviewing
comment:3 Changed on 01/07/2016 at 09:42:02 PM by trev
- Priority changed from Unknown to P3
- Ready set
comment:4 Changed on 01/08/2016 at 01:40:42 PM by trev
- Resolution set to fixed
- Status changed from reviewing to closed
Fixed: https://hg.adblockplus.org/buildtools/rev/4d73b9c76e34