Skip to content

Remove SharedArrayBuffer workaround#5596

Merged
mstange merged 1 commit intofirefox-devtools:mainfrom
mstange:fix-sharedarraybuffer-bug
Sep 4, 2025
Merged

Remove SharedArrayBuffer workaround#5596
mstange merged 1 commit intofirefox-devtools:mainfrom
mstange:fix-sharedarraybuffer-bug

Conversation

@mstange
Copy link
Contributor

@mstange mstange commented Sep 4, 2025

Production | Main branch | Deploy preview

instanceof SharedArrayBuffer throws if window.crossOriginIsolated is false (would need COOP and COEP)

This fixes a regression from #5584.

…throwing if window.crossOriginIsolated is false (need COOP and COEP)
@codecov
Copy link

codecov bot commented Sep 4, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.73%. Comparing base (c405844) to head (2a361e2).
⚠️ Report is 10 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #5596   +/-   ##
=======================================
  Coverage   85.73%   85.73%           
=======================================
  Files         309      309           
  Lines       30390    30386    -4     
  Branches     8365     8364    -1     
=======================================
- Hits        26055    26052    -3     
+ Misses       3914     3913    -1     
  Partials      421      421           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@mstange mstange merged commit a44acf1 into firefox-devtools:main Sep 4, 2025
15 checks passed
@canova canova mentioned this pull request Sep 5, 2025
canova added a commit that referenced this pull request Sep 5, 2025
Changes:

[Markus Stange] Some path fixes (#5581)
[depfu[bot]] Update all Yarn dependencies (2025-09-03) (#5585)
[Nazım Can Altınova] Remove the findDOMNode usage from ContextMenu
(#5588)
[Markus Stange] Replace @fetch-mock/jest with plain fetch-mock (#5575)
[Ryan Hunt] Replace zee-worker.js with compression streams API (#5584)
[Markus Stange] Remove unused FilterNavigatorBar animations (#5591)
[Markus Stange] Use different ports for start-prod and start-examples
(#5592)
[Markus Stange] Remove SharedArrayBuffer workaround (#5596)
[Jeff Muizelaar] Make time after responseEnd have a more accurate name
(#5582)
[Ryan Hunt] Move compression and decompression to a worker (#5597)
[Markus Stange] Remove Flow and no-TypeScript settings from
.vscode/settings.json (#5601)
[Markus Stange] Simplify the eslint command in package.json so that
eslint.config.mjs is the only place you need to look if you want to know
if a file is being checked (#5600)
[Markus Stange] Fix search filtering (#5599)
[Nazım Can Altınova] 🔃 Sync: l10n -> main (September 5, 2025) (#5603)

And thanks to our localizers:

de: mstange
tr: Fernando Javier
tr: Rua
tr: Selim Şumlu
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants