1781759 - cross-origin Mp4 Videos not played when loaded as document in iFrame

Status: VERIFIED FIXED

(Core :: Audio/Video: Playback, defect, P1)

Description

[jonathan.fuchs]

User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Firefox/102.0

Steps to reproduce:

Embedded a video in an iFrame. Format is mp4. If I play the video with a normal video tag it works fine but not if its inside an iFrame. See below

https://jsfiddle.net/jonny_fux/d2b6c5af/11/

Actual results:

Error: No Video with supported format and MIME type found

Expected results:

The video should be played. That works in Safari and Chromium Browsers.

Comment 1

[Alice0775 White]

Regression window:
https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=e15164d8b4c6de1bbb016649e32bd63797a7453a&tochange=56757e00ffb4fbea5c67b67d86d081949065dd6d

Comment 2

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

:karlt, since you are the author of the regressor, bug 1735923, could you take a look?
For more information, please visit auto_nag documentation.

Comment 3

[Karl Tomlinson (:karlt back Feb 10)]

Using "Select an iframe as the currently targeted document" in the Web Developer Tools Console, document.getElementsByTagName("video")[0].error.message is "NS_ERROR_CONTENT_BLOCKED (0x805e0006) - opaque and non-opaque responses".

Comment 4

[Karl Tomlinson (:karlt back Feb 10)]

https://pernos.co/debug/d5WGdM2t5qWmAYj9BsVzyw/index.html
https://treeherder.mozilla.org/jobs?repo=try&revision=84fa975973374ebba2e1a61a33fa7920b6abcf19

Comment 5

[Karl Tomlinson (:karlt back Feb 10)]

Attachment: Bug 1781759 factor out ChannelMediaResource::MediaElement() helper r?chunmin

Comment 6

[Karl Tomlinson (:karlt back Feb 10)]

Attachment: Bug 1781759 use media element node principal instead of channel loading principal for cross-origin resource test r?edenchuang

When the media resource is loaded as a document, the response from the initial
document load gets reused, as an optimization, as an emulated load for the
resource of the media host element in the generated HTML document.
https://searchfox.org/mozilla-central/rev/5644fae86d5122519a0e34ee03117c88c6ed9b47/dom/html/VideoDocument.cpp#114
https://html.spec.whatwg.org/multipage/browsing-the-web.html#read-media
The loadingPrincipal of the initial channel comes from the parent document,
and so is not useful to determine whether the response is cross-origin with
the media element.

This change makes the cross-origin test on the initial document load
consistent with subsequent requests on the same resource and non-document
loads, which already use the node principal for the loadingPrincipal.
https://searchfox.org/mozilla-central/rev/1061fae5e225a99ef5e43dbdf560a91a0c0d00d1/dom/media/ChannelMediaResource.cpp#750
https://searchfox.org/mozilla-central/rev/4f2984be127d2e7c788cf1848d63dca63022beec/dom/html/HTMLMediaElement.cpp#1708

Depends on D154039

Comment 7

[Karl Tomlinson (:karlt back Feb 10)]

Attachment: Bug 1781759 Test cross origin load of media document in parts r?chunmin

Depends on D154040

Comment 9

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

The severity field for this bug is relatively low, S3. However, the bug has 5 See Also bugs.
:karlt, could you consider increasing the bug severity?

For more information, please visit auto_nag documentation.

Comment 10

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/4f56c42c36e7
factor out ChannelMediaResource::MediaElement() helper r=chunmin
https://hg.mozilla.org/integration/autoland/rev/69937c47da4b
use media element node principal instead of channel loading principal for cross-origin resource test r=padenot
https://hg.mozilla.org/integration/autoland/rev/b820ae7bac8d
Test cross origin load of media document in parts r=chunmin

Comment 11

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

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

Comment 12

[Cristian Tuns]

https://hg.mozilla.org/mozilla-central/rev/4f56c42c36e7
https://hg.mozilla.org/mozilla-central/rev/69937c47da4b
https://hg.mozilla.org/mozilla-central/rev/b820ae7bac8d

Comment 13

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

Upstream PR merged by moz-wptsync-bot

Comment 14

[Karl Tomlinson (:karlt back Feb 10)]

Comment on attachment 9288955 [details]
Bug 1781759 use media element node principal instead of channel loading principal for cross-origin resource test r?edenchuang

Beta/Release Uplift Approval Request

