Commit Graph

544 Commits

Author SHA1 Message Date
Alexandre Poirot
c6f27577ad Bug 1909548 - [devtools] Removing unused Preview.css file. r=devtools-reviewers,jdescottes
Only Popup.css is actually used for popup previews.

Differential Revision: https://phabricator.services.mozilla.com/D219090
2024-09-02 13:20:39 +00:00
Nicolas Chevobbe
81fc020034 Bug 1912006 - [devtools] Use OutputParser to display values in variable tooltip. r=devtools-reviewers,jdescottes.
Re-use the OutputParser options from the declaration so we can have similar
results than in the Rules view.

Differential Revision: https://phabricator.services.mozilla.com/D219145
2024-08-27 12:08:59 +00:00
Nicolas Chevobbe
5ae1906a02 Bug 1912267 - [devtools] Display <empty> for empty values in variable tooltip. r=devtools-reviewers,jdescottes.
Differential Revision: https://phabricator.services.mozilla.com/D219143
2024-08-27 06:07:50 +00:00
Alexandre Poirot
aeaa6f03fb Bug 1885054 - [devtools] Display tracer data in a debugger sidebar. r=devtools-reviewers,nchevobbe
Differential Revision: https://phabricator.services.mozilla.com/D204446
2024-07-22 18:49:43 +00:00
Sandor Molnar
4c9152a87a Backed out changeset b6571a4222fb (bug 1885054) for causing bug 1909068 CLOSED TREE 2024-07-22 18:24:44 +03:00
Alexandre Poirot
b823250c65 Bug 1885054 - [devtools] Display tracer data in a debugger sidebar. r=devtools-reviewers,nchevobbe
Differential Revision: https://phabricator.services.mozilla.com/D204446
2024-07-18 09:02:06 +00:00
Emily McMinn
079ff41016 Bug 1724300 - Remove What's New Panel, ToolbarPanelHub components & related tests & references r=desktop-theme-reviewers,omc-reviewers,aminomancer,pdahiya,emilio,devtools-reviewers,firefox-desktop-core-reviewers ,home-newtab-reviewers
Differential Revision: https://phabricator.services.mozilla.com/D201867
2024-03-21 22:05:28 +00:00
Emilio Cobos Álvarez
4db9f9713b Bug 1884294 - Move alert-small / info-small to devtools-shared-images. r=devtools-reviewers,nchevobbe
So that they can be loaded by the highlighters.

Differential Revision: https://phabricator.services.mozilla.com/D204014
2024-03-08 13:19:15 +00:00
Alexandre Poirot
f3effea66c Bug 1853899 - [devtools] Introduce a new Menu Button dedicated to source maps. r=devtools-reviewers,nchevobbe
It allows
* toggling Source Map support entirely
* open the source map URL (when a bundle file is currently selected)
* open the mapped source (original or bundle)
* toggle the "open original source by default" setting
* show source map status (source map error, is it original or bundle file, is this a regular source?)

Differential Revision: https://phabricator.services.mozilla.com/D187577
2024-02-20 11:14:33 +00:00
Narcis Beleuzu
c5b3a44ea3 Backed out changeset 749f10073ff4 (bug 1853899) for dt failure on browser_dbg-sourcemaps-bogus.js CLOSED TREE 2024-02-09 03:56:22 +02:00
Alexandre Poirot
25bbec4417 Bug 1853899 - [devtools] Introduce a new Menu Button dedicated to source maps. r=devtools-reviewers,nchevobbe
It allows
* toggling Source Map support entirely
* open the source map URL (when a bundle file is currently selected)
* open the mapped source (original or bundle)
* toggle the "open original source by default" setting
* show source map status (source map error, is it original or bundle file, is this a regular source?)

Differential Revision: https://phabricator.services.mozilla.com/D187577
2024-02-08 16:43:59 +00:00
Nicolas Chevobbe
383dd70b59 Bug 1826505 - [devtools] Don't vendor react-aria-components anymore. r=devtools-reviewers,ochameau.
This is not used anymore, so we can drop it.
This was the only vendored item that was contributing
to vendor.css, so we can delete the file, and cleanup
our webpack config.

Differential Revision: https://phabricator.services.mozilla.com/D193915
2023-11-23 06:27:43 +00:00
Hubert Boma Manilla
5cccdb525f Bug 1858076 - [devtools] Show notification in the console when original variable mapping is disabled r=devtools-reviewers,nchevobbe
This patch displays a notification in the console when the debugger is paused
in an original file with original variable mapping is turned off.

