Bug 157662 introduced the `ResourceWatcher` API, accessible via `toolbox.resourceWatcher`. This API will help listen to data that is being created early, when the document just started loading. We should migrate the whole DevTools codebase to this API for any data that: * DevTools frontend listen to, or care about, **and,** * may be created or be notified early, when the document just starts being loaded. This data will typically be: console messages, errors, warnings, sources, Root element NodeFront, storage values, network events, stylesheets, ... We are typically *not* going to use this API for: * data being fetched on-demand, from user's input. For ex: webconsole evaluation input, or, DOM element expands from the Markup view. * events which we only want to record when the user cares about them. For ex: animation events. For some more high level context, please have a look at [Migration to Fission-compatible APIs](https://docs.google.com/document/d/1O80ElDQw9zQ8B2oziwzrQrc9I73RXh0Tpa8U6Kw453I/edit#heading=h.lfe9ax4ms0tc), which describes all Fission-related refactorings. The typical task for this bug will be about migrating code that: * start listening and register a RDP event listener, * retrieve already existings data, from panel's codebase, to the `ResourceWatcher` module, in the `LegacyListener` object. And then, the panel should use the `ResourceWatcher` instead. Bug 1620234 is a good example of such migration, applied to Console Messages. Bug 1623699 is also useful example as it demonstrates how to write tests for such migration. This bug is about focusing on only one usecase in the console: the JS Errors. JS Errors are currently being listened via `WebConsoleFront.startListeners(["PageError"])`, from here: https://searchfox.org/mozilla-central/rev/9c6e7500c0015a2c60be7b1b888261d95095ce27/devtools/client/webconsole/webconsole-connection-proxy.js#123-131 https://searchfox.org/mozilla-central/rev/9c6e7500c0015a2c60be7b1b888261d95095ce27/devtools/client/webconsole/webconsole-connection-proxy.js#208-223 And already existing errors retrieved from here: https://searchfox.org/mozilla-central/rev/9c6e7500c0015a2c60be7b1b888261d95095ce27/devtools/client/webconsole/webconsole-connection-proxy.js#176-196
Bug 1625905 Comment 0 Edit History
Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.
Bug 1576624 introduced the `ResourceWatcher` API, accessible via `toolbox.resourceWatcher`. This API will help listen to data that is being created early, when the document just started loading. We should migrate the whole DevTools codebase to this API for any data that: * DevTools frontend listen to, or care about, **and,** * may be created or be notified early, when the document just starts being loaded. This data will typically be: console messages, errors, warnings, sources, Root element NodeFront, storage values, network events, stylesheets, ... We are typically *not* going to use this API for: * data being fetched on-demand, from user's input. For ex: webconsole evaluation input, or, DOM element expands from the Markup view. * events which we only want to record when the user cares about them. For ex: animation events. For some more high level context, please have a look at [Migration to Fission-compatible APIs](https://docs.google.com/document/d/1O80ElDQw9zQ8B2oziwzrQrc9I73RXh0Tpa8U6Kw453I/edit#heading=h.lfe9ax4ms0tc), which describes all Fission-related refactorings. The typical task for this bug will be about migrating code that: * start listening and register a RDP event listener, * retrieve already existings data, from panel's codebase, to the `ResourceWatcher` module, in the `LegacyListener` object. And then, the panel should use the `ResourceWatcher` instead. Bug 1620234 is a good example of such migration, applied to Console Messages. Bug 1623699 is also useful example as it demonstrates how to write tests for such migration. This bug is about focusing on only one usecase in the console: the JS Errors. JS Errors are currently being listened via `WebConsoleFront.startListeners(["PageError"])`, from here: https://searchfox.org/mozilla-central/rev/9c6e7500c0015a2c60be7b1b888261d95095ce27/devtools/client/webconsole/webconsole-connection-proxy.js#123-131 https://searchfox.org/mozilla-central/rev/9c6e7500c0015a2c60be7b1b888261d95095ce27/devtools/client/webconsole/webconsole-connection-proxy.js#208-223 And already existing errors retrieved from here: https://searchfox.org/mozilla-central/rev/9c6e7500c0015a2c60be7b1b888261d95095ce27/devtools/client/webconsole/webconsole-connection-proxy.js#176-196