1657153 - Update ResourceWatcher cache on update and destroyed events

Status: RESOLVED FIXED

(DevTools :: Framework, enhancement)

Description

[Alexandre Poirot [:ochameau]]

Use a resourceId attribute in resource-updated-form and resource-destroyed-form in order to:

• update the existing, matching cached resource which is in the cache array
• remove it from the cache when it is destroyed

There is many ways to support how to update the cache resource in resource-updated-form.
For now, I think only network_event would really benefit from that, so we may drive decision from bug 1644191's requirements.
NETWORK_EVENT server implementation is most likely going to land preffed off for a bit, and so we may change this depending on the progress of this particular resource.
We should be careful if some other resource starts depending on this particular feature.

The resourceId for NETWORK_EVENT will be the nsIChannel's channelId. In other cases, if we don't have an ID coming from platform objects, we may use actorID or an integer increment as last resort?

Comment 1

[Chris Peterson [:cpeterson]]

Tracking dt-fission-m2-mvp bugs for Fission Nightly milestone (M6c).

Comment 2

[Hubert Boma Manilla (:bomsy)]

Attachment: Bug 1657153 - Use the resource id for updating and deleting a resource r=ochameau

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/a99b36412992
Use the resource id for updating and deleting a resource r=ochameau

Comment 4

[Cristian Brindusan [:cbrindusan]]

https://hg.mozilla.org/mozilla-central/rev/a99b36412992