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

Unified Diff: tests/isolate/isolate.status

Issue 11028076: Fix timer tests: timer tests were not using expectAsync to wait for (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Mark some tests as flaky in FF Created 8 years, 2 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
« no previous file with comments | « pkg/unittest/unittest.dart ('k') | tests/isolate/multiple_timer_test.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tests/isolate/isolate.status
diff --git a/tests/isolate/isolate.status b/tests/isolate/isolate.status
index bd9b3df8beb39202a644b784c8ac8cec65118751..016e2e9ba61532891385c0c3de5d3286b33a4a3e 100644
--- a/tests/isolate/isolate.status
+++ b/tests/isolate/isolate.status
@@ -37,6 +37,11 @@ mixed_test: Pass,Fail # Depends on 32/64 bit Safari. See Issue 112
mixed2_test: Pass,Fail # Depends on 32/64 bit Safari. See Issue 1120
message_test: Skip
+[ $runtime == ff ]
+# FF setTimeout can fire early: https://bugzilla.mozilla.org/show_bug.cgi?id=291386
+timer_test: Pass, Fail
+multiple_timer_test: Pass, Fail
+
# TODO(ager): Update these.
[ $runtime == ie && $system == windows ]
v2*: Skip
« no previous file with comments | « pkg/unittest/unittest.dart ('k') | tests/isolate/multiple_timer_test.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698