1763801 - Add non-subdomain "zoom.us" to OffscreenCanvas allowlist

Status: VERIFIED FIXED

(Core :: Graphics: Canvas2D, defect, P3)

Description

[Andrew Osmond [:aosmond] (he/him)]

Sometimes video links are on zoom.us instead of a subdomain e.g. us04web.zoom.us, mozilla.zoom.us, etc.

Comment 1

[Andrew Osmond [:aosmond] (he/him)]

Attachment: Bug 1763801 - Enable OffscreenCanvas on zoom.us without subdomains.

Comment 2

[Andrew Osmond [:aosmond] (he/him)]

As a quick estimate, I would say this impacts ~1/4 of Zoom meetings, extrapolating from crash reports data of https://zoom.us/wc (571 reports) vs https://*.zoom.us/wc (1504 reports) over the last 3 months.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/bdaa0e8fb333
Enable OffscreenCanvas on zoom.us without subdomains. r=gfx-reviewers,jrmuizel

Comment 4

[Iulian Moraru]

https://hg.mozilla.org/mozilla-central/rev/bdaa0e8fb333

Comment 5

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

The patch landed in nightly and beta is affected.
:aosmond, 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.

Comment 6

[Andrew Osmond [:aosmond] (he/him)]

I manually verified that using zoom.us now results in getting gallery mode.

Comment 7

[Andrew Osmond [:aosmond] (he/him)]

Comment on attachment 9271460 [details]
Bug 1763801 - Enable OffscreenCanvas on zoom.us without subdomains.

Beta/Release Uplift Approval Request

• User impact if declined: Users who go to zoom.us URLs instead of subdomain.zoom.us URLs won't get gallery mode / OffscreenCanvas
• Is this code covered by automated tests?: Yes
• Has the fix been verified in Nightly?: Yes
• 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): We have already enabled this on *.zoom.us. This just is a pref change to enable it on zoom.us as well. I manually verified both continue to work in the latest nightly.
• String changes made/needed:

Comment 8

[Andrew Osmond [:aosmond] (he/him)]

Comment on attachment 9271460 [details]
Bug 1763801 - Enable OffscreenCanvas on zoom.us without subdomains.

Beta/Release Uplift Approval Request

• User impact if declined: Users who go to zoom.us URLs instead of subdomain.zoom.us URLs won't get gallery mode / OffscreenCanvas
• Is this code covered by automated tests?: Yes
• Has the fix been verified in Nightly?: Yes
• 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): We have already enabled this on *.zoom.us. This just is a pref change to enable it on zoom.us as well. I manually verified both continue to work in the latest nightly.
• String changes made/needed:

Comment 9

[Dianna Smith [:diannaS]]

Comment on attachment 9271460 [details]
Bug 1763801 - Enable OffscreenCanvas on zoom.us without subdomains.

Approved for 100.0b5

Comment 10

[Dianna Smith [:diannaS]]

https://hg.mozilla.org/releases/mozilla-beta/rev/1343695f125e

Comment 11

[Donal Meehan [:dmeehan]]

Comment on attachment 9271460 [details]
Bug 1763801 - Enable OffscreenCanvas on zoom.us without subdomains.

Approved for 99.0.1

Comment 12

[Donal Meehan [:dmeehan]]

https://hg.mozilla.org/releases/mozilla-release/rev/0f814a8ab240