DescriptionRevert of Stop copying script contents for each RenderFrames. (patchset #4 id:60001 of https://codereview.chromium.org/2278003002/ )
Reason for revert:
Crashes renderer while destroying UserScriptSet via ChromeExtensionsRendererClient LazyInstance...
Original issue's description:
> Stop copying script contents for each RenderFrames.
>
> Instead, after unpickling the script content, put them in a WebString
> on its first use (GetJsSources) in a frame. Reuse them for subsequent
> frames.
>
> The cleanup is done when UserScriptSet releases its shared memory
> connection.
>
> A completely naive test shows that on WSJ site, with ABPlus extension
> installed, this reduced (24025 * 11 + 14785 * 10) = 412125 char copies!
>
> BUG=622464
> Test=No visible change is expected.
>
> Committed: https://crrev.com/0f702e9e3ccd2f408b198d2b2255d363da0310b9
> Cr-Commit-Position: refs/heads/master@{#417018}
TBR=rdevlin.cronin@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=622464, 645357
Patch Set 1 #
Messages
Total messages: 6 (3 generated)
|