Commit Graph
5902 Commits
Author SHA1 Message Date
ozcoder 7679883d8a Bug 2020186 - [har] Add support for _priority / _initialPriority custom fields. r=jdescottes,devtools-reviewers,necko-reviewers
Added a tooltip to the Priority column to show final priority.
Added getRequestPriorityAsNumber function to convert HAR priority string back into a value.
Update representation of the priority value of a resource when it changes.
Added field priority to HAR file export and importing.
Linting wants double quotes.
priority **needs** to be gathered first before assigning into resource.
Check if the priority has changed to decide if Priority should re-render.
Added test to check priority of images above and below the fold.
Changed test HTML file into a hand written example with some explanation.
Guard against null priority when updating it.
Use isInteger to check priority is really a number.
We do not want to overwrite priority with "undefined".
Defer setting up the first test image a little bit, but do not check its priority.
Test HTML needs to be large.

Differential Revision: https://phabricator.services.mozilla.com/D287020
2026-06-12 10:08:50 +00:00
Alexandru Marc 8d0d6fdd91 Revert "Bug 2044644 - Turn on clang-tidy's bugprone-forwarding-reference-overload r=sylvestre" for causing sm bustages @ Simulator-vixl.h
This reverts commit 80834d18d5.

Revert "Bug 2044644 - Make HeapSnapshot.cpp compatible with bugprone-forwarding-reference-overload check r=jandem"

This reverts commit 81ddd9da57.

Revert "Bug 2044644 - Make http/Capsule.h compatible with bugprone-forwarding-reference-overload check r=necko-reviewers,valentin"

This reverts commit 5443d86b53.

Revert "Bug 2044644 - Make xpcom/threads/nsThreadUtils.h compatible with bugprone-forwarding-reference-overload check r=xpcom-reviewers,nika"

This reverts commit 912dd4e34d.

Revert "Bug 2044644 - Use standard [[maybe_unused]] instead of custom approach to inform the compiler about potential usage of a variable r=spidermonkey-reviewers,jandem"

This reverts commit 0ea4c039c9.
2026-06-12 08:03:08 +00:00
Nicolas Chevobbe 7b550220da Bug 2043977 - [devtools] Add StyleRuleActor.getCssExplainersData. r=devtools-reviewers,devtools-backward-compat-reviewers,ochameau.
Differential Revision: https://phabricator.services.mozilla.com/D303983
2026-06-12 07:25:31 +00:00
serge-sans-paille 81ddd9da57 Bug 2044644 - Make HeapSnapshot.cpp compatible with bugprone-forwarding-reference-overload check r=jandem
More about the check in the documentation^[0], but this basically allow
for better move & copy constructor in some subtle situations that do
occur in our code base (I tried to just delete those and the build fails).

[0] https://clang.llvm.org/extra/clang-tidy/checks/bugprone/forwarding-reference-overload.html

Differential Revision: https://phabricator.services.mozilla.com/D304426
2026-06-12 07:23:18 +00:00
Keith Cirkel 0a230dd2c8 Bug 2044945 - Add <select> pseudo elements to devtools style inspector r=devtools-reviewers,nchevobbe
Differential Revision: https://phabricator.services.mozilla.com/D304688
2026-06-12 02:15:10 +00:00
canalun 88326e4dd9 Bug 1655941 - Add storage.sync support to the extension storage inspector. r=nchevobbe,devtools-backward-compat-reviewers,devtools-reviewers
Display storage.sync items alongside storage.local items in the extension
storage inspector.

Because storage.local and storage.sync can each contain an item with the same
name, the item name alone is no longer unique. Use a compound uniqueKey
consisting of the item name and storage area to disambiguate them within the
same table. This approach follows the one used by the cookies storage actor.

The Storage Inspector client automatically uses the first field returned by
getFields() as the unique identifier for table rows, so exposing the compound
uniqueKey there lets existing interfaces such as getValuesForHost(),
editItem(), and removeItem() receive it without changes to their external
signatures.

Differential Revision: https://phabricator.services.mozilla.com/D304326
2026-06-11 14:18:50 +00:00
Eden Chuang c28bc7e5ca Bug 2040092 - Set animationsPlayBackRateMultiplier on the top browsing context from DevTools. r=devtools-reviewers,nchevobbe
animationsPlayBackRateMultiplier is a synced BrowsingContext field whose CanSet
requires IsTop(). When a root target actor (e.g. the Browser Toolbox) has an
iframe selected as the currently targeted document via the iframe picker,
this.browsingContext is no longer the top context, so setting the multiplier on
it crashed with "CanSet failed for field(s): AnimationsPlayBackRateMultiplier".

