1686294 - videocontrols.js restricts some video control behaviour based on documentURI which is bogus

Status: NEW

(Toolkit :: UI Widgets, defect, P3)

Description

[:Gijs (he/him)]

STR:

1. open https://www.w3.org/2010/05/video/mediaevents.html
2. check document.documentURI (starts with https)
3. call window.open("about:blank", "_blank", "height=500") from the same place, and check document.documentURI for that (won't start with https, I think - if not, same thing but for file URIs or data: uris or whatever)

ER:

absence/presence of controls / bindings shouldn't depend on documentURI

AR:
they do. Not sure why.

Comment 1

[:Gijs (he/him)]

This is from https://phabricator.services.mozilla.com/D26804 but I don't see a discussion there or in bug 1543128 about this restriction. Mike, do you recall why this is?

Comment 2

[Brian Grinstead [:bgrins]]

Looks like an original landing that got backed out didn't have the check https://hg.mozilla.org/integration/autoland/rev/a2e73d143aba and then the reland at https://hg.mozilla.org/integration/autoland/rev/3ac660e7f45c did.

Comment 3

[:Gijs (he/him)]

(In reply to Brian Grinstead [:bgrins] from comment #2)

Looks like an original landing that got backed out didn't have the check https://hg.mozilla.org/integration/autoland/rev/a2e73d143aba and then the reland at https://hg.mozilla.org/integration/autoland/rev/3ac660e7f45c did.

Looks like maybe related bug 1544257? But I can't tell if this was an attempt to fix it that ended up being necessary or not.

Comment 4

[Mark Striemer [:mstriemer]]

Mike, do you recall why the check Brian mentioned in comment 2 changed?

Comment 5

[Mike Conley (:mconley) (:⚙️)]

Wooo 1yr old needinfo. Sorry, gang.

I'm afraid I don't recall. :( I also can't think of a good reason why this would be necessary.