• User impact if declined: Some videos or audio do not play sometimes or always when loaded into a cross-origin iframe.
• Is this code covered by automated tests?: Yes
• Has the fix been verified in Nightly?: No
• Needs manual test from QE?: No
• If yes, steps to reproduce:
• List of other uplifts needed: None
• Risk to taking this patch: Low
• Why is the change risky/not risky? (and alternatives if risky): The change is small and affects only media loaded as a document, which is currently often refusing to play.
• String changes made/needed: None.
• Is Android affected?: Yes

Comment 16

[Ryan VanderMeulen [:RyanVM]]

Comment on attachment 9288954 [details]
Bug 1781759 factor out ChannelMediaResource::MediaElement() helper r?chunmin

Approved for 105.0b3.

Comment 17

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-beta/rev/2b26aea96e2e
https://hg.mozilla.org/releases/mozilla-beta/rev/90c539587e43
https://hg.mozilla.org/releases/mozilla-beta/rev/682edecc6385

Comment 19

[Karl Tomlinson (:karlt back Feb 10)]

[Tracking Requested - why for this release]:
Regression on 102 branch.

ESR91 is affected but https://fx-trains.herokuapp.com/release/?version=esr says there are no more ESR91 releases to come.

Comment 20

[Karl Tomlinson (:karlt back Feb 10)]

Comment on attachment 9288955 [details]
Bug 1781759 use media element node principal instead of channel loading principal for cross-origin resource test r?edenchuang

ESR Uplift Approval Request

• If this is not a sec:{high,crit} bug, please state case for ESR consideration: This fixes a regression introduced in 102.
• User impact if declined: Some videos or audio do not play sometimes or always when loaded into a cross-origin iframe.
• Fix Landed on Version: 106 and uplifted to 105
• Risk to taking this patch: Low
• Why is the change risky/not risky? (and alternatives if risky): The change is small and affects only media loaded as a document, which is currently often refusing to play.

Comment 21

[Karl Tomlinson (:karlt back Feb 10)]

Comment on attachment 9288955 [details]
Bug 1781759 use media element node principal instead of channel loading principal for cross-origin resource test r?edenchuang

Beta/Release Uplift Approval Request

• User impact if declined: Most videos or audio do not play sometimes or always when loaded into a cross-origin iframe.
• Is this code covered by automated tests?: Yes
• Has the fix been verified in Nightly?: No
• Needs manual test from QE?: No
• If yes, steps to reproduce:
• List of other uplifts needed: None
• Risk to taking this patch: Low
• Why is the change risky/not risky? (and alternatives if risky): The change is small and affects only media loaded as a document, which is currently often refusing to play.
• String changes made/needed: None.
• Is Android affected?: Yes

Comment 22

[Ardelean Oana, Desktop Test Engineering [:oardelean]]

Hello,
I managed to reproduce this issue on FIrefox 104.0(build ID: 20220818191623) on Ubuntu 22.04, using the STR and URL from the Description. Verified as fixed on FIrefox 105.0b4(build ID: 20220828190315) and Nightly 106.0a1(build ID: 20220828210513) on Ubuntu 22.04, macOS 12, WIndows 10.

Comment 23

[Ryan VanderMeulen [:RyanVM]]

Comment on attachment 9288954 [details]
Bug 1781759 factor out ChannelMediaResource::MediaElement() helper r?chunmin

Approved for 102.3esr.

Comment 24

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-esr102/rev/12a6e769cb41
https://hg.mozilla.org/releases/mozilla-esr102/rev/e73a7cfc006e
https://hg.mozilla.org/releases/mozilla-esr102/rev/075e4be3af87

Comment 25

[Ardelean Oana, Desktop Test Engineering [:oardelean]]

Hello,
I confirm this fix is verified on Firefox 102.3.0esr(build ID: 20220829144321) on macOS 12, Ubuntu 22.04, Windows 10.

Comment 26

[Dianna Smith [:diannaS]]

Comment on attachment 9288954 [details]
Bug 1781759 factor out ChannelMediaResource::MediaElement() helper r?chunmin

Approved for 104.0.2

Comment 27

[Dianna Smith [:diannaS]]

https://hg.mozilla.org/releases/mozilla-release/rev/cbc68df8cd36
https://hg.mozilla.org/releases/mozilla-release/rev/617bb429e869
https://hg.mozilla.org/releases/mozilla-release/rev/96ceaebfc21c

Comment 28

[Peter Magyari (Desktop QA)]

I have verified the fix on Firefox 104.0.2 (20220901135416)