Set the field on this.browsingContext.top instead, which matches the field's
getter (which reads from Top()) and is a no-op for the regular tab toolbox where
the targeted context is already top.

Differential Revision: https://phabricator.services.mozilla.com/D305790
2026-06-10 19:52:24 +00:00
Silaghi 98bf5b0321 Revert "Bug 204494, Bug 2041004 - Add <select> pseudo elements to devtools style inspector r=devtools-reviewers,nchevobbe"
This reverts commit 739a225b8a.

Revert "Bug 2041004 - Make ::picker-icon generated content, add UA styles r=layout-reviewers,firefox-style-system-reviewers,emilio"

This reverts commit 4b7b2a4083.
2026-06-10 16:55:02 +00:00
Keith Cirkel 739a225b8a Bug 2044945 - Add <select> pseudo elements to devtools style inspector r=devtools-reviewers,nchevobbe
Differential Revision: https://phabricator.services.mozilla.com/D304688
2026-06-10 15:27:47 +00:00
Alexandre Poirot 5c80ff3304 Bug 2031714 - [devtools] Implement "Local Mode" backend. r=devtools-reviewers,devtools-backward-compat-reviewers,bomsy,nchevobbe
This allows opening local files from https via DevTools.

Differential Revision: https://phabricator.services.mozilla.com/D294192
2026-06-10 12:46:16 +00:00
Alexandre Poirot 6335531d2f Bug 2031714 - [devtools] Expose a flag on the TargetFront to know if the page is an error page. r=devtools-reviewers,nchevobbe
This will then help know if Local Mode logic should reload the page or
not if we register a mapping while the currently debugged page has the
same origin **and** is an error page.

Differential Revision: https://phabricator.services.mozilla.com/D302981
2026-06-10 12:46:13 +00:00
Silaghi 50913f8313 Revert "Bug 2044945, Bug 2041004 - Add <select> psuedo elements to devtools style inspector r=devtools-reviewers,nchevobbe" for causing wr failures at select-appearance-disabled.html
This reverts commit 6b8775f47f.

Revert "Bug 2041004 - Make ::picker-icon generated content, add UA styles r=layout-reviewers,firefox-style-system-reviewers,emilio"

This reverts commit e3529d76f3.
2026-06-10 10:08:51 +00:00
Keith Cirkel 6b8775f47f Bug 2044945 - Add <select> psuedo elements to devtools style inspector r=devtools-reviewers,nchevobbe
Differential Revision: https://phabricator.services.mozilla.com/D304688
2026-06-10 08:20:23 +00:00
Alexandru Marc 85aff4e02f Revert "Bug 2031714 - [devtools] Implement "Local Mode" backend. r=devtools-reviewers,devtools-backward-compat-reviewers,bomsy,nchevobbe" for causing Android build bustages
This reverts commit f2d8553f79.

Revert "Bug 2031714 - [devtools] Expose a flag on the TargetFront to know if the page is an error page. r=devtools-reviewers,nchevobbe"

This reverts commit 0ef861f993.
2026-06-10 01:13:30 +00:00
Alexandre Poirot f2d8553f79 Bug 2031714 - [devtools] Implement "Local Mode" backend. r=devtools-reviewers,devtools-backward-compat-reviewers,bomsy,nchevobbe
This allows opening local files from https via DevTools.

Differential Revision: https://phabricator.services.mozilla.com/D294192
2026-06-09 20:35:02 +00:00
Alexandre Poirot 0ef861f993 Bug 2031714 - [devtools] Expose a flag on the TargetFront to know if the page is an error page. r=devtools-reviewers,nchevobbe
This will then help know if Local Mode logic should reload the page or
not if we register a mapping while the currently debugged page has the
same origin **and** is an error page.

Differential Revision: https://phabricator.services.mozilla.com/D302981
2026-06-09 20:34:59 +00:00
Alexandru Marc ca229e12ac Revert (Bug 2044945, Bug 2041004) for causing wr failures @ select-explicit-size.html
This reverts commit b434b10988.

Revert "Bug 2041004 - Make ::picker-icon generated content, add UA styles r=layout-reviewers,firefox-style-system-reviewers,emilio"

