searching for reviewer(ochameau)
93dffd0aaf9738e3c766fad7b60851142a581d2f: Bug 1775071 - [devtools] Guard profiler markers for DevTools actor events behind a profiler.IsActive check r=ochameau
Julian Descottes <jdescottes@mozilla.com> - Wed, 06 Jul 2022 09:38:53 +0000 - rev 623108
Push 39943 by nbeleuzu@mozilla.com at Wed, 06 Jul 2022 15:15:42 +0000
Bug 1775071 - [devtools] Guard profiler markers for DevTools actor events behind a profiler.IsActive check r=ochameau DevTools actor events can be a very hot path and we should avoid computing the marker label if possible. Differential Revision: https://phabricator.services.mozilla.com/D150996
3c57ee2b7602d32d15492d4a22aeb3a497331ff0: Bug 1776145 - [devtools] Remove WebConsoleConnectionProxy. r=ochameau.
Nicolas Chevobbe <nchevobbe@mozilla.com> - Mon, 04 Jul 2022 13:27:52 +0000 - rev 622925
Push 39935 by ctuns@mozilla.com at Mon, 04 Jul 2022 21:42:52 +0000
Bug 1776145 - [devtools] Remove WebConsoleConnectionProxy. r=ochameau. Differential Revision: https://phabricator.services.mozilla.com/D150087
d1f042aba3a97b8afec93f7b0271318e4b38e46c: Bug 1776145 - [devtools] Remove unnecessary WebConsoleUI additionalProxies. r=ochameau,devtools-backward-compat-reviewers,jdescottes.
Nicolas Chevobbe <nchevobbe@mozilla.com> - Mon, 04 Jul 2022 13:27:51 +0000 - rev 622924
Push 39935 by ctuns@mozilla.com at Mon, 04 Jul 2022 21:42:52 +0000
Bug 1776145 - [devtools] Remove unnecessary WebConsoleUI additionalProxies. r=ochameau,devtools-backward-compat-reviewers,jdescottes. `WebConsoleConnectionProxy` is now only used for: - listening to `lastPrivateContextExited`, which can be done from the top-level console front instead - setting `NetworkMonitor.saveRequestAndResponseBodies` for toolbox we don't have network resource support for, which would be better done in `startWatchingNetworkResources`. This means we don't need to keep a Map of additional proxies. The only impact is on `clearMessagesCache`, where we now fetch all fronts with `getAllFronts` instead of looping through the Map of additional proxies. This change highlighted some race condition in tests, as it's now slightly slower. So we add a new `clearMessagesCacheAsync` function, which is doing the same thing as `clearMessagesCache`, except it's not `oneway`, so we can know when the cache was indeed cleared, and we emit an event to indicate when the cache was cleared. We can't simply remove `oneway` from `clearMessagesCache` as it causes backward compatibility issues we can't avoid. This also highlighted an issue with the cache not being cleared on `console.clear` when it is batched, so we fix this (a test was failing without it). The next patch in the queue will completely remove `WebConsoleConnectionProxy`. Differential Revision: https://phabricator.services.mozilla.com/D150085
c036b48c5be1f74d25fd7a0713fff0c5e88235a6: Bug 1766128 - [devtools] Don't define empty default values for sessionData in getSessionData. r=ochameau
Nicolas Chevobbe <nchevobbe@mozilla.com> - Thu, 30 Jun 2022 13:44:42 +0000 - rev 622699
Push 39920 by smolnar@mozilla.com at Thu, 30 Jun 2022 21:24:30 +0000
Bug 1766128 - [devtools] Don't define empty default values for sessionData in getSessionData. r=ochameau A test is added to check that having a script with a debugger statement does not lead to the thread pausing. Differential Revision: https://phabricator.services.mozilla.com/D150356
3661ef687ece133a5e78f3f5ea1fb74f00be3895: Bug 1776167 - [devtools] Fix chromeContext for the legacy-listeners codepath r=ochameau
Hubert Boma Manilla <hmanilla@mozilla.com> - Mon, 27 Jun 2022 13:22:16 +0000 - rev 622142
Push 39908 by imoraru@mozilla.com at Mon, 27 Jun 2022 21:50:36 +0000
Bug 1776167 - [devtools] Fix chromeContext for the legacy-listeners codepath r=ochameau browser_console_enable_network_monitoring.js is failing on beta because MBT is not enabled by default so the the leagcy-listener copepath is used. This adds `chromeContext` to the network resource and fixes the issue. Differential Revision: https://phabricator.services.mozilla.com/D150328
92862991e711e3e9ac6a57645244b112f460ec4a: Bug 1776144 - [devtools] Remove unused TARGET_AVAILABLE action in WebConsole. r=ochameau.
Nicolas Chevobbe <nchevobbe@mozilla.com> - Thu, 23 Jun 2022 08:29:44 +0000 - rev 621855
Push 39889 by smolnar@mozilla.com at Thu, 23 Jun 2022 15:57:48 +0000
Bug 1776144 - [devtools] Remove unused TARGET_AVAILABLE action in WebConsole. r=ochameau. Differential Revision: https://phabricator.services.mozilla.com/D150084
e77c68a3da9a7377d8f26e03c345d2827ef70e3c: Bug 1775411 - [devtools] Fix append new stylesheet link. r=ochameau.
Nicolas Chevobbe <nchevobbe@mozilla.com> - Wed, 22 Jun 2022 09:30:05 +0000 - rev 621708
Push 39883 by ctuns@mozilla.com at Wed, 22 Jun 2022 16:10:22 +0000
Bug 1775411 - [devtools] Fix append new stylesheet link. r=ochameau. The link was missing an event listener, that we add in this patch. A test case is added to make sure the link does work. Differential Revision: https://phabricator.services.mozilla.com/D149953
b54af8bf56142f765f8c48b5c5e188c466d7291e: Bug 1775113 - Make Loader.jsm honour ac_add_options --enable-debug-js-modules r=devtools-reviewers,ochameau
Michael Ratcliffe <mratcliffe@mozilla.com> - Wed, 22 Jun 2022 08:34:21 +0000 - rev 621666
Push 39883 by ctuns@mozilla.com at Wed, 22 Jun 2022 16:10:22 +0000
Bug 1775113 - Make Loader.jsm honour ac_add_options --enable-debug-js-modules r=devtools-reviewers,ochameau Differential Revision: https://phabricator.services.mozilla.com/D149778
bc573f999f803cf82221d0a55b473d5d834502e3: Bug 1775084 - [devtools] Support clearing of root-resources r=ochameau,devtools-backward-compat-reviewers
Hubert Boma Manilla <hmanilla@mozilla.com> - Mon, 20 Jun 2022 15:32:16 +0000 - rev 621450
Push 39874 by csabou@mozilla.com at Mon, 20 Jun 2022 21:49:56 +0000
Bug 1775084 - [devtools] Support clearing of root-resources r=ochameau,devtools-backward-compat-reviewers Differential Revision: https://phabricator.services.mozilla.com/D149635
7f9d6652f16ad436552bf892eb5af670bc1b74dc: Bug 1775084 - [devtools] Add framework capabilities to clear resources r=ochameau,devtools-backward-compat-reviewers,jdescottes
Hubert Boma Manilla <hmanilla@mozilla.com> - Mon, 20 Jun 2022 15:32:15 +0000 - rev 621449
Push 39874 by csabou@mozilla.com at Mon, 20 Jun 2022 21:49:56 +0000
Bug 1775084 - [devtools] Add framework capabilities to clear resources r=ochameau,devtools-backward-compat-reviewers,jdescottes Add support for clearing resources to the framework A resource command test is added in D148694 after clearing for network events is added Differential Revision: https://phabricator.services.mozilla.com/D148139
487946273b1065df84ec08088c3f5f1463500634: Bug 1775078 - [devtools] Don't await in for loop in NodePicker.#stop. r=ochameau.
Nicolas Chevobbe <nchevobbe@mozilla.com> - Mon, 20 Jun 2022 14:55:21 +0000 - rev 621444
Push 39874 by csabou@mozilla.com at Mon, 20 Jun 2022 21:49:56 +0000
Bug 1775078 - [devtools] Don't await in for loop in NodePicker.#stop. r=ochameau. Instead, populate an array of Promise we can await on outside of the loop. Differential Revision: https://phabricator.services.mozilla.com/D149751
d9270fca4a3ffb6b33b0ecd30fc7d694ec6ca227: Bug 1744240 - [devtools] Set navigation message source to CONSOLE_FRONTEND. r=ochameau.
Nicolas Chevobbe <nchevobbe@mozilla.com> - Mon, 20 Jun 2022 14:03:18 +0000 - rev 621436
Push 39874 by csabou@mozilla.com at Mon, 20 Jun 2022 21:49:56 +0000
Bug 1744240 - [devtools] Set navigation message source to CONSOLE_FRONTEND. r=ochameau. This fixes an intermittent on browser_webconsole_warn_about_replaced_api.js where an assertion was failing because the navigation message had the `console-api` class. We take this as an opportunity to add a navigation marker specific component instead of reusing the ConsoleApiCall one. Differential Revision: https://phabricator.services.mozilla.com/D149739
b2776977a909759b47b0e3be6c72b8ae00973f24: Bug 1771249 - [devtools] Fix priviledged requests showing as content messages r=ochameau
Hubert Boma Manilla <hmanilla@mozilla.com> - Mon, 20 Jun 2022 13:48:26 +0000 - rev 621432
Push 39874 by csabou@mozilla.com at Mon, 20 Jun 2022 21:49:56 +0000
Bug 1771249 - [devtools] Fix priviledged requests showing as content messages r=ochameau Differential Revision: https://phabricator.services.mozilla.com/D147684
2528596d754a942edeec559f57f5caa688eda93e: Bug 1742979 - [bidi] Implement basic support for "script.evaluate" command r=webdriver-reviewers,whimboo,ochameau
Julian Descottes <jdescottes@mozilla.com> - Thu, 16 Jun 2022 19:44:43 +0000 - rev 621193
Push 39861 by nfay@mozilla.com at Fri, 17 Jun 2022 09:39:07 +0000
Bug 1742979 - [bidi] Implement basic support for "script.evaluate" command r=webdriver-reviewers,whimboo,ochameau Adds a new root script module, and a new windowglobal script module. The root script module supports the public command evaluate, with the following limitations: - awaitPromise is not supported - the RealmTarget type is not supported - sandbox is not supported for the ContextTarget type - evaluation return values are not supported - exception handling is not supported - ownership model is not supported wdspec tests will be added via https://github.com/web-platform-tests/wpt/pull/34162 Differential Revision: https://phabricator.services.mozilla.com/D148907
034347a53a0209052ebeab15f5338b08d5dd98cc: Bug 1773994 - [devtools] Remove unnecessary code for network messages in MESSAGE_OPEN. r=ochameau.
Nicolas Chevobbe <nchevobbe@mozilla.com> - Wed, 15 Jun 2022 12:57:08 +0000 - rev 620980
Push 39856 by abutkovits@mozilla.com at Wed, 15 Jun 2022 21:49:08 +0000
Bug 1773994 - [devtools] Remove unnecessary code for network messages in MESSAGE_OPEN. r=ochameau. With the changes from Bug 1686440, there's no benefit keeping this code around. Differential Revision: https://phabricator.services.mozilla.com/D149139
1a2b8a3c8055176bd4d681efd819072ecf8247b9: Bug 1773993 - [devtools] Remove unused removeIds variable in addMessage. r=ochameau.
Nicolas Chevobbe <nchevobbe@mozilla.com> - Wed, 15 Jun 2022 12:57:08 +0000 - rev 620979
Push 39856 by abutkovits@mozilla.com at Wed, 15 Jun 2022 21:49:08 +0000
Bug 1773993 - [devtools] Remove unused removeIds variable in addMessage. r=ochameau. Nothing was added to the array, so we can remove it. Differential Revision: https://phabricator.services.mozilla.com/D149138
c678f72199826ead4f936cf173b32ba216618a24: Bug 1764501 - [devtools] Speedup adding messages out of order. r=ochameau.
Nicolas Chevobbe <nchevobbe@mozilla.com> - Wed, 15 Jun 2022 12:57:07 +0000 - rev 620978
Push 39856 by abutkovits@mozilla.com at Wed, 15 Jun 2022 21:49:08 +0000
Bug 1764501 - [devtools] Speedup adding messages out of order. r=ochameau. In the console we might receive messages that need to be inserted at a specific position in the output to respect the chronological order. This is very visible when evaluating an expression in the console while we are fetching cached messages, or when the expression is logging lots of messages, since the evaluation result doesn't go through the ResourceCommand itself. Since we were relying on the order of insertion in `mutableMessagesById` for the order of messages in the output, we had no choice but clone the Map to insert the new message at the specific position. With lots of messages, this can be utterly slow, and is something people are experiencing. To fix this, we're adding a new state property, `mutableMessagesOrder`, which holds message ids in the chronological order. It's then used to compute `visibleMessages`. As we don't have to clone the Map anymore, it makes reordering as fast as looking for an index in the array and splicing it (which is fast). We also take this opportunity to switch away from our `findIndex` usage; most often than not, the messages will be reinserted at the end of the array, so we can loop through it backward to find the index faster. We still keep `mutableMessagesById` as a Map so lookup are fast, and we don't have to deal with impacts we'd have if we'd switch to an object. The existing node test helper to get a message at a specific index are migrated to make use of `mutableMessagesOrder`, and a few test cases and assertions are added to check that the new property is handled correctly. Differential Revision: https://phabricator.services.mozilla.com/D148658
657af115a0b736c258f062ed78b2a02e12196ea0: Bug 1773986 - [devtools] Add DAMP test case to cover adding out-of-order messages. r=ochameau,perftest-reviewers,sparky.
Nicolas Chevobbe <nchevobbe@mozilla.com> - Wed, 15 Jun 2022 12:57:07 +0000 - rev 620977
Push 39856 by abutkovits@mozilla.com at Wed, 15 Jun 2022 21:49:08 +0000
Bug 1773986 - [devtools] Add DAMP test case to cover adding out-of-order messages. r=ochameau,perftest-reviewers,sparky. This test demonstrates that the patch in Bug 1764501 does fix the performance issue we were seeing. Having it in tree will ensure we don't regress this. Differential Revision: https://phabricator.services.mozilla.com/D149110
6be112a33fb5ba5af477ae94678f01b23235c0c4: Bug 1772363 - [devtools] Ensure actually watching resources before pause/resume UI is updated r=ochameau
Hubert Boma Manilla <hmanilla@mozilla.com> - Tue, 14 Jun 2022 15:38:07 +0000 - rev 620869
Push 39850 by ctuns@mozilla.com at Tue, 14 Jun 2022 21:37:29 +0000
Bug 1772363 - [devtools] Ensure actually watching resources before pause/resume UI is updated r=ochameau Changing the pattern for toggling recording to ensure that pause/resumes actually start or stops watching resources before the ui states gets updated. Differential Revision: https://phabricator.services.mozilla.com/D148953
5092df1913d5091b814b9b98fa617921702f8ed4: Bug 1753048 - [devtools] Include the fallback window to the docShells array for webextension targets r=ochameau
Julian Descottes <jdescottes@mozilla.com> - Tue, 14 Jun 2022 06:52:49 +0000 - rev 620794
Push 39850 by ctuns@mozilla.com at Tue, 14 Jun 2022 21:37:29 +0000
Bug 1753048 - [devtools] Include the fallback window to the docShells array for webextension targets r=ochameau Depends on D148489 This allows to properly set the cssErrorReportingEnabled flag on its docShell. While we are not interested in css errors for this fallback window, the windowglobal target actor will automatically set the flag on new root browsing contexts only if the previous one had the flag set. This way when we switch to a valid webextension document (eg a popup), the flag will be set and CSS errors will be reported. Differential Revision: https://phabricator.services.mozilla.com/D148836
845c0d85f221d7d52795f4fa186c7d6791ba7e88: Bug 1753048 - [devtools] Append the addonId to the DevTools fallback window URL r=ochameau
Julian Descottes <jdescottes@mozilla.com> - Tue, 14 Jun 2022 06:52:49 +0000 - rev 620793
Push 39850 by ctuns@mozilla.com at Tue, 14 Jun 2022 21:37:29 +0000
Bug 1753048 - [devtools] Append the addonId to the DevTools fallback window URL r=ochameau This window does not share the regular extension principal so we cannot fetch the addonId from the principal in the same way as we do for other documents. Instead we append the addonId to the URL and provide a new helper in browser-context-helpers to extract the addonId from a browsingContext. This helper should work transparently from the parent process and the content process, and should support browsing context for regular extension documents as well as browsing contexts for fallback windows. Differential Revision: https://phabricator.services.mozilla.com/D148489
e0d328cd054ba7ea7e131aa5b0a24d486b3ff643: Bug 1772363 - [devtools] Fix intermittent on browser_net_pause.js r=ochameau
Hubert Boma Manilla <hmanilla@mozilla.com> - Thu, 09 Jun 2022 09:34:31 +0000 - rev 620274
Push 39824 by smolnar@mozilla.com at Thu, 09 Jun 2022 22:00:48 +0000
Bug 1772363 - [devtools] Fix intermittent on browser_net_pause.js r=ochameau Differential Revision: https://phabricator.services.mozilla.com/D148674
d193aa033e1e6bdcad73c87b4211774d817eedc0: Bug 1771277 - [devtools] Remove the websocket and server sent events prefs r=ochameau
Hubert Boma Manilla <hmanilla@mozilla.com> - Tue, 07 Jun 2022 12:18:49 +0000 - rev 619947
Push 39812 by imoraru@mozilla.com at Tue, 07 Jun 2022 21:47:25 +0000
Bug 1771277 - [devtools] Remove the websocket and server sent events prefs r=ochameau Differential Revision: https://phabricator.services.mozilla.com/D147413
e9828113f050abbc521e5aa58c491b926c170032: Bug 1764348 - Pause / Resume should toggle listening to network resources r=ochameau
Hubert Boma Manilla <hmanilla@mozilla.com> - Tue, 07 Jun 2022 12:18:49 +0000 - rev 619946
Push 39812 by imoraru@mozilla.com at Tue, 07 Jun 2022 21:47:25 +0000
Bug 1764348 - Pause / Resume should toggle listening to network resources r=ochameau With this patch Pause and Resume, now stop and start listening to network requests. Differential Revision: https://phabricator.services.mozilla.com/D148488
e440a0831a58b4ea86124bea3197264f0eb6f0dd: Bug 1771659 - [devtools] Fix perma failure in BETA r=ochameau
Hubert Boma Manilla <hmanilla@mozilla.com> - Tue, 07 Jun 2022 07:39:42 +0000 - rev 619921
Push 39810 by ccozmuta@mozilla.com at Tue, 07 Jun 2022 15:57:48 +0000
Bug 1771659 - [devtools] Fix perma failure in BETA r=ochameau The multiprocess browser toolbox is on by default in BETA, so we need to switch it on explicitly for the addon extensions to be listed. Differential Revision: https://phabricator.services.mozilla.com/D148392
d4191d470750ee1b00010bc99a88d63fbbfa420c: Bug 1772093 - Part 9: Use plain object for lazy getter in devtools/startup/. r=devtools-reviewers,ochameau
Tooru Fujisawa <arai_a@mac.com> - Mon, 06 Jun 2022 07:09:55 +0000 - rev 619781
Push 39804 by nbeleuzu@mozilla.com at Mon, 06 Jun 2022 15:43:14 +0000
Bug 1772093 - Part 9: Use plain object for lazy getter in devtools/startup/. r=devtools-reviewers,ochameau Differential Revision: https://phabricator.services.mozilla.com/D147905
5d2ed8641b200d02c238c0ba7f98182b2d061635: Bug 1772093 - Part 8: Use plain object for lazy getter in devtools/shared/loader/. r=devtools-reviewers,ochameau
Tooru Fujisawa <arai_a@mac.com> - Mon, 06 Jun 2022 07:09:54 +0000 - rev 619780
Push 39804 by nbeleuzu@mozilla.com at Mon, 06 Jun 2022 15:43:14 +0000
Bug 1772093 - Part 8: Use plain object for lazy getter in devtools/shared/loader/. r=devtools-reviewers,ochameau Differential Revision: https://phabricator.services.mozilla.com/D147904
39176a410dc894623770d3ddd607a03998bfa8b4: Bug 1772093 - Part 7: Use plain object for lazy getter in devtools/server/connectors/. r=devtools-reviewers,ochameau
Tooru Fujisawa <arai_a@mac.com> - Mon, 06 Jun 2022 07:09:54 +0000 - rev 619779
Push 39804 by nbeleuzu@mozilla.com at Mon, 06 Jun 2022 15:43:14 +0000
Bug 1772093 - Part 7: Use plain object for lazy getter in devtools/server/connectors/. r=devtools-reviewers,ochameau Differential Revision: https://phabricator.services.mozilla.com/D147903
60b22cd1f3bb1ec1eb75e5ed9b130929b4dcd04a: Bug 1772093 - Part 6: Use plain object for lazy getter in devtools/server/actors/. r=devtools-reviewers,ochameau
Tooru Fujisawa <arai_a@mac.com> - Mon, 06 Jun 2022 07:09:54 +0000 - rev 619778
Push 39804 by nbeleuzu@mozilla.com at Mon, 06 Jun 2022 15:43:14 +0000
Bug 1772093 - Part 6: Use plain object for lazy getter in devtools/server/actors/. r=devtools-reviewers,ochameau Differential Revision: https://phabricator.services.mozilla.com/D147902
69abd3204293a6c9a5c457ff076d856105e5a5be: Bug 1772093 - Part 5: Use plain object for lazy getter in devtools/client/styleeditor/. r=devtools-reviewers,ochameau
Tooru Fujisawa <arai_a@mac.com> - Mon, 06 Jun 2022 07:09:53 +0000 - rev 619777
Push 39804 by nbeleuzu@mozilla.com at Mon, 06 Jun 2022 15:43:14 +0000
Bug 1772093 - Part 5: Use plain object for lazy getter in devtools/client/styleeditor/. r=devtools-reviewers,ochameau Differential Revision: https://phabricator.services.mozilla.com/D147901
5c3774a15e991f4e09ff975ba5a2aca60fbb9e5d: Bug 1772093 - Part 4: Use plain object for lazy getter in devtools/client/storage/. r=devtools-reviewers,ochameau
Tooru Fujisawa <arai_a@mac.com> - Mon, 06 Jun 2022 07:09:53 +0000 - rev 619776
Push 39804 by nbeleuzu@mozilla.com at Mon, 06 Jun 2022 15:43:14 +0000
Bug 1772093 - Part 4: Use plain object for lazy getter in devtools/client/storage/. r=devtools-reviewers,ochameau Differential Revision: https://phabricator.services.mozilla.com/D147900
1b43ca2a6774bb42f82830c11470c669194f09ae: Bug 1772093 - Part 3: Use plain object for lazy getter in devtools/client/shared/. r=devtools-reviewers,ochameau
Tooru Fujisawa <arai_a@mac.com> - Mon, 06 Jun 2022 07:09:53 +0000 - rev 619775
Push 39804 by nbeleuzu@mozilla.com at Mon, 06 Jun 2022 15:43:14 +0000
Bug 1772093 - Part 3: Use plain object for lazy getter in devtools/client/shared/. r=devtools-reviewers,ochameau Differential Revision: https://phabricator.services.mozilla.com/D147899
e531a1b64b4e1a5c72a3388e19c415f162e33dea: Bug 1772093 - Part 2: Use plain object for lazy getter in devtools/client/jsonview/. r=ochameau
Tooru Fujisawa <arai_a@mac.com> - Mon, 06 Jun 2022 07:09:52 +0000 - rev 619774
Push 39804 by nbeleuzu@mozilla.com at Mon, 06 Jun 2022 15:43:14 +0000
Bug 1772093 - Part 2: Use plain object for lazy getter in devtools/client/jsonview/. r=ochameau Differential Revision: https://phabricator.services.mozilla.com/D147898
5913f29e0b783d3dce96d11b48aa3a3161698d9f: Bug 1772093 - Part 1: Use plain object for lazy getter in devtools/client/framework/. r=ochameau
Tooru Fujisawa <arai_a@mac.com> - Mon, 06 Jun 2022 07:09:52 +0000 - rev 619773
Push 39804 by nbeleuzu@mozilla.com at Mon, 06 Jun 2022 15:43:14 +0000
Bug 1772093 - Part 1: Use plain object for lazy getter in devtools/client/framework/. r=ochameau Differential Revision: https://phabricator.services.mozilla.com/D147897
569af989260a57e1acacfef2e487ce7a1d3c4016: Bug 1772093 - Part 0: Eagerly import some modules in devtools/. r=ochameau
Tooru Fujisawa <arai_a@mac.com> - Mon, 06 Jun 2022 07:09:51 +0000 - rev 619772
Push 39804 by nbeleuzu@mozilla.com at Mon, 06 Jun 2022 15:43:14 +0000
Bug 1772093 - Part 0: Eagerly import some modules in devtools/. r=ochameau Differential Revision: https://phabricator.services.mozilla.com/D148178
176c288277f4fba9ded850f635f18c25f7049cb8: Bug 1772239 - [devtools] Stop reading client preference devtools.browsertoolbox.fission in the devtools server r=ochameau,devtools-backward-compat-reviewers,bomsy
Julian Descottes <jdescottes@mozilla.com> - Fri, 03 Jun 2022 16:51:51 +0000 - rev 619670
Push 39793 by mlaza@mozilla.com at Fri, 03 Jun 2022 21:43:55 +0000
Bug 1772239 - [devtools] Stop reading client preference devtools.browsertoolbox.fission in the devtools server r=ochameau,devtools-backward-compat-reviewers,bomsy The devtools.browsertoolbox.fission should not be read on the server but should instead be provided to the server by the client. The breakage can be reproduced if you try to debug a Firefox desktop with devtools.browsertoolbox.fission = false from a Firefox with devtools.browsertoolbox.fission = true. Differential Revision: https://phabricator.services.mozilla.com/D148129
21c03813c9d0790422129c332a0c797d4aec6380: Bug 1771277 - [devtools] Remove the websocket and server sent events prefs r=ochameau
Hubert Boma Manilla <hmanilla@mozilla.com> - Thu, 02 Jun 2022 14:35:25 +0000 - rev 619576
Push 39788 by nfay@mozilla.com at Fri, 03 Jun 2022 03:41:38 +0000
Bug 1771277 - [devtools] Remove the websocket and server sent events prefs r=ochameau Differential Revision: https://phabricator.services.mozilla.com/D147413
0ee89f81d7da0bad363aab7527fb4ac122f13972: Bug 1764348 - Pause / Resume should toggle listening to network resources r=ochameau
Hubert Boma Manilla <hmanilla@mozilla.com> - Thu, 02 Jun 2022 14:35:24 +0000 - rev 619575
Push 39788 by nfay@mozilla.com at Fri, 03 Jun 2022 03:41:38 +0000
Bug 1764348 - Pause / Resume should toggle listening to network resources r=ochameau With this patch Pause and Resume, now stop and start listening to network requests Differential Revision: https://phabricator.services.mozilla.com/D146445
d7c44ddd5e6ff764a7007a2df88eedbf91236716: Bug 1770869 - remove unused client certificate authentication for remote devtools r=ochameau
Dana Keeler <dkeeler@mozilla.com> - Wed, 01 Jun 2022 23:20:57 +0000 - rev 619493
Push 39785 by bszekely@mozilla.com at Thu, 02 Jun 2022 09:36:47 +0000
Bug 1770869 - remove unused client certificate authentication for remote devtools r=ochameau Client certificate authentication for remote devtools was used in WebIDE, which was removed from Firefox 71 in bug 1539462. Differential Revision: https://phabricator.services.mozilla.com/D147121
5a138d047be11719557da49970087039aa233aba: Bug 1770869 - remove unused client certificate authentication for remote devtools r=ochameau
Dana Keeler <dkeeler@mozilla.com> - Wed, 01 Jun 2022 20:00:57 +0000 - rev 619473
Push 39785 by bszekely@mozilla.com at Thu, 02 Jun 2022 09:36:47 +0000
Bug 1770869 - remove unused client certificate authentication for remote devtools r=ochameau Client certificate authentication for remote devtools was used in WebIDE, which was removed from Firefox 71 in bug 1539462. Differential Revision: https://phabricator.services.mozilla.com/D147121
a71be00e24970e7139225fd13511e788bc9ca6e4: Bug 1770160 - [devtools] Don't filter out platform-specific errors in Browser Console/Browser Toolbox. r=ochameau.
Nicolas Chevobbe <nchevobbe@mozilla.com> - Fri, 20 May 2022 05:47:23 +0000 - rev 618324
Push 39723 by abutkovits@mozilla.com at Fri, 20 May 2022 15:37:03 +0000
Bug 1770160 - [devtools] Don't filter out platform-specific errors in Browser Console/Browser Toolbox. r=ochameau. This is as simple as checking the session context type. Differential Revision: https://phabricator.services.mozilla.com/D146818
79b0fc57aff3e7ca985068f4a8ec3d2968d50ec8: Bug 1761554 - Use a Map-based cache for ResourceCommand r=ochameau
Doug Thayer <dothayer@mozilla.com> - Thu, 19 May 2022 16:39:14 +0000 - rev 618258
Push 39722 by nfay@mozilla.com at Fri, 20 May 2022 09:31:26 +0000
Bug 1761554 - Use a Map-based cache for ResourceCommand r=ochameau This just intends to alleviate the more noticeable problem when the cache grows too large by avoiding iterating through the array every time something changes. There still remains the work of limiting the size of this based on what the UI is capped at, but that can come later. Differential Revision: https://phabricator.services.mozilla.com/D142279
ea32c4c1d987ddef9f831edbb1c4cb187930dc16: Bug 1700092 - [devtools] Add supported targets and resources to session context. r=bomsy,ochameau.
Nicolas Chevobbe <nchevobbe@mozilla.com> - Thu, 19 May 2022 05:24:53 +0000 - rev 618194
Push 39719 by smolnar@mozilla.com at Thu, 19 May 2022 16:03:14 +0000
Bug 1700092 - [devtools] Add supported targets and resources to session context. r=bomsy,ochameau. This allows us to simplify and fix `hasStyleSheetWatcherSupportForTarget`. In the browser toolbox, since we're in a "all" session context type, `TargetActorRegistry.getTargetActors` would only return an array of a single element, the parent process target. In case of window global targets, in the browser toolbox context, we wouldn't find the target we're looking for, which was making `hasStyleSheetWatcherSupportForTarget` return false. We can now simply look into the targetActor session context. Depends on D146584 Differential Revision: https://phabricator.services.mozilla.com/D146193
597785337a95b29ebbc11ed3c1834653e2552acd: Bug 1766237 - [devtools] Get the WebExtensionPolicy from the ExtensionProcessScript global r=jdescottes,rpl,ochameau
Hubert Boma Manilla <hmanilla@mozilla.com> - Wed, 18 May 2022 08:56:13 +0000 - rev 618090
Push 39716 by ncsoregi@mozilla.com at Wed, 18 May 2022 21:42:45 +0000
Bug 1766237 - [devtools] Get the WebExtensionPolicy from the ExtensionProcessScript global r=jdescottes,rpl,ochameau The issue here is it seems like WebExtensionPolicy is not always available on the Cu global, therefore failing when trying to get the extension policy name. in this example when adding sources from the browser toolbox (i'm still not completely sure why) The solution here is changing to using the ExtensionProcessScript which works for the web and browser toolbox. Also added a `console.warn` so we get some notification when this fails. Differential Revision: https://phabricator.services.mozilla.com/D144787
e7bc7584f0c21b0b12847b6d180b3e5008fe5689: Bug 1755197 - [devtools] Remove breakpoint shifting on breakpoint syncing r=ochameau
Hubert Boma Manilla <hmanilla@mozilla.com> - Fri, 13 May 2022 03:35:18 +0000 - rev 617152
Push 39689 by bszekely@mozilla.com at Fri, 13 May 2022 09:35:38 +0000
Bug 1755197 - [devtools] Remove breakpoint shifting on breakpoint syncing r=ochameau This patch focuses on removing breakpoint shifting which is mostly used in non-sourcemapping scenarios. Differential Revision: https://phabricator.services.mozilla.com/D138625
bc3ad1de463538b2da49c35a1923b19d8aa99d8b: Bug 1762865 - [devtools] Add assertion for server breakpoints in breakpoints tests r=ochameau
Hubert Boma Manilla <hmanilla@mozilla.com> - Fri, 29 Apr 2022 14:23:51 +0000 - rev 615686
Push 39630 by apavel@mozilla.com at Sat, 30 Apr 2022 09:19:22 +0000
Bug 1762865 - [devtools] Add assertion for server breakpoints in breakpoints tests r=ochameau This patch also adds assertions for breakpoints on the server to the breakpoints reload tests. This also asserts that the breakpoint is not removed on the server when related source no longer exists after a reload Differential Revision: https://phabricator.services.mozilla.com/D142545
c8fee25f3eab73612c88bb831493db08842b5ce0: Bug 1764055 - [devtools] Add tests for breakpoints in original source whose generated source content changes on reload r=ochameau
Hubert Boma Manilla <hmanilla@mozilla.com> - Fri, 29 Apr 2022 14:23:51 +0000 - rev 615685
Push 39630 by apavel@mozilla.com at Sat, 30 Apr 2022 09:19:22 +0000
Bug 1764055 - [devtools] Add tests for breakpoints in original source whose generated source content changes on reload r=ochameau Updating the bug id Differential Revision: https://phabricator.services.mozilla.com/D142047
87cb99924fc44a0a599b8119a977b36486331eef: Bug 1762865 - [devtools] Add assertion for server breakpoints in breakpoints tests r=ochameau
Hubert Boma Manilla <hmanilla@mozilla.com> - Mon, 25 Apr 2022 11:46:23 +0000 - rev 615195
Push 39613 by ctuns@mozilla.com at Mon, 25 Apr 2022 21:36:55 +0000
Bug 1762865 - [devtools] Add assertion for server breakpoints in breakpoints tests r=ochameau This patch also adds assertions for breakpoints on the server to the breakpoints reload tests. This also asserts that the breakpoint is not removed on the server when related source no longer exists after a reload Differential Revision: https://phabricator.services.mozilla.com/D142545
83c8bb7087f50dec500df7ddd6e353b13483b964: Bug 1764055 - [devtools] Add tests for breakpoints in original source whose generated source content changes on reload r=ochameau
Hubert Boma Manilla <hmanilla@mozilla.com> - Mon, 25 Apr 2022 11:46:22 +0000 - rev 615194
Push 39613 by ctuns@mozilla.com at Mon, 25 Apr 2022 21:36:55 +0000
Bug 1764055 - [devtools] Add tests for breakpoints in original source whose generated source content changes on reload r=ochameau Updating the bug id Differential Revision: https://phabricator.services.mozilla.com/D142047
ae9861178cd12cf79fc2460bcdacc6bd47b13e5e: Bug 1765748 - [devtools] Remove SplitView active summary getter and rename setter. r=ochameau.
Nicolas Chevobbe <nchevobbe@mozilla.com> - Fri, 22 Apr 2022 13:56:01 +0000 - rev 615109
Push 39603 by smolnar@mozilla.com at Fri, 22 Apr 2022 21:28:46 +0000
Bug 1765748 - [devtools] Remove SplitView active summary getter and rename setter. r=ochameau. Differential Revision: https://phabricator.services.mozilla.com/D144302