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

Unified Diff: tests/compiler/dart2js/deprecated_features_test.dart

Issue 14907008: Remove support for interface in dart2js. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Fix constructor name bug. Created 7 years, 8 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: tests/compiler/dart2js/deprecated_features_test.dart
diff --git a/tests/compiler/dart2js/deprecated_features_test.dart b/tests/compiler/dart2js/deprecated_features_test.dart
index bb884bf15bae39d066889f3861b588aeb8d2370c..282e0b3ab344cca4d21b12485cb6ae38baccf00d 100644
--- a/tests/compiler/dart2js/deprecated_features_test.dart
+++ b/tests/compiler/dart2js/deprecated_features_test.dart
@@ -56,20 +56,15 @@ main() {
// short-term solution and should eventually changed to include
// a symbolic reference to a MessageKind.
"0<#library('test');>::${deprecatedMessage('# tags')}\n"
- "38<interface>::${deprecatedMessage('interface declarations')}\n"
"19<part 'part.dart';>::${deprecatedMessage('missing part-of tag')}\n"
"0<>:/part.dart:info: Note: This file has no part-of tag, but it is being"
" used as a part.\n"
- "163<Fisk>::${deprecatedMessage('interface factories')}\n"
-
- // TODO(ahe): Should be <Fisk.hest>.
- "183<Fisk>::${deprecatedMessage('interface factories')}\n"
// TODO(ahe): Should be <bar>.
- "109<Foo>::${deprecatedMessage('conflicting constructor')}\n"
+ "52<Foo>::${deprecatedMessage('conflicting constructor')}\n"
- "129<bar>::info: This member conflicts with a constructor.\n"
- "205<()>::${deprecatedMessage('getter parameters')}\n",
+ "72<bar>::info: This member conflicts with a constructor.\n"
+ "103<()>::${deprecatedMessage('getter parameters')}\n",
messages.toString());
}
@@ -85,24 +80,15 @@ const Map<String, String> TEST_SOURCE =
part 'part.dart';
-interface Fisk default Foo {
- Fisk();
- Fisk.hest();
-}
-
class Foo {
Foo.bar();
static bar() => new Foo.bar();
- factory Fisk() {}
- factory Fisk.hest() {}
get x() => null;
}
main() {
var a = Foo.bar();
var b = new Foo.bar();
- new Fisk();
- new Fisk.hest();
}
""",
// TODO(ahe): Why isn't this 'part.dart'? Why the leading slash?

Powered by Google App Engine
This is Rietveld 408576698