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

Issue 39333: Fix test to run in both snapshot and non-snapshot case (Closed)

Created:
11 years, 9 months ago by Søren Thygesen Gjesse
Modified:
9 years, 7 months ago
CC:
v8-dev
Visibility:
Public.

Description

Change the tests which looks at the number of scripts in the VM after startup to ignore native scripts without name. Native scripts without name are currently only the script holding the empty script and the script holding the source for the empty function. These two unnamed scripts are created for each context. When running with snapshot an additional context is created during startup adding two more native scripts that in the non-snapshot case. Committed: http://code.google.com/p/v8/source/detail?r=1448

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+6 lines, -8 lines) Patch
M test/mjsunit/debug-script.js View 1 chunk +6 lines, -8 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
Søren Thygesen Gjesse
Make the tree less tired.
11 years, 9 months ago (2009-03-09 10:28:50 UTC) #1
Mads Ager (chromium)
11 years, 9 months ago (2009-03-09 10:31:28 UTC) #2
LGTM.

Powered by Google App Engine
This is Rietveld 408576698