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

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: Rebased Created 7 years, 7 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 | « tests/compiler/dart2js/dart_backend_test.dart ('k') | tests/compiler/dart2js/type_checker_test.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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..588e77f6d84012ffaa43369f93aad6594151aa7b 100644
--- a/tests/compiler/dart2js/deprecated_features_test.dart
+++ b/tests/compiler/dart2js/deprecated_features_test.dart
@@ -43,7 +43,7 @@ main() {
throw 'Compilation failed: ${messages}';
}
Expect.stringEquals(
- // This string is comprised of lines of the following format:
+ // This string is composed of lines of the following format:
//
// offset<source>:path:kind: message
//
@@ -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?
« no previous file with comments | « tests/compiler/dart2js/dart_backend_test.dart ('k') | tests/compiler/dart2js/type_checker_test.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698