Chromium Code Reviews
DescriptionReland of Oilpan: Remove page navigation GCs from the prologues of V8's GCs (patchset #1 id:1 of https://codereview.chromium.org/1688483002/ )
Reason for revert:
Test is disabled, reverting the revert.
Original issue's description:
> Revert of Oilpan: Remove page navigation GCs from the prologues of V8's GCs (patchset #1 id:1 of https://codereview.chromium.org/1679683003/ )
>
> Reason for revert:
> Speculative revert to fix a Mac browser test that relies on 'loadstop' being transmitted in certain contexts.
>
> Original issue's description:
> > Oilpan: Remove page navigation GCs from the prologues of V8's GCs
> >
> > I've removed synchronous Oilpan's GCs from the epilogues of V8's major GCs
> > but it didn't fix the regression of v8_gc_total metrics caused by Oilpan's shipping.
> >
> > This CL tries to remove page navigation GCs from the prologues of V8's GCs.
> > I confirmed that this doesn't cause regression in memory.top_7_stress.
> >
> > BUG=582831
> >
> > Committed: https://crrev.com/00a90c6253df77840fcf2a638cfa4346a71479be
> > Cr-Commit-Position: refs/heads/master@{#374395}
>
> TBR=sigbjornf@opera.com,haraken@chromium.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=582831
>
> Committed: https://crrev.com/328df273bffd0dbd30c3ecd681a4249be9020825
> Cr-Commit-Position: refs/heads/master@{#374464}
TBR=sigbjornf@opera.com,haraken@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=582831
Committed: https://crrev.com/3501f840dc01a9f96a5b7442b2ef9abe842eaf74
Cr-Commit-Position: refs/heads/master@{#374569}
Patch Set 1 #
Messages
Total messages: 6 (1 generated)
|
|||||||||||||||||||