Closed Bug 1763801 Opened 8 months ago Closed 8 months ago

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

Categories

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

defect

Tracking

()

VERIFIED FIXED
101 Branch
Tracking Status
firefox-esr91 --- unaffected
firefox99 --- fixed
firefox100 --- fixed
firefox101 --- fixed

People

(Reporter: aosmond, Assigned: aosmond)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

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

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.

Pushed by aosmond@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/bdaa0e8fb333
Enable OffscreenCanvas on zoom.us without subdomains. r=gfx-reviewers,jrmuizel
Status: NEW → RESOLVED
Closed: 8 months ago
Resolution: --- → FIXED
Target Milestone: --- → 101 Branch

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.

Flags: needinfo?(aosmond)

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

Status: RESOLVED → VERIFIED
Flags: needinfo?(aosmond)

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:
Attachment #9271460 - Flags: approval-mozilla-beta?

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:
Attachment #9271460 - Flags: approval-mozilla-release?

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

Approved for 100.0b5

Attachment #9271460 - Flags: approval-mozilla-beta? → approval-mozilla-beta+

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

Approved for 99.0.1

Attachment #9271460 - Flags: approval-mozilla-release? → approval-mozilla-release+
You need to log in before you can comment on or make changes to this bug.