This reverts commit e92bda8433.
2026-06-09 19:14:13 +00:00
Keith Cirkel b434b10988 Bug 2044945 - Add <select> psuedo elements to devtools style inspector r=devtools-reviewers,nchevobbe
Differential Revision: https://phabricator.services.mozilla.com/D304688
2026-06-09 17:37:49 +00:00
Nicolas Chevobbe 73330bebe6 Bug 2045975 - [devtools] Update MDN compat data for 153 cycle. r=devtools-reviewers,bomsy.
Differential Revision: https://phabricator.services.mozilla.com/D305472
2026-06-09 08:57:42 +00:00
Sandor Molnar 697ae9e2f9 Revert "Bug 2044945 - Add <select> psuedo elements to devtools style inspector r=devtools-reviewers,nchevobbe"
This reverts commit ef5241b45b.
2026-06-08 17:32:42 +00:00
Keith Cirkel ef5241b45b Bug 2044945 - Add <select> psuedo elements to devtools style inspector r=devtools-reviewers,nchevobbe
Differential Revision: https://phabricator.services.mozilla.com/D304688
2026-06-08 16:00:38 +00:00
agoloman 02843a95cb Revert "Bug 2031714 - [devtools] Implement "Local Mode" backend. r=devtools-reviewers,devtools-backward-compat-reviewers,bomsy" for causing bc failures @browser_devtools.js.
This reverts commit 13896a724c.

Revert "Bug 2031714 - [devtools] Expose a flag on the TargetFront to know if the page is an error page. r=devtools-reviewers,nchevobbe"

This reverts commit d6ca9ef038.
2026-06-04 19:14:21 +00:00
Alexandre Poirot 13896a724c Bug 2031714 - [devtools] Implement "Local Mode" backend. r=devtools-reviewers,devtools-backward-compat-reviewers,bomsy
This allows opening local files from https via DevTools.

Differential Revision: https://phabricator.services.mozilla.com/D294192
2026-06-04 10:28:10 +00:00
Alexandre Poirot d6ca9ef038 Bug 2031714 - [devtools] Expose a flag on the TargetFront to know if the page is an error page. r=devtools-reviewers,nchevobbe
This will then help know if Local Mode logic should reload the page or
not if we register a mapping while the currently debugged page has the
same origin **and** is an error page.

Differential Revision: https://phabricator.services.mozilla.com/D302981
2026-06-04 10:28:07 +00:00
Julian Descottes 217a06bb62 Bug 2041421 - [devtools] Avoid leaks in devtools network-observer mochitests r=devtools-reviewers,nchevobbe
A failing test with fail-if was throwing midway and leaving unhandled promises and creating leaks.
Move to skip-if.
The auth listener test seems to also create frequent leaks, skip it on ASAN.

Differential Revision: https://phabricator.services.mozilla.com/D304370
2026-06-03 16:24:20 +00:00
Norisz Fay 1a8f4c9241 Revert "Bug 2031714 - [devtools] Implement "Local Mode" backend. r=devtools-reviewers,devtools-backward-compat-reviewers,bomsy" for causing dt failure @ browser_networkobserver_localMode.js
This reverts commit 78970467d8.

Revert "Bug 2031714 - [devtools] Expose a flag on the TargetFront to know if the page is an error page. r=devtools-reviewers,nchevobbe"

This reverts commit 6b862813ef.
2026-06-03 13:57:44 +00:00
Alexandre Poirot 78970467d8 Bug 2031714 - [devtools] Implement "Local Mode" backend. r=devtools-reviewers,devtools-backward-compat-reviewers,bomsy
This allows opening local files from https via DevTools.

Differential Revision: https://phabricator.services.mozilla.com/D294192
2026-06-03 13:02:35 +00:00
Alexandre Poirot 6b862813ef Bug 2031714 - [devtools] Expose a flag on the TargetFront to know if the page is an error page. r=devtools-reviewers,nchevobbe
This will then help know if Local Mode logic should reload the page or
not if we register a mapping while the currently debugged page has the
same origin **and** is an error page.

