[tbb-commits] [Git][tpo/applications/tor-browser][tor-browser-102.10.0esr-12.0-1] 5 commits: Bug 1685403 - Use NS_GetFinalChannelURI at some places, r=necko-reviewers, valentin

[Richard Pospesel (@richard)]

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


Commits:
299c2669 by Kershaw Chang at 2023-04-11T20:04:48+00:00
Bug 1685403 - Use NS_GetFinalChannelURI at some places, r=necko-reviewers,valentin

Differential Revision: https://phabricator.services.mozilla.com/D164656
- - - - -
85ac56c0 by Hubert Boma Manilla at 2023-04-11T20:04:49+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
- - - - -
6c6c0e3e by Hubert Boma Manilla at 2023-04-11T20:04:49+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

- - - - -
049f5f77 by Jon Coppeard at 2023-04-11T20:04:49+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
- - - - -
957cd5b1 by Jon Coppeard at 2023-04-11T20:04:50+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/cebb3c6e322f0aab2f0566596354c3450af59057...957cd5b15b067120cff66742b6fcdb538010fb43

-- 
View it on GitLab: https://gitlab.torproject.org/tpo/applications/tor-browser/-/compare/cebb3c6e322f0aab2f0566596354c3450af59057...957cd5b15b067120cff66742b6fcdb538010fb43
You're receiving this email because of your account on gitlab.torproject.org.


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.torproject.org/pipermail/tbb-commits/attachments/20230412/3ae980fa/attachment.htm>