The notification is displayed when console input is focused, also if the user selects
a generated file, or is no longer paused or switches on varaible mapping, the notification is removed
once the console input is refocused.

Also added support for specific notifications to control displaying the close button.

Differential Revision: https://phabricator.services.mozilla.com/D191568
2023-11-14 07:35:49 +00:00
Hubert Boma Manilla
75543a9d59 Bug 1856270 - [devtools] Refactor the UI experience for loading original scopes r=devtools-reviewers,ochameau,nchevobbe,perftest-reviewers,sparky
Highlights of this patch
- Only show original scopes when map scopes is enabled
- When map scopes is disabled or when original content is still loading show useful info in the scopes panel for the user.
- Removed styling to make all the side panel content consistent.

Differential Revision: https://phabricator.services.mozilla.com/D189735
2023-11-14 07:35:47 +00:00
Sandor Molnar
3f48748109 Backed out 4 changesets (bug 1858652, bug 1858032, bug 1858076, bug 1856270) for causing talos damp failures
Backed out changeset 569b6a151d3e (bug 1858076)
Backed out changeset 3c66fbe494de (bug 1858652)
Backed out changeset 966ce8c9e339 (bug 1858032)
Backed out changeset 3e08029e8f01 (bug 1856270)
2023-11-08 07:21:25 +02:00
Hubert Boma Manilla
678b32f0df Bug 1858076 - [devtools] Show notification in the console when original variable mapping is disabled r=devtools-reviewers,nchevobbe
This patch displays a notification in the console when the debugger is paused
in an original file with original variable mapping is turned off.

The notification is displayed when console input is focused, also if the user selects
a generated file, or is no longer paused or switches on varaible mapping, the notification is removed
once the console input is refocused.

Also added support for specific notifications to control displaying the close button.

Differential Revision: https://phabricator.services.mozilla.com/D191568
2023-11-07 19:51:31 +00:00
Hubert Boma Manilla
6f26e1e363 Bug 1856270 - [devtools] Refactor the UI experience for loading original scopes r=devtools-reviewers,ochameau,nchevobbe
Highlights of this patch
- Only show original scopes when map scopes is enabled
- When map scopes is disabled or when original content is still loading show useful info in the scopes panel for the user.
- Removed styling to make all the side panel content consistent.

Differential Revision: https://phabricator.services.mozilla.com/D189735
2023-11-07 19:51:30 +00:00
Iulian Moraru
0359529a07 Backed out 4 changesets (bug 1856270, bug 1858652, bug 1858032, bug 1858076) for causing dt failures on browser_dbg-reducer-cleanup-on-target-removal.js. CLOSED TREE
Backed out changeset 492052d366fe (bug 1858076)
Backed out changeset a5a765a10588 (bug 1858652)
Backed out changeset 7797ecc20515 (bug 1858032)
Backed out changeset b8a3c66f7fdc (bug 1856270)
2023-11-06 17:54:12 +02:00
Hubert Boma Manilla
6232077736 Bug 1858076 - [devtools] Show notification in the console when original variable mapping is disabled r=devtools-reviewers,nchevobbe
This patch displays a notification in the console when the debugger is paused
in an original file with original variable mapping is turned off.

The notification is displayed when console input is focused, also if the user selects
a generated file, or is no longer paused or switches on varaible mapping, the notification is removed
once the console input is refocused.

Also added support for specific notifications to control displaying the close button.

Differential Revision: https://phabricator.services.mozilla.com/D191568
2023-11-06 14:00:53 +00:00
Hubert Boma Manilla
44be490ff5 Bug 1856270 - [devtools] Refactor the UI experience for loading original scopes r=devtools-reviewers,ochameau,nchevobbe
Highlights of this patch
- Only show original scopes when map scopes is enabled
- When map scopes is disabled or when original content is still loading show useful info in the scopes panel for the user.
- Removed styling to make all the side panel content consistent.

Differential Revision: https://phabricator.services.mozilla.com/D189735
2023-11-06 14:00:52 +00:00
Nicolas Chevobbe
fb52bbafba Bug 1862144 - [devtools] Remove A11yIntention component. r=devtools-reviewers,bomsy.
It doesn't do anything we could do with CSS only and might actually cause
issues with our focus indicators.

Depends on D191738

