AuroraMiddleware/v8
1
0
Fork 0
Code Issues 2 Pull Requests Releases Wiki Activity

Allow for no samples in test-cpu-profiler/SampleWhenFrameIsNotSetup

Browse Source 
The test should only check that there are no sample stacks that never possible in the JS code being profiled.

BUG=v8:2628
R=svenpanne@chromium.org

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

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14723 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
This commit is contained in:
yurys@chromium.org 2013-05-17 15:10:25 +00:00
parent f61bec13c0
commit 3ad62f5ee1
2 changed files with 5 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
test/cctest/cctest.status
View File

@ -39,10 +39,6 @@ test-api/ApplyInterruption: PASS || TIMEOUT
# when snapshot is on, so I am marking it PASS || FAIL
test-heap-profiler/HeapSnapshotsDiff: PASS || FAIL
# BUG(2628): These tests are flaky and sometimes fail, but should not crash.
test-cpu-profiler/CollectCpuProfile: PASS || FAIL
test-cpu-profiler/SampleWhenFrameIsNotSetup: PASS || FAIL
# These tests always fail. They are here to test test.py. If
# they don't fail then test.py has failed.
test-serialize/TestThatAlwaysFails: FAIL
@ -87,7 +83,8 @@ test-serialize/DeserializeFromSecondSerialization: SKIP
# BUG(2628): Signal may come when pc is close to frame enter/exit code and on
# simulator the stack frame is not set up when it is expected to be for the pc
# value.
test-cpu-profiler/SampleWhenFrameIsNotSetup: SKIP
test-cpu-profiler/CollectCpuProfile: PASS || FAIL
test-cpu-profiler/SampleWhenFrameIsNotSetup: PASS || FAIL
##############################################################################
[ $arch == android_arm || $arch == android_ia32 ]

4
test/cctest/test-cpu-profiler.cc
View File

@ -608,7 +608,9 @@ TEST(SampleWhenFrameIsNotSetup) {
CheckChildrenNames(root, names);
const v8::CpuProfileNode* startNode = FindChild(root, "start");
if (startNode->GetChildrenCount() > 0) {
// On slow machines there may be no meaningfull samples at all, skip the
// check there.
if (startNode && startNode->GetChildrenCount() > 0) {
CHECK_EQ(1, startNode->GetChildrenCount());
const v8::CpuProfileNode* delayNode = FindChild(startNode, "delay");
if (delayNode->GetChildrenCount() > 0) {