1720732 - Fix out of order resources

Status: NEW

(DevTools :: Framework, enhancement)

Description

[Alexandre Poirot [:ochameau]]

https://searchfox.org/mozilla-central/source/devtools/shared/commands/resource/resource-command.js#571-573

  async _onResourceAvailable({ targetFront, watcherFront }, resources) {
    [...]
    for (let resource of resources) {
      [...]
      if (watcherFront) {
        targetFront = await this._getTargetForWatcherResource(resource);
      }

This async call to _getTargetForWatcherResource may potentialy delay some resources, introducing some unwanted delay between resource reception by ResourceCommand, and their addition to the throttling queue.
Because of this resources having some async delay here may be added out of order in the throttling queue.
This was something we identified in bug 1717050 and was quite puzzling.

Comment 1

[Alexandre Poirot [:ochameau]]

Attachment: Bug 1720732 - [devtools] Make WatcherFront.getBrowsingContextTarget synchronous for client side-top target.

This helps ResourceCommand immediately throttle parent process resources.
Without this, when we don't have server side target switching enabled,
and still create the top level target via the TabDescriptor, we are still
asynchronous and this can be confusing from ResourceCommand.onResourceAvailable.

May be we could have two versions of this function?
One synchronous for ResourceCommand and another, more powerful but async
for inspector codebase (which should also work for BC ids of same-process iframes).

Comment 2

[BugBot [:suhaib / :marco/ :calixte]]

There's a r+ patch which didn't land and no activity in this bug for 2 weeks.
:ochameau, could you have a look please?
For more information, please visit auto_nag documentation.