1795783 - Replace custom {same, cross}-origin settings by get_host_info

Status: RESOLVED FIXED

(Core :: Audio/Video: Web Codecs, enhancement)

Description

[C.M.Chang[:chunmin]]

videoFrame-construction.crossOriginSource.sub.html has tests to check if the images can be loaded from same-origin and cross-origin sites. Now the {same, cross}-origin are set by using wptserve's magic directly here:

const SAMEORIGIN_BASE = '{{location[scheme]}}://{{location[host]}}';
const CROSSORIGIN_BASE = 'http://{{hosts[][www]}}:{{ports[http][0]}}';

However, the idiom is to using get_host_info() to get the domains, like this

Comment 1

[C.M.Chang[:chunmin]]

Attachment: Bug 1795783 - Replace custom {same, cross}-origin settings by get_host_info

Comment 2

[Pascal Chevrel:pascalc]

You set this bug as P1/S1 and requested tracking for all releases in flight. This is the signal for a release blocker (and we ship 106 today), yet the patch attached to this bug only changes a test file. Are the flags correctly set?

Comment 3

[C.M.Chang[:chunmin]]

(In reply to Pascal Chevrel:pascalc from comment #2)

You set this bug as P1/S1 and requested tracking for all releases in flight. This is the signal for a release blocker (and we ship 106 today), yet the patch attached to this bug only changes a test file. Are the flags correctly set?

It should be S4

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/9738efe9d3ad
Replace custom {same, cross}-origin settings by get_host_info r=padenot

Comment 5

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/36533 for changes under testing/web-platform/tests

Comment 6

[Sandor Molnar[:smolnar]]

https://hg.mozilla.org/mozilla-central/rev/9738efe9d3ad

Comment 7

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Upstream PR merged by jgraham