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

Richard Pospesel (@richard) git at gitlab.torproject.org
Wed Apr 12 16:33:56 UTC 2023



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


Commits:
c2c2ae6b by Kershaw Chang at 2023-04-12T16:33:02+00:00
Bug 1685403 - Use NS_GetFinalChannelURI at some places, r=necko-reviewers,valentin

Differential Revision: https://phabricator.services.mozilla.com/D164656
- - - - -
6e2aeeed by Hubert Boma Manilla at 2023-04-12T16:33:03+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
- - - - -
56ac62b9 by Hubert Boma Manilla at 2023-04-12T16:33:03+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

- - - - -
fd6ffb84 by Jon Coppeard at 2023-04-12T16:33:04+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
- - - - -
52a82664 by Jon Coppeard at 2023-04-12T16:33:04+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/b53567ebae7fd52202962f9c0ba792823f2a2dca...52a82664620b9a251aaaeaed889993bedc67c03e

-- 
View it on GitLab: https://gitlab.torproject.org/tpo/applications/tor-browser/-/compare/b53567ebae7fd52202962f9c0ba792823f2a2dca...52a82664620b9a251aaaeaed889993bedc67c03e
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/tor-commits/attachments/20230412/4293559d/attachment-0001.htm>


More information about the tor-commits mailing list