Differential Revision: https://phabricator.services.mozilla.com/D192308
2023-11-02 06:15:37 +00:00
Emilio Cobos Álvarez
bd1b363ff9 Bug 1824886 - Rewrite AnonymousContent to use a shadow tree. r=smaug,TYLin,sfoster,devtools-reviewers,nchevobbe
Subtle things:

 * We now have shadow trees in NAC, inception! Only a couple lines of
   code in the style system had to be changed to match :host rules and
   so properly.

 * Had to make highlighters.css contentaccessible, because otherwise we
   can't load it from the shadow tree. I don't think it's a big deal.

 * I removed some of the code from highlighters.css that claimed that
   stuff inherited from the html element. That's just no longer true.

 * Had to switch from `setAttribute("style", ...)` to `.style = ...;`.
   This is needed because CSSOM from chrome code bypasses CSP (as
   AnonymousContent did), but setAttribute() doesn't, see bug 1424474.

Differential Revision: https://phabricator.services.mozilla.com/D173998
2023-07-09 11:34:05 +00:00
Stanca Serban
15c8455338 Backed out changeset 1c835137f532 (bug 1824886) for causing multiple failures. CLOSED TREE 2023-07-06 02:11:59 +03:00
Emilio Cobos Álvarez
3338831b84 Bug 1824886 - Rewrite AnonymousContent to use a shadow tree. r=smaug,TYLin,sfoster,devtools-reviewers,nchevobbe
Subtle things:

 * We now have shadow trees in NAC, inception! Only a couple lines of
   code in the style system had to be changed to match :host rules and
   so properly.

 * Had to make highlighters.css contentaccessible, because otherwise we
   can't load it from the shadow tree. I don't think it's a big deal.

 * I removed some of the code from highlighters.css that claimed that
   stuff inherited from the html element. That's just no longer true.

 * Had to switch from `setAttribute("style", ...)` to `.style = ...;`.
   This is needed because CSSOM from chrome code bypasses CSP (as
   AnonymousContent did), but setAttribute() doesn't, see bug 1424474.

Differential Revision: https://phabricator.services.mozilla.com/D173998
2023-07-05 21:59:44 +00:00
Cosmin Sabou
fb92d5f42f Backed out 2 changesets (bug 1824886) for causing failures on browser_all_files_referenced.js. CLOSED TREE
Backed out changeset 650ef77d1c4d (bug 1824886)
Backed out changeset 1c835137f532 (bug 1824886)
2023-07-06 00:42:53 +03:00
Emilio Cobos Álvarez
454c0b932e Bug 1824886 - Rewrite AnonymousContent to use a shadow tree. r=smaug,TYLin,sfoster,devtools-reviewers,nchevobbe
Subtle things:

 * We now have shadow trees in NAC, inception! Only a couple lines of
   code in the style system had to be changed to match :host rules and
   so properly.

 * Had to make highlighters.css contentaccessible, because otherwise we
   can't load it from the shadow tree. I don't think it's a big deal.

 * I removed some of the code from highlighters.css that claimed that
   stuff inherited from the html element. That's just no longer true.

 * Had to switch from `setAttribute("style", ...)` to `.style = ...;`.
   This is needed because CSSOM from chrome code bypasses CSP (as
   AnonymousContent did), but setAttribute() doesn't, see bug 1424474.

Differential Revision: https://phabricator.services.mozilla.com/D173998
2023-07-05 18:21:31 +00:00
Alexandre Poirot
be2c2a346f Bug 1837048 - [devtools] Remove preffed-off highlight calls feature. r=devtools-reviewers,bomsy
Differential Revision: https://phabricator.services.mozilla.com/D180193
2023-06-12 10:01:55 +00:00
Alexandre Poirot
3093557b91 Bug 1831149 - [devtools] Fold ManagedTree into SourcesTree. r=bomsy
This help avoid this unecessary indirection which is ManagedTree.
And hopefully this may help simplify "expanded" and "focused" data
which is currently stored in redux but rather looks like something
that belongs to this React component's state.

Differential Revision: https://phabricator.services.mozilla.com/D177066
2023-05-22 16:09:34 +00:00
Nicolas Chevobbe
3d0c4ffa20 Bug 1832810 - [devtools] Tweak style of MDN links in Compatibility panel. r=jdescottes,devtools-reviewers.
We make the link underlined, and we display a mdn icon next to it.

