Commit Graph
23 Commits
Author SHA1 Message Date
Sylvestre Ledru 074b376055 Bug 2023419 - Remove Emacs/Vim modelines from devtools/ r=emilio,devtools-backward-compat-reviewers,devtools-reviewers,nchevobbe
replaced by .editorconfig
# ignore-this-changeset

Differential Revision: https://phabricator.services.mozilla.com/D287841
2026-03-28 10:21:10 +00:00
Alexandru Marc 630e6cf0e8 Revert "Bug 2023419 - Remove Emacs/Vim modelines from devtools/ r=emilio,devtools-backward-compat-reviewers,devtools-reviewers,nchevobbe" for causing devtools failures
This reverts commit ed535dace9.
2026-03-17 12:46:01 +00:00
Sylvestre Ledru ed535dace9 Bug 2023419 - Remove Emacs/Vim modelines from devtools/ r=emilio,devtools-backward-compat-reviewers,devtools-reviewers,nchevobbe
replaced by .editorconfig

Differential Revision: https://phabricator.services.mozilla.com/D287841
2026-03-17 12:03:34 +00:00
Mark Banner b38cd3110b Bug 1999554 - Automatically fix ESLint rule jsdoc/check-types for devtools/. r=devtools-reviewers,devtools-backward-compat-reviewers,profiler-reviewers,canaltinova
Differential Revision: https://phabricator.services.mozilla.com/D272173
2025-11-21 16:16:08 +00:00
Mark Banner 2542790f53 Bug 1998787 - Automatic fixes for enabling ESLint jsdoc/tag-line rule everywhere. r=frontend-codestyle-reviewers,mtigley,perftest-reviewers,nchevobbe,home-newtab-reviewers,Gijs,mconley,sparky
Differential Revision: https://phabricator.services.mozilla.com/D271745
2025-11-10 21:27:12 +00:00
Alexandre Poirot 9677d792ee Bug 1971817 - [devtools] Clear the source map loader when removing sources. r=devtools-reviewers,bomsy
If a worker thread was using source maps, we weren't clearing the source map worker.
This will also be important when removing pretty printed source in order to stop
having the minimized source to be mapped when we disable pretty print.

Differential Revision: https://phabricator.services.mozilla.com/D255805
2025-07-28 15:29:35 +00:00
Serban Stanca 014d790c57 Revert "Bug 1977637, Bug 1971817 - [devtools] Better highlight pretty print state in Footers. r=devtools-reviewers,nchevobbe"
This reverts commit 46299ee751.

This reverts commit 54ad0c64a0.

This reverts commit fd568f41df.

This reverts commit cba6b633df.

This reverts commit 66e8617afe.

This reverts commit fa37bc5a94.
2025-07-23 01:31:38 +00:00
Alexandre Poirot cba6b633df Bug 1971817 - [devtools] Clear the source map loader when removing sources. r=devtools-reviewers,bomsy
If a worker thread was using source maps, we weren't clearing the source map worker.
This will also be important when removing pretty printed source in order to stop
having the minimized source to be mapped when we disable pretty print.

Differential Revision: https://phabricator.services.mozilla.com/D255805
2025-07-22 15:04:10 +00:00
Tooru Fujisawa 850a941bc2 Bug 1938858 - Remove the jsm support from the source map loader. r=devtools-reviewers,nchevobbe
Differential Revision: https://phabricator.services.mozilla.com/D233742
2025-01-23 12:09:38 +00:00
Julian Descottes 1bb7ba8e3b Bug 1899389 - [devtools] Use credentials for devtools sourcemap fetch call r=devtools-reviewers,ochameau
Differential Revision: https://phabricator.services.mozilla.com/D212059
2024-05-30 08:27:28 +00:00
Alexandre Poirot eae3d6b2b7 Bug 1834725 - [devtools] Retrieve the resolved source map URL and source map error. r=devtools-reviewers,nchevobbe
Extract information out of the source map worker to be stored into the Debugger store.

Renamed SourceMapLoader getOriginalURLs to loadSourceMap to better translate the heavy process involved in this method.
While doing that, also inline the transfer of source map ignore list in this call (instead of always having another worker message right after).

This will be used and tested in the next changeset.

