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)
Core
Networking
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.
Assignee | ||
Updated•10 months ago
|
Updated•9 months ago
|
Severity: -- → N/A
Priority: -- → P2
Whiteboard: [necko-triaged]
Assignee | ||
Comment 1•9 months ago
|
||
WPT doc mentions "Tests that need to inspect the browser-sent request" in https://web-platform-tests.org/writing-tests/server-features.html
Assignee | ||
Comment 2•8 months ago
|
||
Updated•8 months ago
|
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
Assignee | ||
Updated•8 months ago
|
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
Comment 4•8 months ago
|
||
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
status-firefox128:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 128 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•