New issue
Advanced search Search tips

Issue 642347 link

Starred by 1 user
Status: Fixed
Owner:
jochen@chromium.org
Closed: Aug 2016
Cc:
hablich@chromium.org
haraken@chromium.org
verwa...@chromium.org
marja@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 2
Type: Bug



Sign in to add a comment

background parser data held alive unnecessarily long by oilpan

Project Member Reported by jochen@chromium.org, Aug 30 2016 
we could release the data (mostly zone memory) right after compiling.
Project Member

Comment 1 by bugdroid1@chromium.org, Aug 30 2016 

The following revision refers to this bug:
  https://chromium.googlesource.com/v8/v8.git/+/60c75fbe4124215a7e0a5beabe24f16bd977c1cb

commit 60c75fbe4124215a7e0a5beabe24f16bd977c1cb
Author: jochen <jochen@chromium.org>
Date: Tue Aug 30 14:10:25 2016

Reland of "Release streamed script resources after it was compiled"

Original issue's description:
> Release streamed script resources after it was compiled
>
> Otherwise, we'd hold on to the resources until the embedder frees them
> which might take a long time
>
> R=marja@chromium.org,verwaest@chromium.org
> BUG=
>
> Committed: https://crrev.com/877dac34465c018bb534b7781fbe242ae4e33c32
> Cr-Commit-Position: refs/heads/master@{#38999}

TBR=marja@chromium.org,verwaest@chromium.org
BUG= chromium:642347 

Review-Url: https://codereview.chromium.org/2296733002
Cr-Commit-Position: refs/heads/master@{#39018}

[modify] https://crrev.com/60c75fbe4124215a7e0a5beabe24f16bd977c1cb/src/api.cc
[modify] https://crrev.com/60c75fbe4124215a7e0a5beabe24f16bd977c1cb/src/background-parsing-task.h

Comment 2 by jochen@chromium.org, Aug 30 2016

Status: Fixed (was: Assigned)

Comment 3 by verwa...@chromium.org, Aug 30 2016

Cc: hablich@chromium.org
We probably want to backmerge to M54?

Comment 4 by hablich@chromium.org, Sep 6 2016 

Given that this is not a bug fix for a regression I would not merge this to M54.
►

Sign in to add a comment