New issue
Advanced search Search tips

Issue 752711 link

Starred by 2 users

Issue metadata

Status: Started
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 2
Type: Bug

Blocked on:
issue 784875



Sign in to add a comment

Decouple "decoded data" handling from ScriptResource

Project Member Reported by hirosh...@chromium.org, Aug 5 2017

Issue description

Draft CL: https://chromium-review.googlesource.com/c/602606/

To resolve  Issue 692856 , this issue tracks efforts to introduce
ScriptResourceData.
ScriptResource represents everything needed by the users of ScriptResource after loading finished, that is, almost everything needed by the users of ScriptResource except for ScriptStreamer.
On load finish, ScriptResourceData is created (contains Request URL, ResourceResponse, SharedBuffer/String, etc.) and all other classes refer to ScriptResourceData, instead of ScriptResource directly.
By doing so, revalidation of ScriptResource no longer affects ScriptResourceClients that are already notified of finish, because revalidation modifies ScriptResource, but doesn't modify the existing ScriptResourceData. In this way, this fixes  Issue 692856 .

Design doc to be written.
 
Blocking: 692856 749773
Labels: M-62
Sounds great!
Blocking: -692856 -749773
Labels: -Pri-1 -M-62 Pri-2
I choose another short-term fix for  Issue 692856 .
Unblocking the related issues, making Pri-2, removing M-62 target.

I postponed this issue, because I found more dependencies around CachedMetadataHandler.

Blockedon: 784875

Sign in to add a comment