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

Issue 407093004: Set the state of a swept page after being done with sweeping. (Closed)

Created:
6 years, 5 months ago by Hannes Payer (out of office)
Modified:
6 years, 5 months ago
Reviewers:
Michael Achenbach
CC:
v8-dev
Project:
v8
Visibility:
Public.

Description

Set the state of a swept page after being done with sweeping. BUG= R=machenbach@chromium.org Committed: https://code.google.com/p/v8/source/detail?r=22532

Patch Set 1 #

Total comments: 2

Patch Set 2 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+40 lines, -16 lines) Patch
M src/mark-compact.cc View 5 chunks +22 lines, -14 lines 0 comments Download
M test/mjsunit/mjsunit.status View 1 chunk +10 lines, -2 lines 0 comments Download
M test/webkit/webkit.status View 1 1 chunk +8 lines, -0 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
Hannes Payer (out of office)
6 years, 5 months ago (2014-07-22 13:15:21 UTC) #1
Michael Achenbach
https://codereview.chromium.org/407093004/diff/1/test/webkit/webkit.status File test/webkit/webkit.status (right): https://codereview.chromium.org/407093004/diff/1/test/webkit/webkit.status#newcode55 test/webkit/webkit.status:55: 'fastt/js/excessive-comma-usage': [SKIP] one t
6 years, 5 months ago (2014-07-22 13:17:39 UTC) #2
Hannes Payer (out of office)
https://codereview.chromium.org/407093004/diff/1/test/webkit/webkit.status File test/webkit/webkit.status (right): https://codereview.chromium.org/407093004/diff/1/test/webkit/webkit.status#newcode55 test/webkit/webkit.status:55: 'fastt/js/excessive-comma-usage': [SKIP] On 2014/07/22 13:17:38, Michael Achenbach wrote: > ...
6 years, 5 months ago (2014-07-22 13:18:19 UTC) #3
Michael Achenbach
lgtm
6 years, 5 months ago (2014-07-22 14:01:00 UTC) #4
Hannes Payer (out of office)
6 years, 5 months ago (2014-07-22 16:30:30 UTC) #5
Message was sent while issue was closed.
Committed patchset #2 manually as r22532 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698