Differential Revision: https://phabricator.services.mozilla.com/D302981
2026-06-03 13:02:33 +00:00
Serban Stanca d976ff2d6e Revert "Bug 2020186 - [har] Add support for _priority / _initialPriority custom fields. r=jdescottes,devtools-reviewers,necko-reviewers" for causing devtools failures in browser_net_har_image_priority.js.
This reverts commit ec557789a0.
2026-06-02 21:27:02 +00:00
ozcoder ec557789a0 Bug 2020186 - [har] Add support for _priority / _initialPriority custom fields. r=jdescottes,devtools-reviewers,necko-reviewers
Added a tooltip to the Priority column to show final priority.
Added getRequestPriorityAsNumber function to convert HAR priority string back into a value.
Update representation of the priority value of a resource when it changes.
Added field priority to HAR file export and importing.
Linting wants double quotes.
priority **needs** to be gathered first before assigning into resource.
Check if the priority has changed to decide if Priority should re-render.
Added test to check priority of images above and below the fold.
Changed test HTML file into a hand written example with some explanation.
Guard against null priority when updating it.
Use isInteger to check priority is really a number.
We do not want to overwrite priority with "undefined".

Differential Revision: https://phabricator.services.mozilla.com/D287020
2026-06-02 19:33:12 +00:00
Tom Schuster 13e7ca66b5 Bug 2044268 - Remove test-only workerify function. r=devtools-reviewers,nchevobbe
Differential Revision: https://phabricator.services.mozilla.com/D304018
2026-06-02 11:04:45 +00:00
Julian Descottes aa80c68700 Bug 2043123 - [devtools] Update failure message for DevTools js object generated tests r=ochameau,devtools-reviewers
Differential Revision: https://phabricator.services.mozilla.com/D302978
2026-06-02 10:56:47 +00:00
agoloman d69d1d8e0e Revert "Bug 2031717, Bug 2031714 - [devtools] Expose local mode settings in DevTools options panel. r=devtools-reviewers,fluent-reviewers,bolsson,nchevobbe" for causing bc failures @browser_misused_characters_in_strings.js.
This reverts commit b7f48700d7.

Revert "Bug 2031714 - [devtools] Implement "Local Mode" backend. r=devtools-reviewers,devtools-backward-compat-reviewers,bomsy"

This reverts commit 0f1b6c66ab.

Revert "Bug 2031714 - [devtools] Expose a flag on the TargetFront to know if the page is an error page. r=devtools-reviewers,nchevobbe"

This reverts commit 4cbb3ac689.
2026-06-02 09:35:20 +00:00
Hubert B Manilla 54503735e3 Bug 2043924 - [devtools] Assert that a method which expects a bulk response throws when a value is returned r=devtools-reviewers,ochameau
Differential Revision: https://phabricator.services.mozilla.com/D303666
2026-06-02 09:18:13 +00:00
Hubert B Manilla 6c0d6e099d Bug 2043924 - [devtools] Improve the error message for a method which expect bulk response but return a value r=devtools-reviewers,ochameau
Differential Revision: https://phabricator.services.mozilla.com/D303667
2026-06-02 09:18:10 +00:00
Alexandre Poirot 0f1b6c66ab Bug 2031714 - [devtools] Implement "Local Mode" backend. r=devtools-reviewers,devtools-backward-compat-reviewers,bomsy
This allows opening local files from https via DevTools.

Differential Revision: https://phabricator.services.mozilla.com/D294192
2026-06-02 08:42:02 +00:00
Alexandre Poirot 4cbb3ac689 Bug 2031714 - [devtools] Expose a flag on the TargetFront to know if the page is an error page. r=devtools-reviewers,nchevobbe
This will then help know if Local Mode logic should reload the page or
not if we register a mapping while the currently debugged page has the
same origin **and** is an error page.

Differential Revision: https://phabricator.services.mozilla.com/D302981
2026-06-02 08:42:00 +00:00
Atila Butkovits b58730abd8 Revert "Bug 2043924 - [devtools] Assert that a method which expects a bulk response throws when a value is returned r=devtools-reviewers,ochameau" for causing failures at test_bulk_error.js.
This reverts commit cc333d322a.

Revert "Bug 2043924 - [devtools] Improve the error message for a method which expect bulk response but return a value r=devtools-reviewers,ochameau"

