Closed Bug 1676956 Opened 1 year ago Closed 10 months ago

webRequest streamfilter tests are run with in-process extensions only, not oop extensions

Categories

(WebExtensions :: Request Handling, task, P2)

task

Tracking

(firefox85 fixed, firefox86 fixed)

RESOLVED FIXED
86 Branch
Tracking Status
firefox85 --- fixed
firefox86 --- fixed

People

(Reporter: robwu, Assigned: robwu)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

In bug 1649873, the StreamFilter tests were moved from xpcshell-common.ini to xpcshell-remote.ini, because the tests weren't working when e10s is off.

xpcshell-remote.ini is e10s on, OOP webextensions off.
We should run the tests when e10s is on, to more closely match the real-world situation. This can be done by moving the logic to xpcshell-remote.ini.

If we care about running the tests with both oop extensions and in-process extensions (e.g. because we still use in-process extensions on Android until bug 1535365 is fixed), then we have the following options:

  • Repeat the tests in xpcshell-remote.ini and xpcshell-e10s.ini
  • Add the test to xpcshell-common.ini, but add a skip_if to the add_task in the individual test files.
  • Introduce a new test manifest (.ini file) that gets included by both xpcshell-e10s.ini and xpcshell-remote.ini (this is basically the same as xpcshell-common.ini, except not being included by xpcshell.ini).
Component: General → Request Handling

We just discussed this, and are considering the second option, i.e. moving the tests back to xpcshell-common.ini and then using skip_if in the files themselves. The downside of that is that it is not obvious that the tests are skipped (when looking at test manifests), but the current state of affairs, where tests are not run with our primary supported configuration is much worse.

Assignee: nobody → rob
Severity: -- → S3
Status: NEW → ASSIGNED
Priority: -- → P2
Pushed by rob@robwu.nl:
https://hg.mozilla.org/integration/autoland/rev/5e7653247208
Run StreamFilter tests with oop extensions too r=mixedpuppy
Blocks: 1683159

Going to reland, the failure from comment 4 will be addressed in bug 1683159.

Flags: needinfo?(rob)
Pushed by rob@robwu.nl:
https://hg.mozilla.org/integration/autoland/rev/557c3876e77d
Run StreamFilter tests with oop extensions too r=mixedpuppy
Status: ASSIGNED → RESOLVED
Closed: 10 months ago
Resolution: --- → FIXED
Target Milestone: --- → 86 Branch
Blocks: 1678734

Comment on attachment 9191352 [details]
Bug 1676956 - Run StreamFilter tests with oop extensions too

Beta/Release Uplift Approval Request

  • User impact if declined: test only changes necessary to uplift bug 1678734
  • Is this code covered by automated tests?: Yes
  • Has the fix been verified in Nightly?: Yes
  • Needs manual test from QE?: No
  • If yes, steps to reproduce: n/a
  • List of other uplifts needed: Bug 1678734
  • Risk to taking this patch: Low
  • Why is the change risky/not risky? (and alternatives if risky): test only changes
  • String changes made/needed: n/a
Attachment #9191352 - Flags: approval-mozilla-beta?

Comment on attachment 9191352 [details]
Bug 1676956 - Run StreamFilter tests with oop extensions too

a=test-only

Attachment #9191352 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
You need to log in before you can comment on or make changes to this bug.