Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(194)

Issue 1284673002: Version 4.5.103.21 (cherry-pick) (Closed)

Created:
5 years, 4 months ago by Hannes Payer (out of office)
Modified:
5 years, 4 months ago
CC:
v8-dev, Paweł Hajdan Jr.
Base URL:
https://chromium.googlesource.com/v8/v8.git@4.5
Target Ref:
refs/pending/branch-heads/4.5
Project:
v8
Visibility:
Public.

Description

Version 4.5.103.21 (cherry-pick) Merged b4cfd60e07f53a7df3201c473929b17df29a202f Merged d8ad147944bf761f322482f68b804454321ac245 Merged 0cf86bd2c8372c49241b2f5df9fa2309b3c699fe Merged 9aff1d352d772ddcca8d317c31d6ced895bf115c Merged ddcec9f6bf50779bc23bf43b731c0699cc19c4ea Ensure the memory reduces makes progress. Grow heap slowly after running memory reducer. Use conservative heap growing factor for background tab. Perform full GC in background idle notification. Sample allocation rate in memory reducer. BUG=chromium:515873, chromium:518805 LOG=N R=hablich@chromium.org, jkummerow@chromium.org Committed: https://chromium.googlesource.com/v8/v8/+/6575237d3d0d9a8ef065c488db4ec4d742bc124b

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+144 lines, -51 lines) Patch
M include/v8-version.h View 1 chunk +1 line, -1 line 0 comments Download
M src/heap/heap.h View 1 chunk +4 lines, -0 lines 0 comments Download
M src/heap/heap.cc View 4 chunks +10 lines, -0 lines 0 comments Download
M src/heap/memory-reducer.h View 3 chunks +34 lines, -17 lines 0 comments Download
M src/heap/memory-reducer.cc View 7 chunks +43 lines, -21 lines 0 comments Download
M test/unittests/heap/memory-reducer-unittest.cc View 10 chunks +52 lines, -12 lines 0 comments Download

Messages

Total messages: 6 (1 generated)
Hannes Payer (out of office)
5 years, 4 months ago (2015-08-10 13:31:44 UTC) #1
Michael Hablich
On 2015/08/10 13:31:44, Hannes Payer wrote: lgtm
5 years, 4 months ago (2015-08-10 13:39:22 UTC) #2
Jakob Kummerow
lgtm
5 years, 4 months ago (2015-08-10 13:44:58 UTC) #4
Hannes Payer (out of office)
Committed patchset #1 (id:1) manually as 6575237d3d0d9a8ef065c488db4ec4d742bc124b (tree was closed).
5 years, 4 months ago (2015-08-10 13:53:52 UTC) #5
ulan
5 years, 4 months ago (2015-08-10 16:23:45 UTC) #6
Message was sent while issue was closed.
On 2015/08/10 13:53:52, Hannes Payer wrote:
> Committed patchset #1 (id:1) manually as
> 6575237d3d0d9a8ef065c488db4ec4d742bc124b (tree was closed).

lgtm, thank for merging!

Powered by Google App Engine
This is Rietveld 408576698