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

Issue 135153007: Dispose Isolate at end of cctest. (Closed)

Created:
6 years, 10 months ago by Sven Panne
Modified:
6 years, 10 months ago
Reviewers:
ulan
CC:
v8-dev
Visibility:
Public.

Description

Dispose Isolate at end of cctest. No measurable change in test running time, we clean up only trivial known things at disposal time and do not walk the heap, run real finalizers, etc. R=ulan@chromium.org Committed: https://code.google.com/p/v8/source/detail?r=19047

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -2 lines) Patch
M test/cctest/cctest.h View 1 chunk +1 line, -2 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
Sven Panne
6 years, 10 months ago (2014-02-03 10:30:20 UTC) #1
ulan
lgtm
6 years, 10 months ago (2014-02-03 15:09:51 UTC) #2
Sven Panne
6 years, 10 months ago (2014-02-04 08:30:41 UTC) #3
Message was sent while issue was closed.
Committed patchset #1 manually as r19047 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698