Differential Revision: https://phabricator.services.mozilla.com/D177912
2023-05-17 05:26:31 +00:00
Alexandre Poirot
8bf6d69ee3 Bug 1830240 - [devtools] Rename SearchBar to SearchInFileBar. r=devtools-reviewers,bomsy
As we have various search component, it would help to be slightly more precise
and this helps connect this component with "file" search selectors, reducers, actions.

Differential Revision: https://phabricator.services.mozilla.com/D176669
2023-05-09 10:42:51 +00:00
Hubert Boma Manilla
048b95a8df Bug 1811000 - [devtools] Move the project search to the side panel r=devtools-reviewers,nchevobbe,perftest-reviewers,sparky
Differential Revision: https://phabricator.services.mozilla.com/D172985
2023-03-29 18:43:01 +00:00
Sandor Molnar
ab42dc3bba Backed out changeset b93a5fd6ab2c (bug 1811000) for causing talos failures. CLOSED TREE 2023-03-27 20:32:08 +03:00
Hubert Boma Manilla
4b59ce7061 Bug 1811000 - [devtools] Move the project search to the side panel r=devtools-reviewers,nchevobbe
Differential Revision: https://phabricator.services.mozilla.com/D172985
2023-03-27 13:45:40 +00:00
Alexandre Poirot
5cb1635d84 Bug 1803616 - [devtools] Implement a Javascript Tracer in the debugger. r=devtools-reviewers,devtools-backward-compat-reviewers,nchevobbe
On the server side, this patch introduces:
* a new "tracer" Target Scope actor to start and stop tracing per target
* a new TRACING_STATE resource in order to report to the client when we start/stop tracing and with which log method.

On the frontend side, this patch introduces:
* a global tracer button, which will enable/disable tracing for all targets/threads
  all at once.
* a global header, similar to pause, reporting if any target is tracing or not.
The header reuses the pause header and we may want to followup to better coordinate case
where we pause and trace at the same time. Only one of the two states is displayed.

We may want to followup here to be able to trace only one target and see the state per target.

Differential Revision: https://phabricator.services.mozilla.com/D163614
2023-03-14 17:17:05 +00:00
Nicolas Chevobbe
4a124d54ef Bug 1773549 - [devtools] Add widget for linear easing function. r=devtools-reviewers,jdescottes.
This adds an authoring tool similar to the one we have for cubic bezier.
A swatch is added next to `linear()` functions and clicking on it will show
the widget.
Modifying the line in the widget will update the value of the `linear()` function
in the rule view.

Differential Revision: https://phabricator.services.mozilla.com/D153333
2023-03-08 17:39:37 +00:00
Hubert Boma Manilla
835172e052 Bug 1811004 - [devtools] Make the search modifiers a shared component r=ochameau
This patch makes the search modifiers a shared component so we can
use this for project search.

Differential Revision: https://phabricator.services.mozilla.com/D168931
2023-02-20 08:48:59 +00:00
Julien Wajsberg
a9521c9c6c Bug 1811853 - Move the panel related files to a new subdirectory panel/ r=canaltinova
Differential Revision: https://phabricator.services.mozilla.com/D167850
2023-02-13 16:46:41 +00:00
nchevobbe
bce2a203a8 Bug 1795876 - [devtools] Change toggle selector highlighter icon. r=jdescottes.
Differential Revision: https://phabricator.services.mozilla.com/D159686
2022-10-19 15:57:08 +00:00
Alexandre Poirot
7e5e4979c4 Bug 1780912 - [devtools] Make the local web extension toolbox be always on top. r=nchevobbe,eemeli
This allows to keep the DevTools visible while interacting with the Firefox
window where the extension is running.

This behavior is enabled by default, but can be disable on-demand via a button
in the top toolbar.
Note that it requires to close and reopen the window/toolbox as platform APIs
disallow changing this behavior "live" on a given window.

Differential Revision: https://phabricator.services.mozilla.com/D155843
2022-09-15 07:31:31 +00:00
Noemi Erli
539bbccc5d Backed out 2 changesets (bug 1780912) for causing failures in browser_aboutdebugging_addons_debug_console.js CLOSED TREE
Backed out changeset d2360a2a3d70 (bug 1780912)
Backed out changeset 21f1e9031be0 (bug 1780912)
2022-09-08 01:59:18 +03:00
Alexandre Poirot
80c9c861dd Bug 1780912 - [devtools] Make the local web extension toolbox be always on top. r=nchevobbe,eemeli
This allows to keep the DevTools visible while interacting with the Firefox
window where the extension is running.

