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

Side by Side Diff: dart/tests/try/web/incremental_compilation_update_test.dart

Issue 755223002: Always enable RTI for incremental compilation. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge
Patch Set: Merged with r41955. Created 6 years 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2014, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2014, the Dart project authors. Please see the AUTHORS file
2 // for details. All rights reserved. Use of this source code is governed by a 2 // for details. All rights reserved. Use of this source code is governed by a
3 // BSD-style license that can be found in the LICENSE file. 3 // BSD-style license that can be found in the LICENSE file.
4 4
5 library trydart.incremental_compilation_update_test; 5 library trydart.incremental_compilation_update_test;
6 6
7 import 'dart:html' hide 7 import 'dart:html' hide
8 Element; 8 Element;
9 9
10 import 'dart:async' show 10 import 'dart:async' show
(...skipping 848 matching lines...) Expand 10 before | Expand all | Expand 10 after
859 } 859 }
860 } 860 }
861 """, 861 """,
862 const <String>['v2']), 862 const <String>['v2']),
863 ], 863 ],
864 864
865 // Test that removing a class is supported, using a static method. 865 // Test that removing a class is supported, using a static method.
866 const <ProgramResult>[ 866 const <ProgramResult>[
867 const ProgramResult( 867 const ProgramResult(
868 r""" 868 r"""
869 import 'dart:collection';
Johnni Winther 2014/12/01 08:50:19 Import in both ProgramResult's? The only differenc
ahe 2014/12/01 10:15:02 Below I use the MockCompiler to serialize scope in
870
869 class C { 871 class C {
870 static m() { 872 static m() {
871 print('v1'); 873 print('v1');
872 } 874 }
873 } 875 }
874 876
875 main() { 877 main() {
876 try { 878 // TODO(ahe): The incremental compiler magter ikke static tear-off closures.
Johnni Winther 2014/12/01 08:50:19 Elaborate on what goes wrong.
ahe 2014/12/01 10:15:02 Since I've fixed it already, I'd rather not get in
877 // TODO(ahe): Incremental compiler can't handle new noSuchMethod 879 new HashMap(equals: identical);
878 // situations, crashes when compiling a constructor which uses type
879 // arguments.
880 C.missing();
881 } catch (e) {
882 }
883 try { 880 try {
884 C.m(); 881 C.m();
885 } catch (e) { 882 } catch (e) {
886 print('v2'); 883 print('v2');
887 } 884 }
888 } 885 }
889 """, 886 """,
890 const <String>['v1']), 887 const <String>['v1']),
891 const ProgramResult( 888 const ProgramResult(
892 r""" 889 r"""
(...skipping 342 matching lines...) Expand 10 before | Expand all | Expand 10 after
1235 position: absolute; 1232 position: absolute;
1236 left: 0px; 1233 left: 0px;
1237 width: 3em; 1234 width: 3em;
1238 text-align: right; 1235 text-align: right;
1239 background-color: lightgoldenrodyellow; 1236 background-color: lightgoldenrodyellow;
1240 } 1237 }
1241 '''); 1238 ''');
1242 style.type = 'text/css'; 1239 style.type = 'text/css';
1243 return style; 1240 return style;
1244 } 1241 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698