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

Unified Diff: tests/html/custom/attribute_changed_callback_test.dart

Issue 1054863002: CssClassSet upgrade (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Fix webcomponents test, IE toggle. Created 5 years, 9 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 | « sdk/lib/html/dartium/html_dartium.dart ('k') | tests/html/element_classes_svg_test.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tests/html/custom/attribute_changed_callback_test.dart
diff --git a/tests/html/custom/attribute_changed_callback_test.dart b/tests/html/custom/attribute_changed_callback_test.dart
index 3eaa446f92d41c2033f3f40d1ea8ce277462ee79..71ffaec3fbc3bb06b4d8f32af34b26b49b2b70c1 100644
--- a/tests/html/custom/attribute_changed_callback_test.dart
+++ b/tests/html/custom/attribute_changed_callback_test.dart
@@ -84,9 +84,7 @@ main() {
b.attributes.remove('data-v');
expect(B.invocations, ['data-v: x => null']);
});
- });
- group('unsupported_on_polyfill', () {
test('add, change ID', () {
B.invocations = [];
@@ -98,6 +96,12 @@ main() {
b.attributes.remove('id');
expect(B.invocations, ['id: x => null']);
});
+ });
+
+ group('unsupported_on_polyfill', () {
+
+ // If these tests start passing, don't remove the status suppression. Move
+ // the tests to the fullYy_supported group.
test('add, change classes', () {
var b = new B();
« no previous file with comments | « sdk/lib/html/dartium/html_dartium.dart ('k') | tests/html/element_classes_svg_test.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698