This behavior is enabled by default, but can be disable on-demand via a button
in the top toolbar.
Note that it requires to close and reopen the window/toolbox as platform APIs
disallow changing this behavior "live" on a given window.

Differential Revision: https://phabricator.services.mozilla.com/D155843
2022-09-07 21:00:17 +00:00
Julian Descottes
cbd1e77ab4 Bug 1789580 - [devtools] StyleEditor should use a different icon when the stylesheet is disabled r=nchevobbe
Only modified the provided SVGs to have fill-context & fill-stroke for colors.

Differential Revision: https://phabricator.services.mozilla.com/D156655
2022-09-07 15:23:06 +00:00
Nicolas Chevobbe
450f0ae918 Bug 1784467 - [devtools] Add icon for select node in accessibility tree. r=jdescottes.
Since the select node in inspector simply re-use the inspector icon,
we're doing here, using a sized-down version of the a11y panel icon.

Differential Revision: https://phabricator.services.mozilla.com/D154509
2022-08-12 13:10:05 +00:00
Nicolas Chevobbe
6f8744625b Bug 1776853 - [devtools] Add BrowserToolbox-specific toolbar. r=ochameau,jdescottes,fluent-reviewers,flod.
This adds an additional toolbar in the Browser Toolbox which will contain specific
tools and options.
At the moment we only display a couple input button to be able to switch from
Parent process only to multiprocess mode.
We remove the similar UI in the iframe picker and adapt the existing test.

A trait is added to not show the toolbar when debugging server where we wouldn't
get the `isSwitchingMode` property in `onTargetDestroyed`, as this can cause
misbehavior in various tool when switching between different modes.

Differential Revision: https://phabricator.services.mozilla.com/D150575
2022-08-01 09:33:17 +00:00
Bogdan Szekely
f9a469e37c Backed out 5 changesets (bug 1777253, bug 1776853) for causing devtools failures on browser_console_modes.js CLOSED TREE
Backed out changeset dbd68a657d6f (bug 1777253)
Backed out changeset f401150fb142 (bug 1776853)
Backed out changeset 209aade12fc6 (bug 1776853)
Backed out changeset dc24308ff8ef (bug 1776853)
Backed out changeset ea98ed858bae (bug 1776853)
2022-08-01 10:43:40 +03:00
Nicolas Chevobbe
9d9761ff10 Bug 1776853 - [devtools] Add BrowserToolbox-specific toolbar. r=ochameau,jdescottes,fluent-reviewers,flod.
This adds an additional toolbar in the Browser Toolbox which will contain specific
tools and options.
At the moment we only display a couple input button to be able to switch from
Parent process only to multiprocess mode.
We remove the similar UI in the iframe picker and adapt the existing test.

A trait is added to not show the toolbar when debugging server where we wouldn't
get the `isSwitchingMode` property in `onTargetDestroyed`, as this can cause
misbehavior in various tool when switching between different modes.

Differential Revision: https://phabricator.services.mozilla.com/D150575
2022-08-01 06:55:54 +00:00
Hubert Boma Manilla
5f3826ec54 Bug 1778113 - Use thicker icons for Watch expressions and breakpoints r=nchevobbe
Differential Revision: https://phabricator.services.mozilla.com/D151023
2022-07-06 21:57:31 +00:00
Nicolas Chevobbe
b5a99726eb Bug 1668219 - [devtools] Remove widgets only used on old perf panel. r=bomsy.
This removes FlameGraph.js, Graphs.js, GraphsWorker.js, LineGraphWidget.js, MountainGraphWidget.js
and their related tests.
DevToolsWorker tests that were relying on GraphsWorker.js are updated to use a test-specific blob worker.

Differential Revision: https://phabricator.services.mozilla.com/D145462
2022-05-06 17:16:27 +00:00
Nicolas Chevobbe
30db855c84 Bug 1668219 - [devtools] Remove devtools/client/performance. r=julienw.
Differential Revision: https://phabricator.services.mozilla.com/D145461
2022-05-06 17:16:27 +00:00
Nicolas Chevobbe
06e5af3c22 Bug 1765239 - [devtools] Move SplitView.jsm to styleeditor. r=ochameau.
It's only used here, so let's move it back here

Differential Revision: https://phabricator.services.mozilla.com/D144148
2022-04-20 15:47:05 +00:00