This reverts commit 19e0d76011.
2026-06-01 18:20:21 +00:00
Hubert B Manilla cc333d322a Bug 2043924 - [devtools] Assert that a method which expects a bulk response throws when a value is returned r=devtools-reviewers,ochameau
Differential Revision: https://phabricator.services.mozilla.com/D303666
2026-06-01 16:00:40 +00:00
Hubert B Manilla 19e0d76011 Bug 2043924 - [devtools] Improve the error message for a method which expect bulk response but return a value r=devtools-reviewers,ochameau
Differential Revision: https://phabricator.services.mozilla.com/D303667
2026-06-01 16:00:38 +00:00
Antonette Caldwell 4c36030555 Bug 2031735 - Throw Error instead of Actor for invalid bulk response return value in protocol Actor. r=devtools-reviewers,bomsy
Differential Revision: https://phabricator.services.mozilla.com/D302684
2026-05-31 20:11:15 +00:00
Rob Wu e723c97c5a Bug 2034168 - Enforce file access permission in devtools_page r=zombie,devtools-reviewers,ochameau
Differential Revision: https://phabricator.services.mozilla.com/D302931
2026-05-29 23:22:00 +00:00
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
Alexandre Poirot d41c6f61d1 Bug 2041661 - [devtools] Gate RDP packet logging in the transport layer. r=devtools-reviewers,jdescottes
This incoming variable is the JSON packet as a string,
and so is possibly large and forces to create a new string
even if we aren't logging.

Differential Revision: https://phabricator.services.mozilla.com/D302982
2026-05-28 12:50:37 +00:00
Nicolas Chevobbe 7cfad5d34c Bug 2014751 - [devtools] Handle modern attr() syntax in OutputParser. r=devtools-reviewers,ochameau.
Check if the attribute value does match the definedd attr() attr-type parameter
to see if the attribute value or the fallback is being used.
In the attr() tooltip, indicate when the attribute value doesn't match the specified
type.

Differential Revision: https://phabricator.services.mozilla.com/D301490
2026-05-23 08:05:32 +00:00
Nicolas Chevobbe 9d79ade0e0 Bug 2040393 - [devtools] Remove 151 backward compatibility code for the Rules view. r=devtools-reviewers,devtools-backward-compat-reviewers,jdescottes.
Differential Revision: https://phabricator.services.mozilla.com/D301460
2026-05-21 08:45:03 +00:00
Nicolas Chevobbe 336b659dbd Bug 2040393 - [devtools] Remove 151 backward compatibility code for the animation panel. r=devtools-reviewers,devtools-backward-compat-reviewers,jdescottes.
Differential Revision: https://phabricator.services.mozilla.com/D301458
2026-05-21 08:45:00 +00:00
Ryan VanderMeulen 47fd6ce131 Bug 1989886 - Update protobuf to v35.0. r=firefox-build-system-reviewers,timhuang,sergesanspaille
- Re-vendored protobuf at v35.0 alongside third_party/utf8_range, a
  build-time dependency pulled in by extension_set_inl.h. Expanded
  moz.yaml excludes to cover the reorganized upstream layout (json/,
  util/, bridge/, compiler/, plus test/build-config files leaked past
  the default exclude pattern), and dropped the now-upstreamed
  remove_iostream_header.patch.
- Added /third_party/abseil-cpp to chromium-config.mozbuild's
  LOCAL_INCLUDES so transitive absl/* includes resolve in every
  protobuf consumer that includes chromium-config; added direct
  LOCAL_INCLUDES on the few that don't.
- Added a pragma to abseil's check_op.h to suppress sign-compare in
  CHECK_OP macros, scoping the suppression to translation units that
  include this header rather than globally.
- Added USE_LIBS += ["abseil"] to the content_analysis_sdk_agent
  Program() so it can link absl symbols.
- Audited moz.build SOURCES and EXPORTS against actual usage and
  removed full-runtime files (well-known type .pb.cc/.pb.h,
  reflection_ops, dynamic_message, text_format, descriptor, etc.)
  that our lite-mode generated code doesn't reach.
- Taught generate_files.py to update gradle/libs.versions.toml's
  `protobuf` (Java/Maven) value to match the vendored C++ revision
  on future re-vendors.
- Suppressed nodiscard warnings on pre-existing SerializeToString
  calls in url-classifier with (void) casts.

Differential Revision: https://phabricator.services.mozilla.com/D300213
2026-05-20 20:19:02 +00:00
Alexandre Poirot 81fd28204b Bug 1622857 - [devtools] Remove now unused dumpv/verbose logging logic. r=devtools-reviewers,nchevobbe
The transport layer was the only one using this.

Differential Revision: https://phabricator.services.mozilla.com/D299058
2026-05-18 09:12:41 +00:00
Alexandre Poirot ff764d01fd Bug 1622857 - [devtools] Use a console instance for logging DevToolsWorker messages. r=devtools-reviewers,nchevobbe
This the last usage of "wantVerbose" flag.

Differential Revision: https://phabricator.services.mozilla.com/D299057
2026-05-18 09:12:37 +00:00