Chris H-C
d89527a234
Bug 2039650 - Remove Legacy Telemetry devtools probes r=jdescottes,devtools-reviewers
...
Differential Revision: https://phabricator.services.mozilla.com/D300855
2026-05-28 16:32:18 +00:00
Alexandra Borovova
574778d7e5
Bug 2005054 - [devtools] Extract "formatMessageParametersAndText" into a shared module to make it easier to reuse in WebDriver BiDi. r=devtools-reviewers,nchevobbe
...
Differential Revision: https://phabricator.services.mozilla.com/D289754
2026-03-31 09:40:31 +00:00
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
Dave Townsend
03f5305808
Bug 2018127: Migrate ContentTask.spawn to SpecialPowers.spawn. r=firefox-desktop-core-reviewers ,credential-management-reviewers,devtools-reviewers,sessionstore-reviewers,translations-reviewers,omc-reviewers,home-newtab-reviewers,urlbar-reviewers,dom-worker-reviewers,ai-ondevice-reviewers,mconley,jdescottes,aminomancer,sthompson,dimi,asuth
...
Differential Revision: https://phabricator.services.mozilla.com/D284482
2026-03-05 19:48:23 +00:00
Alexandru Marc
9d384036a0
Revert "Bug 2018127: Migrate ContentTask.spawn to SpecialPowers.spawn. r=firefox-desktop-core-reviewers ,credential-management-reviewers,devtools-reviewers,sessionstore-reviewers,translations-reviewers,omc-reviewers,home-newtab-reviewers,urlbar-reviewers,dom-worker-reviewers,ai-ondevice-reviewers,mconley,jdescottes,aminomancer,sthompson,dimi,asuth" for causing bc failures @ browser_resume_bkg_video_on_tab_hover.js
...
This reverts commit 3c14ee2dce .
2026-03-05 14:39:32 +00:00
Dave Townsend
3c14ee2dce
Bug 2018127: Migrate ContentTask.spawn to SpecialPowers.spawn. r=firefox-desktop-core-reviewers ,credential-management-reviewers,devtools-reviewers,sessionstore-reviewers,translations-reviewers,omc-reviewers,home-newtab-reviewers,urlbar-reviewers,dom-worker-reviewers,ai-ondevice-reviewers,mconley,jdescottes,aminomancer,sthompson,dimi,asuth
...
Differential Revision: https://phabricator.services.mozilla.com/D284482
2026-03-04 21:12:43 +00:00
Anutrix
0a87d2f2c8
Bug 2016968 - Replace Python's json with 'mozfile.json' throughout devtools directory. r=devtools-reviewers,ochameau
...
Differential Revision: https://phabricator.services.mozilla.com/D283446
2026-02-18 20:53:00 +00:00
Alexandru Marc
d296caccba
Revert "Bug 2006716 - Remove unused default_bindir() from ruff.py r=ahal,linter-reviewers" for causing toolchain bustages
...
This reverts commit 76033c983e .
Revert "Bug 2006716 - Add `ruff-format` linter r=ahal,linter-reviewers"
This reverts commit 19fd8c23ca .
Revert "Bug 2006716 - Remove `black` linter r=ahal,linter-reviewers"
This reverts commit 0b01c783a6 .
Revert "Bug 2006716 - Fix `ruff` linter `stderr` callback broken in D190696 r=ahal,linter-reviewers"
This reverts commit 72cab75517 .
Revert "Bug 2006716 - Update `ruff` to version `0.14.9` and use consistent versions in `lint.txt` and `python-test.txt` r=ahal,linter-reviewers"
This reverts commit a434d066ce .
Revert "Bug 2006716 - Reformat entire repo with `ruff-format` (`black` replacement) r=ahal,sylvestre,linter-reviewers,perftest-reviewers,mozperftest-reviewers,geckoview-reviewers,extension-reviewers,media-playback-reviewers,webrtc-reviewers,releng-reviewers,sessionstore-reviewers,win-reviewers,dom-storage-reviewers,translations-reviewers,omc-reviewers,android-reviewers,home-newtab-reviewers,toolkit-telemetry-reviewers,firefox-style-system-reviewers,layout-reviewers,dom-worker-reviewers,geckodriver-reviewers,Sasha,emz,fchasen,sthompson,dholbert,sparky,padenot,emilio,Gijs,mconley,tcampbell,mjf,emcminn,willdurand,gstoll,jgraham,asuth"
This reverts commit 3ed06912eb .
2026-01-07 00:54:28 +00:00
Alex Hochheiden
3ed06912eb
Bug 2006716 - Reformat entire repo with ruff-format (black replacement) r=ahal,sylvestre,linter-reviewers,perftest-reviewers,mozperftest-reviewers,geckoview-reviewers,extension-reviewers,media-playback-reviewers,webrtc-reviewers,releng-reviewers,sessionstore-reviewers,win-reviewers,dom-storage-reviewers,translations-reviewers,omc-reviewers,android-reviewers,home-newtab-reviewers,toolkit-telemetry-reviewers,firefox-style-system-reviewers,layout-reviewers,dom-worker-reviewers,geckodriver-reviewers,Sasha,emz,fchasen,sthompson,dholbert,sparky,padenot,emilio,Gijs,mconley,tcampbell,mjf,emcminn,willdurand,gstoll,jgraham,asuth
...
# ignore-this-changeset
Differential Revision: https://phabricator.services.mozilla.com/D276894
2026-01-07 00:32:15 +00:00
Nicolas Chevobbe
4f13eaf338
Bug 2004579 - [devtools] Compute string index from character length in analyzeInputString. r=devtools-reviewers,bomsy.
...
We were only increasing the index by 1, which wouldn't account for surrogate pairs (e.g. emojis).
We need to increase the index by the character length so the string can be manipulated accurately
after.
Test case is added to cover the fix.
Differential Revision: https://phabricator.services.mozilla.com/D275419
2025-12-09 08:39:36 +00:00
Tom Marble
5ce09d7446
Bug 1995583 - test manifest cleanup for idiomatic usage r=jmaher,necko-reviewers,webcompat-reviewers,geckoview-reviewers,application-update-reviewers,media-playback-reviewers,sessionstore-reviewers,tabbrowser-reviewers,translations-reviewers,omc-reviewers,browser-installer-reviewers,twisniewski,masayuki,dom-core,smaug,nalexander,emcminn,sthompson,alwu,ai-ondevice-reviewers,valentin
...
For TOML manifests:
1. Change processor -> arch
2. Use arch instead of bits bits == '64' =>
aarch64 (default on mac 15.30) or x86_64 bits == '32' => x86
3. Replace android_version with corresponding os_version
4. Remove conditions with linux os_version=="18.04"
(unless they contain verify or verify-standalone)
5. Remove conditions with mac os_version=='11.20',
(unless they contain verify or verify-standalone)
6. Remove conditions with win os_version=='11.2009' or win11_2009,
(unless they contain verify or verify-standalone)
7. Linux conditions should have os_version, arch and display
(simple "os == 'linux'" will get expanded to these two)
"os == 'linux' && os_version == '22.04' && arch == 'x86_64' && display == 'wayland'"
"os == 'linux' && os_version == '24.04' && arch == 'x86_64' && display == 'x11'"
8. Mac conditions should have os_version and arch
(simple "os == 'mac'" will get expanded to these three)
"os == 'mac' && os_version == '10.15' && arch == 'x86_64'",
"os == 'mac' && os_version == '14.70' && arch == 'x86_64'",
"os == 'mac' && os_version == '15.30' && arch == 'aarch64'",
9. Replace apple_catalina with
os == 'mac' && os_version == '10.15' && arch == 'x86_64'
10. Replace apple_silicon with
os == 'mac' && os_version = '15.30' && arch == 'aarch64'
11. Replace win10_2009 with
os == 'win' && os_version = '10.2009' && arch == 'x86_64'
12. Replace "!debug" with three conditions "asan", "opt", or "tsan"
13. Replace literal boolean comparison with variables
(e.g. "debug == false" => "!debug)
Resolves 2000285
Signed-off-by: Tom Marble <[email protected] >
Differential Revision: https://phabricator.services.mozilla.com/D272815
2025-11-22 00:30:39 +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
Julian Descottes
b5a1d0ac85
Bug 1275096 - [devtools] Fix eslint no aArgs violations r=frontend-codestyle-reviewers,devtools-reviewers,nchevobbe
...
Differential Revision: https://phabricator.services.mozilla.com/D268679
2025-10-15 16:47:26 +00:00
Florian Quèze
adb44fa5a4
Bug 1989356 - Add run-if = ["os != 'android'"] conditions in desktop only xpcshell test manifests, r=jmaher,application-update-reviewers,search-reviewers,devtools-reviewers,sync-reviewers,places-reviewers,profiler-reviewers,win-reviewers,omc-reviewers,migration-reviewers,urlbar-reviewers,browser-installer-reviewers,profiles-reviewers,layout-reviewers,mossop,mstange,dholbert,gstoll,bytesized,scunnane,nchevobbe,aminomancer.
...
Differential Revision: https://phabricator.services.mozilla.com/D265322
2025-09-19 20:56:51 +00:00
Joel Maher
1a4fb71e2c
Bug 1975729 - adjust whitespace and newlines for most mochitest .toml files, also remove unnecessary skip-if=verify conditions. r=florian,webdriver-reviewers,necko-reviewers,mtigley,webcompat-reviewers,extension-reviewers,application-update-reviewers,media-playback-reviewers,credential-management-reviewers,search-reviewers,devtools-reviewers,anti-tracking-reviewers,places-reviewers,profiler-reviewers,dom-storage-reviewers,translations-reviewers,omc-reviewers,home-newtab-reviewers,sidebar-reviewers,urlbar-reviewers,firefox-ai-ml-reviewers,twisniewski,bytesized,emcminn,Standard8,karlt,padenot,jdescottes,dimi,kershaw,robwu,canaltinova,nbarrett,asuth
...
Differential Revision: https://phabricator.services.mozilla.com/D256136
2025-07-12 16:26:27 +00:00
Hubert Boma Manilla
fc5a8ad266
Bug 1557795 - [devtools] Start triggering multiple network events updates r=devtools-reviewers,jdescottes
...
Lets start triggering updates at certain keypoint in the cycle instead
batching into one update at the end.
This should depend on mechanisim for throttling resources so as not to spam the
frontend with lots of updates
Differential Revision: https://phabricator.services.mozilla.com/D247424
2025-07-01 23:01:47 +00:00
Nicolas Chevobbe
4339627f90
Bug 1933140 - [devtools] Make sure using keyword is in autocompletion result when expected. r=devtools-reviewers,ochameau
...
Differential Revision: https://phabricator.services.mozilla.com/D254969
2025-06-27 05:46:46 +00:00
Nicolas Chevobbe
172e8b8336
Bug 1933140 - [devtools] Add shared module for GenerateReservedWordsJS.py and GenerateReservedWords.py to read reserved words list. r=arai.
...
This adds a ReservedWordReader module that will return the list of reserved words
defined in ReservedWords.h.
This is especially helpful for devtools as we should automatically get any
new/updated keywords on the platform.
Differential Revision: https://phabricator.services.mozilla.com/D254938
2025-06-27 05:46:42 +00:00
Gijs Kruitbosch
9c5be039df
Bug 1971751 - update tests that misuse Assert.ok after no-comparison-or-assignment-inside-ok update, r=frontend-codestyle-reviewers,webdriver-reviewers,extension-reviewers,settings-reviewers,application-update-reviewers,pip-reviewers,credential-management-reviewers,search-reviewers,devtools-reviewers,sync-reviewers,cookie-reviewers,sessionstore-reviewers,tabbrowser-reviewers,places-reviewers,profiler-reviewers,omc-reviewers,migration-reviewers,home-newtab-reviewers,webcompat-reviewers,urlbar-reviewers,permissions-reviewers,firefox-ai-ml-reviewers,profiles-reviewers,robwu,julienw,bytesized,sthompson,valentin,jhirsch,markh,twisniewski,Standard8,dimi,nbarrett,mconley,timhuang
...
Differential Revision: https://phabricator.services.mozilla.com/D253662
2025-06-26 10:33:09 +00:00
agoloman
fd1e9e499d
Revert "Bug 1971751 - update no-comparison-or-assignment-inside-ok to also cover Assert.ok, r=frontend-codestyle-reviewers,Standard8" for causing lint failures @test_maybeCapExpiry.js.
...
This reverts commit 2744165954 .
Revert "Bug 1971751 - update tests that misuse Assert.ok after no-comparison-or-assignment-inside-ok update, r=frontend-codestyle-reviewers,webdriver-reviewers,extension-reviewers,settings-reviewers,application-update-reviewers,pip-reviewers,credential-management-reviewers,search-reviewers,devtools-reviewers,sync-reviewers,cookie-reviewers,sessionstore-reviewers,tabbrowser-reviewers,places-reviewers,profiler-reviewers,omc-reviewers,migration-reviewers,home-newtab-reviewers,webcompat-reviewers,urlbar-reviewers,permissions-reviewers,firefox-ai-ml-reviewers,profiles-reviewers,robwu,julienw,bytesized,sthompson,valentin,jhirsch,markh,twisniewski,Standard8,dimi,nbarrett,mconley,timhuang"
This reverts commit ba7298d756 .
Revert "Bug 1971751 - manual test updates where linter autofix creates issues, r=frontend-codestyle-reviewers,Standard8"
This reverts commit e12e0d3181 .
2025-06-26 01:11:00 +00:00
Gijs Kruitbosch
ba7298d756
Bug 1971751 - update tests that misuse Assert.ok after no-comparison-or-assignment-inside-ok update, r=frontend-codestyle-reviewers,webdriver-reviewers,extension-reviewers,settings-reviewers,application-update-reviewers,pip-reviewers,credential-management-reviewers,search-reviewers,devtools-reviewers,sync-reviewers,cookie-reviewers,sessionstore-reviewers,tabbrowser-reviewers,places-reviewers,profiler-reviewers,omc-reviewers,migration-reviewers,home-newtab-reviewers,webcompat-reviewers,urlbar-reviewers,permissions-reviewers,firefox-ai-ml-reviewers,profiles-reviewers,robwu,julienw,bytesized,sthompson,valentin,jhirsch,markh,twisniewski,Standard8,dimi,nbarrett,mconley,timhuang
...
Differential Revision: https://phabricator.services.mozilla.com/D253662
2025-06-26 00:07:43 +00:00
Mark Banner
3cc3379c8f
Bug 1881701 - Change the devtools configurations to work with the ESLint flat configuration.r=devtools-reviewers,profiler-reviewers,julienw,nchevobbe
...
This consolidates the jest environment definition, and the xpcshell globals into the devtools/.eslintrc.mjs file, for easier configurations.
It also changes the restart of the configurations to work with flat config.
Differential Revision: https://phabricator.services.mozilla.com/D249948
2025-05-24 17:10:59 +00:00
Mark Banner
d7c7c5369b
Bug 1881701 - Rename many ESLint files to use .mjs. r=frontend-codestyle-reviewers,perftest-reviewers,Gijs,sparky,webcompat-reviewers,twisniewski
...
This is separate to help preserve history of the files.
Differential Revision: https://phabricator.services.mozilla.com/D249945
2025-05-24 17:10:51 +00:00
Joel Maher
ce1cae83af
Bug 1959269 - clean and standardize devtools/**/xpcshell.toml. r=aryx,devtools-reviewers,ochameau
...
Differential Revision: https://phabricator.services.mozilla.com/D244831
2025-04-15 14:47:07 +00:00
Edgar Chen
3d0d8aa9c3
Bug 834208 - Align the document name getter with the spec and other browsers; r=smaug,devtools-reviewers,ochameau
...
Original patch: https://hg.mozilla.org/mozreview/gecko/rev/7d93b7e3c1499b0b81d970722537b98f5d88efdd
Original author: Aryeh Gregor <[email protected] >
According to the spec,
- https://html.spec.whatwg.org/#dom-document-nameditem
- https://html.spec.whatwg.org/#named-access-on-the-window-object
the document and window named getters return different values. This patch creates
a separated content list for document named getter while keeping the original
content list for the window named getter. Additionally, the content list now
includes both names and ids, in addition to the id list (which is still used for
other things like getElementById()).
This means one element might be on up to multiple lists, but this implementation
is necessary. Specifically, we must return a consistent live list for document.foo
and for window.foo, so they need to be stored separately and cannot be generated
on the fly from a single shared list.
Differential Revision: https://phabricator.services.mozilla.com/D240951
2025-03-20 15:42:43 +00:00
Norisz Fay
fbba6901d1
Backed out changeset c28cc6141d04 (bug 834208) for causing bustage on nsHTMLDocument.cpp
2025-03-20 15:50:36 +02:00
Edgar Chen
b60c7d8733
Bug 834208 - Align the document name getter with the spec and other browsers; r=smaug,devtools-reviewers,ochameau
...
Original patch: https://hg.mozilla.org/mozreview/gecko/rev/7d93b7e3c1499b0b81d970722537b98f5d88efdd
Original author: Aryeh Gregor <[email protected] >
According to the spec,
- https://html.spec.whatwg.org/#dom-document-nameditem
- https://html.spec.whatwg.org/#named-access-on-the-window-object
the document and window named getters return different values. This patch creates
a separated content list for document named getter while keeping the original
content list for the window named getter. Additionally, the content list now
includes both names and ids, in addition to the id list (which is still used for
other things like getElementById()).
This means one element might be on up to multiple lists, but this implementation
is necessary. Specifically, we must return a consistent live list for document.foo
and for window.foo, so they need to be stored separately and cannot be generated
on the fly from a single shared list.
Differential Revision: https://phabricator.services.mozilla.com/D240951
2025-03-20 11:41:46 +00:00
Nicolas Chevobbe
db74a1cb6c
Bug 1953942 - [devtools] Check that actor.enumProperties exists before calling it in getConsoleTableMessageItems. r=devtools-reviewers,ochameau,jdescottes.
...
Differential Revision: https://phabricator.services.mozilla.com/D242142
2025-03-19 15:08:22 +00:00
Tan S. B.
e1af221fee
Bug 1610833 - Autocomplete fails after /(
...
Differential Revision: https://phabricator.services.mozilla.com/D236663
2025-02-11 07:50:18 +00:00
Butkovits Atila
a8c64705b1
Backed out changeset da119eca293c (bug 1610833) for causing linting failures. CLOSED TREE
2025-02-04 17:54:51 +02:00
Tan S. B.
58982370e3
Bug 1610833 - Fix console autocomplete after /( . r=devtools-reviewers,nchevobbe
...
Differential Revision: https://phabricator.services.mozilla.com/D236663
2025-02-04 14:31:32 +00:00
Alexandre Poirot
26d9f9bb91
Bug 1713093 - [devtools] Remove TargetMixin.attachThread. r=devtools-reviewers,nchevobbe
...
This code was only used for workers targets, which weren't passing breakpoints
from the server side via "sessionData" concept.
Instead, we were still relying on the frontend to pass them and only after that
manually requesting the thread to resume its execution.
Now, there is still a notion of "attach" around the thread actor.
While all the targets are immediately instantiating their related thread actor,
the thread actor isn't active by default.
It becomes active as soon as:
* SOURCE or THREAD_STATE resource is watched
* breakpoints are set via session data (breakpoint list actor)
* thread configuration are set via session data (thread configuration actor)
I had to do one of these things in order to keep test from working.
In the production code, we do most of these things by default on the toolbox startup.
We could revise this behavior by:
* always attaching the thread actor, but it may start pausing unexpectedly in some tests.
* decouple sources from breakpoints within the thread actor, so that watching for sources doesn't necessarily "attach" the thread actor and activate breakpoints/debugger statements.
* have a new thread configuration to explicitly enable the thread actor pausing features.
I'm often using thread configuration to enable pausing in tests.
This required to improve xpcshell test helpers to be able to use Watcher/Commands.
About browser_target_command_bfcache.js, the DevTools code is now so quick that the
back/forward requests at done too quickly one after the other and fail actually moving to
the other document.
Differential Revision: https://phabricator.services.mozilla.com/D166321
2024-03-20 12:30:17 +00:00
Nicolas Chevobbe
adffa997c1
Bug 1698260 - [devtools] Fix autocomplete for Infinity. r=devtools-reviewers,ochameau.
...
Differential Revision: https://phabricator.services.mozilla.com/D204478
2024-03-15 05:59:56 +00:00
pstanciu
5a1a45a9a8
Backed out 6 changesets (bug 1713093, bug 1691986) for turning Bug 1767717 into permafail. CLOSED TREE
...
Backed out changeset b0285f8abf2e (bug 1713093)
Backed out changeset 6532dbabf879 (bug 1713093)
Backed out changeset 2670f26f6bff (bug 1713093)
Backed out changeset a63d603a1973 (bug 1713093)
Backed out changeset 0118d0bc64ba (bug 1713093)
Backed out changeset e29a81bc7299 (bug 1691986)
2024-03-08 20:32:38 +02:00
Alexandre Poirot
9f0c59245f
Bug 1713093 - [devtools] Remove TargetMixin.attachThread. r=devtools-reviewers,nchevobbe
...
This code was only used for workers targets, which weren't passing breakpoints
from the server side via "sessionData" concept.
Instead, we were still relying on the frontend to pass them and only after that
manually requesting the thread to resume its execution.
Now, there is still a notion of "attach" around the thread actor.
While all the targets are immediately instantiating their related thread actor,
the thread actor isn't active by default.
It becomes active as soon as:
* SOURCE or THREAD_STATE resource is watched
* breakpoints are set via session data (breakpoint list actor)
* thread configuration are set via session data (thread configuration actor)
I had to do one of these things in order to keep test from working.
In the production code, we do most of these things by default on the toolbox startup.
We could revise this behavior by:
* always attaching the thread actor, but it may start pausing unexpectedly in some tests.
* decouple sources from breakpoints within the thread actor, so that watching for sources doesn't necessarily "attach" the thread actor and activate breakpoints/debugger statements.
* have a new thread configuration to explicitly enable the thread actor pausing features.
I'm often using thread configuration to enable pausing in tests.
This required to improve xpcshell test helpers to be able to use Watcher/Commands.
About browser_target_command_bfcache.js, the DevTools code is now so quick that the
back/forward requests at done too quickly one after the other and fail actually moving to
the other document.
Differential Revision: https://phabricator.services.mozilla.com/D166321
2024-03-06 13:43:48 +00:00
Dave Townsend
2af4d349f3
Bug 1864896: Autofix unused function arguments (devtools). r=profiler-reviewers,devtools-reviewers,nchevobbe,julienw
...
Differential Revision: https://phabricator.services.mozilla.com/D202959
2024-03-05 14:21:15 +00:00
Tooru Fujisawa
571e7ae7b8
Bug 1875639 - Part 2: Explicitly use contextual loader in files loaded into DevTools global. r=ochameau,devtools-reviewers
...
Differential Revision: https://phabricator.services.mozilla.com/D199467
2024-02-20 04:49:57 +00:00
Gijs Kruitbosch
3288f177a4
Bug 1780071 - autofix devtools/ violations of no-comparison-or-assignment-inside-ok r=profiler-reviewers,devtools-reviewers,julienw,nchevobbe
...
Differential Revision: https://phabricator.services.mozilla.com/D198975
2024-01-26 13:12:07 +00:00
Nicolas Chevobbe
5a608228ab
Bug 1872689 - [devtools] Prepare codebase for eslint-plugin-react update. r=devtools-reviewers,ochameau.
...
Updating eslint-plugin-react to 7.33.2 raises new eslint errors.
A few component needed to have their proptypes fixed (or added).
JSPropertyProvider was considered as a react component by the plugin,
I guess because it is a function starting with an uppercase letter.
Since it is a bit unusual to have such thing, I renamed it slightly
to make it look more like a function.
Differential Revision: https://phabricator.services.mozilla.com/D197596
2024-01-04 16:06:08 +00:00
Alexandre Poirot
faf55b5e07
Bug 1616524 - [devtools] Allow side effect free console commands to be called in eager evaluation. r=devtools-reviewers,nchevobbe
...
For now, only console command values like $0, $_ and $x were allowed in eager evaluation,
only by luck as these commands were evaluated and cached *before* Frame.evalWithBinding and Object.executeInGlobalWithBindings
started running the console input string.
But methods were cancelled by onNativeCall. This new isSideEffectFree flag allows to identify these methods.
Differential Revision: https://phabricator.services.mozilla.com/D195022
2023-12-02 08:07:15 +00:00
Alexandre Poirot
f47ac5b3fe
Bug 1830915 - [devtools] Introduce a generic mechanism to print web console command usage. r=devtools-reviewers,nchevobbe,flod,fluent-reviewers
...
We can start printing usage description right into the console by using `:command --help` or `:command --usage`.
I also simplified the integration between eval-with-debugger and manager
in order to fully delegate command execution to the manager.
Also improved the jsdoc for the register method. This new feature will be covered by test in the next patch.
Differential Revision: https://phabricator.services.mozilla.com/D193272
2023-11-24 14:02:16 +00:00
Joel Maher
cac31d18e3
Bug 1859873 - batch 13 xpcshell.ini -> .toml. r=aryx,devtools-reviewers,profiler-reviewers
...
Differential Revision: https://phabricator.services.mozilla.com/D194161
2023-11-21 12:32:50 +00:00
Alexandre Poirot
1955b12acf
Bug 1863512 - Disable tests using SharedArrayBuffer on release/beta. r=devtools-reviewers,nchevobbe
...
SharedArrayBuffer can only easily be testing through dom.postMessage.sharedArrayBuffer.bypassCOOP_COEP.insecure.enabled
preference which is locked on beta and release channels.
Differential Revision: https://phabricator.services.mozilla.com/D192942
2023-11-08 17:33:54 +00:00
Alexandre Poirot
dbc1747b07
Bug 1860888 - [devtools] Fix logging SharedArrayBuffer from Worklet/Workers. r=devtools-reviewers,smaug
...
This custom CloneDataPolicy was already using while writing the structured clone,
but not when reading it.
Differential Revision: https://phabricator.services.mozilla.com/D191783
2023-11-06 14:42:52 +00:00
Stanca Serban
9c3f5b4c85
Backed out changeset 4492a1a7f6e7 (bug 1860888) for causing mochitests failures in test_console_serviceworker_cached.html. CLOSED TREE
2023-11-03 12:46:25 +02:00
Alexandre Poirot
0385bcb81d
Bug 1860888 - [devtools] Fix logging SharedArrayBuffer from Worklet/Workers. r=devtools-reviewers,smaug
...
This custom CloneDataPolicy was already using while writing the structured clone,
but not when reading it.
Differential Revision: https://phabricator.services.mozilla.com/D191783
2023-11-03 09:39:41 +00:00
Tom Marble
6c1231e2e8
Bug 1853244 - convert .ini manifests to .toml: batch 6 {caps,devtools,dom}/**/browser.ini r=jmaher,media-playback-reviewers,devtools-reviewers,profiler-reviewers,alwu,nchevobbe,canaltinova
...
Differential Revision: https://phabricator.services.mozilla.com/D188856
2023-09-22 20:42:14 +00:00