1628730 - webrtc/simulcast/basic.https.html ini file needs bug number

Status: RESOLVED FIXED

(Core :: WebRTC: Signaling, defect, P2)

Description

[Byron Campen [:bwc]]

Looks like the relative path on the script element is not working correctly:

<script src="../third_party/sdp/sdp.js"></script>

Test is in testing/web-platform/tests/webrtc/simulcast
sdp.js is in testing/web-platform/tests/webrtc/third_party/sdp/sdp.js

Looks like a harness issue?

Comment 1

[Byron Campen [:bwc]]

Path should probably be /webrtc/third_party/sdp/sdp.js instead.

Comment 2

[Byron Campen [:bwc]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=e4aba54e2b7f234502f4d70ac0cc0ced0ed3fe27

Comment 3

[Byron Campen [:bwc]]

Attachment: Bug 1628730: Update the ini file to point at the next bug that needs fixing. r?jib

Comment 4

[Byron Campen [:bwc]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=f2be958ae84e7aa006cb3cd533e238924c5eabb7

Comment 5

[Philipp Hancke [:fippo]]

beware, this is going to result in a merge conflict since I had to add a "long" meta tag in Chrome to unflake the tests (plus there is a new getStats.https.html in the directory)
Also is it just "../third_party" that is an issue but not ../RTCPeerConnection-helper.js?

Comment 6

[Byron Campen [:bwc]]

../RTCPeerConnection-helper.js seems to work. Maybe the harness has changed, and fixed the pathing issue with sdp.js in the meantime? Let me check that.

Comment 7

[Philipp Hancke [:fippo]]

there are a couple of other tests that have been around for a while that use a similar approach, e.g. https://wpt.live/webrtc/protocol/msid-parse.html so i'd be surprised if that was a regression

Comment 8

[Byron Campen [:bwc]]

Looks like, for whatever reason, the path is not an issue anymore.

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/862e99e19379
Update the ini file to point at the next bug that needs fixing. r=jib

Comment 10

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/862e99e19379

Comment 11

[BugBot [:suhaib / :marco/ :calixte]]

The patch landed in nightly and beta is affected.
:bwc, is this bug important enough to require an uplift?
If not please set status_beta to wontfix.

For more information, please visit auto_nag documentation.