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

Unified Diff: test/cctest/cctest.status

Issue 596533002: Initial implementation of GetStackSample sampling profiler API. (Closed) Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: Created 6 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: test/cctest/cctest.status
diff --git a/test/cctest/cctest.status b/test/cctest/cctest.status
index 44ad3aeb737def958912aa482e5878274d62ccca..cc167ae702d9d480c5dfeabd30faa4bb3421c012 100644
--- a/test/cctest/cctest.status
+++ b/test/cctest/cctest.status
@@ -258,6 +258,10 @@
'test-debug/DebugGetLoadedScripts': [PASS, FLAKY],
'test-debug/DebugStepLinear': [PASS, FLAKY],
'test-debug/DebuggerClearMessageHandler': [PASS, FLAKY],
+
+ # TODO(alph): add support for MacOS.
+ 'test-sampler-api/*': [SKIP],
+
}], # 'system == macos'
##############################################################################
@@ -345,6 +349,11 @@
# This test times out.
'test-threads/ThreadJoinSelf': [SKIP],
+
+ # TODO(alph): Android doesn't seem to support getcontext, so disable the test.
+ # The test could be implemented with POSIX signals.
+ 'test-sampler-api/*': [SKIP],
gholap 2014/09/23 05:39:18 https://codereview.chromium.org/588623002/ uses PO
alph 2014/09/23 07:30:46 Let's address arm tests in a separate patch.
+
}], # 'arch == android_arm or arch == android_ia32'
##############################################################################

Powered by Google App Engine
This is Rietveld 408576698