Richard Pospesel pushed to branch base-browser-102.10.0esr-12.0-1 at The Tor Project / Applications / Tor Browser

Commits: 08871ddf by Kershaw Chang at 2023-04-12T16:30:10+00:00 Bug 1685403 - Use NS_GetFinalChannelURI at some places, r=necko-reviewers,valentin

Differential Revision: https://phabricator.services.mozilla.com/D164656 - - - - - 23e3a6b7 by Hubert Boma Manilla at 2023-04-12T16:30:10+00:00 Bug 1790542 - [devtools] Do not allow redirects on the 'sourceMappingUrl' r=jdescottes,devtools-reviewers

- Block redirects on sourceMappingUrl - Add debugger test

Differential Revision: https://phabricator.services.mozilla.com/D168648 - - - - - 583826d1 by Hubert Boma Manilla at 2023-04-12T16:30:11+00:00 Bug 1790542 - [devtools] Convert browser_source_map-reload.js test from using .sjs files to createTestHTTPServer r=ochameau,devtools-reviewers

.sjs files depend on redirects to load static files. In D168648 we block redirects on sourceMappingUrl, so this browser_source_map-reload.js fails because we use .sjs to serve the content.

This patch refactors to no longer use .sjs files

Differential Revision: https://phabricator.services.mozilla.com/D169739

- - - - - 0e00f51b by Jon Coppeard at 2023-04-12T16:30:11+00:00 Bug 1816158 - Part 1: Disallow GC while iterating global's debugger vector r=sfink

GC can mutate this vector so don't allow that while we are iterating. I think it would be safe to use index-based iteration but it's safer to just ban it entirely.

This fixes the crash produced by the testcase.

Differential Revision: https://phabricator.services.mozilla.com/D169701 - - - - - c4086539 by Jon Coppeard at 2023-04-12T16:30:12+00:00 Bug 1816158 - Part 2: Require no GC when giving out references to the realm's debugger vector r=sfink

To prevent any other instances of this problme we can update the getDebuggers() methods on the global and the realm to require no GC.

Depends on D169701

Differential Revision: https://phabricator.services.mozilla.com/D169702

- - - - -

30 changed files:

- devtools/client/debugger/packages/devtools-source-map/src/source-map.js - devtools/client/debugger/packages/devtools-source-map/src/utils/fetchSourceMap.js - devtools/client/debugger/packages/devtools-source-map/src/utils/privileged-network-request.js - devtools/client/debugger/test/mochitest/browser.ini - + devtools/client/debugger/test/mochitest/browser_dbg-sourcemaps-redirect.js - devtools/client/debugger/test/mochitest/shared-head.js - devtools/client/framework/test/browser.ini - devtools/client/framework/test/browser_source_map-reload.js - − devtools/client/framework/test/code_bundle_reload_1.js - − devtools/client/framework/test/code_bundle_reload_1.js.map - − devtools/client/framework/test/code_bundle_reload_2.js - − devtools/client/framework/test/code_bundle_reload_2.js.map - + devtools/client/framework/test/reload/.eslintrc.js - + devtools/client/framework/test/reload/README.md - + devtools/client/framework/test/reload/package.json - + devtools/client/framework/test/reload/v1/code_bundle_reload.js - + devtools/client/framework/test/reload/v1/code_bundle_reload.js.map - devtools/client/framework/test/code_reload_1.js → devtools/client/framework/test/reload/v1/code_reload_1.js - devtools/client/framework/test/doc_reload.html → devtools/client/framework/test/reload/v1/doc_reload.html - + devtools/client/framework/test/reload/v2/code_bundle_reload.js - + devtools/client/framework/test/reload/v2/code_bundle_reload.js.map - devtools/client/framework/test/code_reload_2.js → devtools/client/framework/test/reload/v2/code_reload_2.js - + devtools/client/framework/test/reload/v2/doc_reload.html - + devtools/client/framework/test/reload/webpack.config.js - − devtools/client/framework/test/sjs_code_bundle_reload_map.sjs - − devtools/client/framework/test/sjs_code_reload.sjs - devtools/client/shared/test/shared-head.js - dom/fetch/FetchDriver.cpp - js/src/debugger/DebugAPI-inl.h - js/src/debugger/DebugAPI.h

The diff was not included because it is too large.

View it on GitLab: https://gitlab.torproject.org/tpo/applications/tor-browser/-/compare/33ad106...