Closed Bug 1653082 Opened 5 years ago Closed 5 years ago

[wpt-sync] Sync PR 24615 - Fix path in subresource.py

Categories

(Testing :: web-platform-tests, task, P4)

Product:
Testing
Component:
web-platform-tests
Type:
task

Tracking

(firefox80 fixed)

Status:
RESOLVED FIXED
Milestone:
mozilla80
Tracking Flags:
Tracking Status
firefox80 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

(
URL
)

Details

(Whiteboard: [wptsync downstream])

Sync web-platform-tests PR 24615 into mozilla-central (this bug is closed when the sync is complete).

PR: https://github.com/web-platform-tests/wpt/pull/24615
Details from upstream follow.

Robert Ma <robertma@chromium.org> wrote:

Fix path in subresource.py

The change was introduced in #24481, which assumed request.GET.get would
return a single value (though not documented explicitly, it's a very
reasonable assumption), but unfortunately it actually returns a list.
This caused widespread timeout in many referrer-policy tests, which were
not caught as affected tests.

This change fixes the immediate issue. #24614 will fix MultiDict to match
the intuition.

PR 24615 applied with additional changes from upstream: 3c8965b973ca10f3cf5a6a817c87e32939e4cb6e
Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/9f586265d0c2 [wpt PR 24615] - Fix path in subresource.py, a=testonly
Whiteboard: [wptsync downstream] → [wptsync upstream]

https://hg.mozilla.org/mozilla-central/rev/9f586265d0c2

Status: NEW → RESOLVED
Closed: 5 years ago
status-firefox80: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla80
Whiteboard: [wptsync upstream] → [wptsync downstream]
Test result changes from PR not available.
You need to log in before you can comment on or make changes to this bug.