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

Unified Diff: tests/compiler/dart2js_native/native_mixin_field_test.dart

Issue 62373009: Field property naming fix - issues 14096, 14806 (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: code review changes Created 7 years, 1 month 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 | « tests/compiler/dart2js_native/dart2js_native.status ('k') | tests/language/language_dart2js.status » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tests/compiler/dart2js_native/native_mixin_field_test.dart
diff --git a/tests/compiler/dart2js_native/native_mixin_field_test.dart b/tests/compiler/dart2js_native/native_mixin_field_test.dart
index 322381ff42bfd142179bcea16d13bd7a815c662d..8ccda280ee4c38f69ce131e86bd398a7a2b96bc0 100644
--- a/tests/compiler/dart2js_native/native_mixin_field_test.dart
+++ b/tests/compiler/dart2js_native/native_mixin_field_test.dart
@@ -16,7 +16,7 @@ class B extends A with M1, M2 native "B" {
}
class M1 {
- var baz;
+ var baz; // This field is not a native field, even when mixed in.
}
class M2 {
@@ -45,7 +45,8 @@ main() {
B b = makeB();
Expect.equals("A-foo", b.foo);
Expect.equals("B-bar", b.bar);
- Expect.equals("M1-baz", b.baz);
+ // Expect.equals("M1-baz", b.baz); // not true, see M1.
+ Expect.isNull(b.baz); // native b.baz is not the same as dart b.baz.
Expect.isNull(b.buz);
M1 m1 = new M1();
« no previous file with comments | « tests/compiler/dart2js_native/dart2js_native.status ('k') | tests/language/language_dart2js.status » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698