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

Issue 2324493003: Merged: Squashed multiple commits. (Closed)

Created:
4 years, 3 months ago by Michael Lippautz
Modified:
4 years, 3 months ago
Reviewers:
Michael Hablich
CC:
v8-reviews_googlegroups.com, v8-merges_googlegroups.com, ulan, Hannes Payer (out of office)
Target Ref:
refs/pending/branch-heads/5.4
Project:
v8
Visibility:
Public.

Description

Merged: Squashed multiple commits. Merged: [heap] GCTracer: Record details for incremental marking Revision: 3866975f1b159f3703429ce50ca228bcdbf28306 Merged: [heap] GCTracer: Properly reset all members for unittests Revision: a4a4e7fa97ef6824ea72260201f71e058672433c Merged: [heap] GCTracer: Fix UB when iterating incremental scopes Revision: 33ffcc8f819d0cf8f86eabae6cd1860c693a4672 BUG=chromium:639818 LOG=N NOTRY=true NOPRESUBMIT=true NOTREECHECKS=true R=hablich@chromium.org

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+274 lines, -309 lines) Patch
M src/heap/gc-tracer.h View 14 chunks +53 lines, -96 lines 0 comments Download
M src/heap/gc-tracer.cc View 14 chunks +167 lines, -212 lines 0 comments Download
M src/heap/incremental-marking.cc View 3 chunks +2 lines, -1 line 0 comments Download
M test/unittests/heap/gc-tracer-unittest.cc View 1 chunk +52 lines, -0 lines 0 comments Download

Messages

Total messages: 9 (4 generated)
Michael Lippautz
4 years, 3 months ago (2016-09-08 10:33:53 UTC) #1
Michael Lippautz
NOTE: This script will no longer automatically update include/v8-version.h and create a tag. This is ...
4 years, 3 months ago (2016-09-08 10:33:54 UTC) #2
Michael Hablich
On 2016/09/08 10:33:54, Michael Lippautz wrote: > NOTE: This script will no longer automatically update ...
4 years, 3 months ago (2016-09-08 10:57:40 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2324493003/1
4 years, 3 months ago (2016-09-08 11:00:00 UTC) #7
commit-bot: I haz the power
4 years, 3 months ago (2016-09-08 11:00:16 UTC) #9
Message was sent while issue was closed.
Committed patchset #1 (id:1)

Powered by Google App Engine
This is Rietveld 408576698