Closed Bug 1102634 Opened 10 years ago Closed 9 years ago

[e10s] Add a pref for add-on shims

Categories

(Firefox :: Extension Compatibility, defect)

34 Branch
x86_64
Linux
defect
Not set
normal

Tracking

()

RESOLVED FIXED
Firefox 36

People

(Reporter: billm, Assigned: billm)

References

Details

Attachments

(1 file)

There was a perf regression from this code reported recently. It would be nice to be able to pref it off to test its impact, so let's add one.
Attachment #8526437 - Flags: review?(ally)
Comment on attachment 8526437 [details] [diff] [review]
interposition-pref

Review of attachment 8526437 [details] [diff] [review]:
-----------------------------------------------------------------

all that work for HAVE_SHIMS, so it goes. Let's see what we can shake out in testing.

Do we plan on reverting this patch if that doesn't plan out for perf?
Attachment #8526437 - Flags: review?(ally) → review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/5b1b37605f2e

I doubt we'll turn off the shims, but the pref will allow us to tell whether they're the cause. Hopefully we'll be able to fix the problem.
https://hg.mozilla.org/mozilla-central/rev/5b1b37605f2e
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 36
You need to log in before you can comment on or make changes to this bug.