Commit 41c9adff authored by yurys@chromium.org's avatar yurys@chromium.org

Relax test expectations for test-cpu-profiler/FunctionApplySample

The profile may look a bit different on the bots (see below). We expected this only under GC stress testing, the change makes regular expectations the same.

[Top down]:
   68     0   (root) [-1] #0 1
    1     1    (program) [-1] #0 2
   67    65    start [-1] #16 3
    2     0      (unresolved function) [-1] #0 4
    2     2        apply [-1] #0 5

BUG=None
TBR=jkummerow@chromium.org

Review URL: https://codereview.chromium.org/18331012

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@15470 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
parent 45264ffb
......@@ -1169,13 +1169,7 @@ TEST(FunctionApplySample) {
CheckChildrenNames(root, names);
}
// In case of GC stress tests all samples may be in GC phase and there
// won't be |start| node in the profile.
bool is_gc_stress_testing =
(i::FLAG_gc_interval != -1) || i::FLAG_stress_compaction;
const v8::CpuProfileNode* startNode = FindChild(root, "start");
CHECK(is_gc_stress_testing || startNode);
if (startNode) {
{
ScopedVector<v8::Handle<v8::String> > names(2);
......@@ -1185,7 +1179,6 @@ TEST(FunctionApplySample) {
}
const v8::CpuProfileNode* testNode = FindChild(startNode, "test");
CHECK(is_gc_stress_testing || testNode);
if (testNode) {
ScopedVector<v8::Handle<v8::String> > names(2);
names[0] = v8::String::New("bar");
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment