Chromium Code Reviews| Index: tests/compiler/dart2js/type_checker_test.dart | 
| diff --git a/tests/compiler/dart2js/type_checker_test.dart b/tests/compiler/dart2js/type_checker_test.dart | 
| index 7404c991d0437da36fc7e5b2ad73665071e90476..00c15d382a587563005ce28eca7c689518204618 100644 | 
| --- a/tests/compiler/dart2js/type_checker_test.dart | 
| +++ b/tests/compiler/dart2js/type_checker_test.dart | 
| @@ -13,8 +13,7 @@ import 'parser_helper.dart'; | 
| import '../../../sdk/lib/_internal/compiler/implementation/elements/modelx.dart' | 
| show ElementX, CompilationUnitElementX; | 
| -import '../../../sdk/lib/_internal/compiler/implementation/dart2jslib.dart' | 
| - hide SourceString; | 
| +import '../../../sdk/lib/_internal/compiler/implementation/dart2jslib.dart'; | 
| import '../../../sdk/lib/_internal/compiler/implementation/dart_types.dart'; | 
| @@ -472,13 +471,13 @@ testMethodInvocationsInClass() { | 
| LibraryElement library = mockLibrary(compiler, CLASS_WITH_METHODS); | 
| compiler.parseScript(CLASS_WITH_METHODS, library); | 
| ClassElement ClassWithMethods = | 
| - library.find(const SourceString("ClassWithMethods")); | 
| + library.find("ClassWithMethods"); | 
| 
 
ngeoffray
2013/10/18 10:19:37
One line?
 
lukas
2013/10/24 16:48:36
Done.
 
 | 
| ClassWithMethods.ensureResolved(compiler); | 
| - Element c = ClassWithMethods.lookupLocalMember(const SourceString('method')); | 
| + Element c = ClassWithMethods.lookupLocalMember('method'); | 
| assert(c != null); | 
| - ClassElement SubClass = library.find(const SourceString("SubClass")); | 
| + ClassElement SubClass = library.find("SubClass"); | 
| SubClass.ensureResolved(compiler); | 
| - Element d = SubClass.lookupLocalMember(const SourceString('method')); | 
| + Element d = SubClass.lookupLocalMember('method'); | 
| assert(d != null); | 
| check(Element element, String text, [expectedWarnings]){ | 
| @@ -740,9 +739,9 @@ testThis() { | 
| }"""; | 
| LibraryElement library = mockLibrary(compiler, script); | 
| compiler.parseScript(script, library); | 
| - ClassElement foo = library.find(const SourceString("Foo")); | 
| + ClassElement foo = library.find("Foo"); | 
| foo.ensureResolved(compiler); | 
| - Element method = foo.lookupLocalMember(const SourceString('method')); | 
| + Element method = foo.lookupLocalMember('method'); | 
| analyzeIn(method, "{ int i = this; }", NOT_ASSIGNABLE); | 
| analyzeIn(method, "{ Object o = this; }"); | 
| analyzeIn(method, "{ Foo f = this; }"); | 
| @@ -761,9 +760,9 @@ testSuper() { | 
| '''; | 
| LibraryElement library = mockLibrary(compiler, script); | 
| compiler.parseScript(script, library); | 
| - ClassElement B = library.find(const SourceString("B")); | 
| + ClassElement B = library.find("B"); | 
| B.ensureResolved(compiler); | 
| - Element method = B.lookupLocalMember(const SourceString('method')); | 
| + Element method = B.lookupLocalMember('method'); | 
| analyzeIn(method, "{ int i = super.field; }", NOT_ASSIGNABLE); | 
| analyzeIn(method, "{ Object o = super.field; }"); | 
| analyzeIn(method, "{ String s = super.field; }"); | 
| @@ -1062,9 +1061,9 @@ void testTypeVariableExpressions() { | 
| }"""; | 
| LibraryElement library = mockLibrary(compiler, script); | 
| compiler.parseScript(script, library); | 
| - ClassElement foo = library.find(const SourceString("Foo")); | 
| + ClassElement foo = library.find("Foo"); | 
| foo.ensureResolved(compiler); | 
| - Element method = foo.lookupLocalMember(const SourceString('method')); | 
| + Element method = foo.lookupLocalMember('method'); | 
| analyzeIn(method, "{ Type type = T; }"); | 
| analyzeIn(method, "{ T type = T; }", NOT_ASSIGNABLE); | 
| @@ -1095,10 +1094,10 @@ class Test<S extends Foo, T> { | 
| LibraryElement library = mockLibrary(compiler, script); | 
| compiler.parseScript(script, library); | 
| - ClassElement classTest = library.find(const SourceString("Test")); | 
| + ClassElement classTest = library.find("Test"); | 
| classTest.ensureResolved(compiler); | 
| FunctionElement methodTest = | 
| - classTest.lookupLocalMember(const SourceString("test")); | 
| + classTest.lookupLocalMember("test"); | 
| 
 
ngeoffray
2013/10/18 10:19:37
One line.
 
lukas
2013/10/24 16:48:36
Done.
 
 | 
| test(String expression, [message]) { | 
| analyzeIn(methodTest, "{ $expression; }", message); | 
| @@ -1137,10 +1136,10 @@ class Test<S extends T, T extends Foo> { | 
| LibraryElement library = mockLibrary(compiler, script); | 
| compiler.parseScript(script, library); | 
| - ClassElement classTest = library.find(const SourceString("Test")); | 
| + ClassElement classTest = library.find("Test"); | 
| classTest.ensureResolved(compiler); | 
| FunctionElement methodTest = | 
| - classTest.lookupLocalMember(const SourceString("test")); | 
| + classTest.lookupLocalMember("test"); | 
| 
 
ngeoffray
2013/10/18 10:19:37
One line.
 
lukas
2013/10/24 16:48:36
Done.
 
 | 
| test(String expression, [message]) { | 
| analyzeIn(methodTest, "{ $expression; }", message); | 
| @@ -1161,10 +1160,10 @@ class Test<S extends T, T extends S> { | 
| LibraryElement library = mockLibrary(compiler, script); | 
| compiler.parseScript(script, library); | 
| - ClassElement classTest = library.find(const SourceString("Test")); | 
| + ClassElement classTest = library.find("Test"); | 
| classTest.ensureResolved(compiler); | 
| FunctionElement methodTest = | 
| - classTest.lookupLocalMember(const SourceString("test")); | 
| + classTest.lookupLocalMember("test"); | 
| 
 
ngeoffray
2013/10/18 10:19:37
One line.
 
lukas
2013/10/24 16:48:36
Done.
 
 | 
| test(String expression, [message]) { | 
| analyzeIn(methodTest, "{ $expression; }", message); | 
| @@ -1809,7 +1808,7 @@ analyze(String text, {errors, warnings, List hints, List infos}) { | 
| Element compilationUnit = | 
| new CompilationUnitElementX(new Script(null, null), compiler.mainApp); | 
| Element function = new ElementX( | 
| - buildSourceString(''), ElementKind.FUNCTION, compilationUnit); | 
| + '', ElementKind.FUNCTION, compilationUnit); | 
| TreeElements elements = compiler.resolveNodeStatement(node, function); | 
| TypeCheckerVisitor checker = new TypeCheckerVisitor(compiler, elements, | 
| types); | 
| @@ -1827,8 +1826,8 @@ void generateOutput(String text) { | 
| var beginToken = message.node.getBeginToken(); | 
| var endToken = message.node.getEndToken(); | 
| int begin = beginToken.charOffset; | 
| - int end = endToken.charOffset + endToken.slowCharCount; | 
| - SourceFile sourceFile = new SourceFile('analysis', text); | 
| + int end = endToken.charOffset + endToken.charCount; | 
| + SourceFile sourceFile = new StringSourceFile('analysis', text); | 
| print(sourceFile.getLocationMessage(message.message.toString(), | 
| begin, end, true, (str) => str)); | 
| } |