Closed Bug 1892734 Opened 10 months ago Closed 8 months ago

Add tests for checking the priority set in our HTTP requests

Categories

(Core :: Networking, task, P2)

task

Tracking

()

RESOLVED FIXED
128 Branch
Tracking Status
firefox128 --- fixed

People

(Reporter: fredw, Assigned: fredw)

References

(Blocks 1 open bug)

Details

(Whiteboard: [necko-triaged])

Attachments

(1 file)

Currently, we have fetchpriority internal WPT tests in

https://searchfox.org/mozilla-central/source/testing/web-platform/mozilla/tests/fetch/fetchpriority/

which uses by nsISupportsPriority to checks the internal priority we set when loading HTTP/2 resources.

We could do more direct testing of our priority set in our HTTP requests.

Probably don't need to involve fetchpriority or be exhaustive, but just check the default behavior for a few cases.

Blocks: 1797715, 1614461
Severity: -- → N/A
Priority: -- → P2
Whiteboard: [necko-triaged]

WPT doc mentions "Tests that need to inspect the browser-sent request" in https://web-platform-tests.org/writing-tests/server-features.html

Depends on: 1864392
Assignee: nobody → fwang
Attachment #9404557 - Attachment description: WIP: Bug 1892734 - Add tests for checking the priority set in our HTTP requests. r=valentin → Bug 1892734 - Add tests for checking the priority set in our HTTP requests. r=valentin
Status: NEW → ASSIGNED
Blocks: 1881040, 1882084
Blocks: 1900764
Pushed by fwang@igalia.com: https://hg.mozilla.org/integration/autoland/rev/b1240a12bbe1 Add tests for checking the priority set in our HTTP requests. r=valentin

Backed out for causing leak related wpt failures

Backout link

Push with failures

Failure log

Flags: needinfo?(fwang)

This is bug 1759310.

Flags: needinfo?(fwang)
Pushed by fwang@igalia.com: https://hg.mozilla.org/integration/autoland/rev/55e360930548 Add tests for checking the priority set in our HTTP requests. r=valentin
Status: ASSIGNED → RESOLVED
Closed: 8 months ago
Resolution: --- → FIXED
Target Milestone: --- → 128 Branch
Regressions: 1901220
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: