Fix out of order resources
Categories
(DevTools :: Framework, enhancement)
Tracking
(Not tracked)
People
(Reporter: ochameau, Assigned: ochameau, NeedInfo)
References
Details
Attachments
(1 file)
(deleted),
text/x-phabricator-request
|
Details |
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.
Assignee | ||
Comment 1•3 years ago
|
||
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•3 years ago
|
||
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.
Updated•3 years ago
|
Description
•