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

Unified Diff: dart/tests/try/web/incremental_compilation_update_test.dart

Issue 739513002: Incremental compilation of removed class. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge
Patch Set: Created 6 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
Index: dart/tests/try/web/incremental_compilation_update_test.dart
diff --git a/dart/tests/try/web/incremental_compilation_update_test.dart b/dart/tests/try/web/incremental_compilation_update_test.dart
index 5484f884600926fe40b3afe1e91ca13f70696f6c..46b00be22cbb3439f3f93743ffd73a422399116c 100644
--- a/dart/tests/try/web/incremental_compilation_update_test.dart
+++ b/dart/tests/try/web/incremental_compilation_update_test.dart
@@ -813,6 +813,39 @@ main() {
""",
const <String>['v2']),
],
+
+ // Test that removing a class is supported.
+ // TODO(ahe): There should be a warning from second program, but there
+ // isn't. Investigate and write test.
Johnni Winther 2014/11/19 12:43:53 All a test in which a static property of C is used
ahe 2014/11/21 10:47:45 Done.
+ const <ProgramResult>[
+ const ProgramResult(
+ r"""
+class C {
+}
+
+main() {
+ try {
+ new C();
+ print('v1');
+ } catch (e) {
+ print('v2');
+ }
+}
+""",
+ const <String>['v1']),
+ const ProgramResult(
+ r"""
+main() {
+ try {
+ new C();
+ print('v1');
+ } catch (e) {
+ print('v2');
+ }
+}
+""",
+ const <String>['v2']),
+ ],
];
void main() {

Powered by Google App Engine
This is Rietveld 408576698