Differential Revision: https://phabricator.services.mozilla.com/D188585
2024-01-15 12:57:19 +00:00
Natalia Csoregi 63d346b505 Backed out 4 changesets (bug 1834725) for causing failures on browser_styleeditor_loading_with_containers.js. CLOSED TREE
Backed out changeset 0965e956200a (bug 1834725)
Backed out changeset 4a8151163607 (bug 1834725)
Backed out changeset a56f42223377 (bug 1834725)
Backed out changeset a920356b63eb (bug 1834725)
2024-01-11 19:57:47 +02:00
Alexandre Poirot e563562e9a Bug 1834725 - [devtools] Retrieve the resolved source map URL and source map error. r=devtools-reviewers,nchevobbe
Extract information out of the source map worker to be stored into the Debugger store.

Renamed SourceMapLoader getOriginalURLs to loadSourceMap to better translate the heavy process involved in this method.
While doing that, also inline the transfer of source map ignore list in this call (instead of always having another worker message right after).

This will be used and tested in the next changeset.

Differential Revision: https://phabricator.services.mozilla.com/D188585
2024-01-11 15:27:25 +00:00
Hubert Boma Manilla 0ab4d94f23 Bug 1824705 - [devtools] Add support for sourcemaps ignore list r=devtools-reviewers,nchevobbe
Also there is a patch to add support for ignoreList to the sourcemap library
https://github.com/mozilla/source-map/pull/481

Differential Revision: https://phabricator.services.mozilla.com/D174357
2023-05-23 08:16:55 +00:00
Butkovits Atila f21dc3465a Backed out 2 changesets (bug 1824705) for causing failures at browser_misused_characters_in_strings.js. CLOSED TREE
Backed out changeset d67c783fcd63 (bug 1824705)
Backed out changeset d6d2c97f858d (bug 1824705)
2023-05-23 00:40:24 +03:00
Hubert Boma Manilla 327f77f3aa Bug 1824705 - [devtools] Add support for sourcemaps ignore list r=devtools-reviewers,nchevobbe
Also there is a patch to add support for ignoreList to the sourcemap library
https://github.com/mozilla/source-map/pull/481

Differential Revision: https://phabricator.services.mozilla.com/D174357
2023-05-22 18:27:16 +00:00
Mark Banner 0666077666 Bug 1826062 - Automatic fixes for upgrading Prettier to 2.8.8. r=mossop,perftest-reviewers,webcompat-reviewers,geckoview-reviewers,denschub,devtools-reviewers,sparky,calu
Differential Revision: https://phabricator.services.mozilla.com/D177027
2023-05-20 12:26:53 +00:00
Alexandre Poirot 58e1fc3a25 Bug 1829607 - [devtools] Fix exception message when original file failed loading. r=devtools-reviewers,nchevobbe
networkEvent was throwing string exception, which were poorly handled.

Differential Revision: https://phabricator.services.mozilla.com/D176348
2023-04-25 13:42:31 +00:00
Hubert Boma Manilla f8f50ae023 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
2023-03-07 16:26:41 +00:00
Alexandre Poirot 864f3ec609 Bug 1801621 - [devtools] Document source map requests module. r=bomsy
Differential Revision: https://phabricator.services.mozilla.com/D162566
2022-11-24 14:57:49 +00:00
Alexandre Poirot a6fdd5313b Bug 1801621 - [devtools] Remove useless createConsumer intermediate module. r=bomsy
Differential Revision: https://phabricator.services.mozilla.com/D162565
2022-11-24 14:57:49 +00:00
Alexandre Poirot 07deae9a5f Bug 1801621 - [devtools] Remove asset root setters. r=bomsy
Instead we could always load the wasm file when the modules are loaded.

Differential Revision: https://phabricator.services.mozilla.com/D162562
2022-11-24 14:57:48 +00:00
Alexandre Poirot 8c00f1d413 Bug 1794693 - [devtools] Move devtools-source-map to regular mozilla-central code. r=bomsy,perftest-reviewers,jdescottes,sparky
This used to be a node package. Let's make it become regular commonjs devtools modules.
This will make it trivial to migrate this to ES Modules.
Also possibly make this code become the unique layer in m-c on top of the source-map package.

We no longer use webpack to build the two bundles (index.js and worker.js),
instead, we are using the toolkit worker loader (require.js) in order to load
all this code without any build step.

As this is no longer a node package, I removed node-specific modules (assertRoot/wasmAsset)
and simplify the definition of wasm file URIs as they are now fixed.

Also moving the debugger to load internal "devtools/client/shared/source-map/source-map.js"
module in jest as running the Web Worker instantiated by source-map/index.js is too complex.

Differential Revision: https://phabricator.services.mozilla.com/D159115
2022-11-08 16:01:01 +00:00