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

Side by Side Diff: frog/tests/leg/src/ClassCodegenTest.dart

Issue 10179001: Use shorter names for constructor arguments. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Make sure mangling is correct and update tests. Created 8 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2012, 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 // Test that parameters keep their names in the output. 4 // Test that parameters keep their names in the output.
5 5
6 #import("compiler_helper.dart"); 6 #import("compiler_helper.dart");
7 #import("parser_helper.dart"); 7 #import("parser_helper.dart");
8 8
9 final String TEST_ONE = @""" 9 final String TEST_ONE = @"""
10 class A { } 10 class A { }
(...skipping 65 matching lines...) Expand 10 before | Expand all | Expand 10 after
76 Expect.isTrue(generated.contains(@"Isolate.$inherits($.B, $.A);")); 76 Expect.isTrue(generated.contains(@"Isolate.$inherits($.B, $.A);"));
77 } 77 }
78 78
79 checkOutput(compileAll(TEST_TWO)); 79 checkOutput(compileAll(TEST_TWO));
80 checkOutput(compileAll(TEST_THREE)); 80 checkOutput(compileAll(TEST_THREE));
81 } 81 }
82 82
83 fieldTest() { 83 fieldTest() {
84 String generated = compileAll(TEST_FOUR); 84 String generated = compileAll(TEST_FOUR);
85 Expect.isTrue(generated.contains(@""" 85 Expect.isTrue(generated.contains(@"""
86 $.B = function B(B_z, B_y, A_x) { 86 $.B = function B(z, y, A_x) {
87 this.z = B_z; 87 this.z = z;
88 this.y = B_y; 88 this.y = y;
89 this.x = A_x; 89 this.x = A_x;
90 };""")); 90 };"""));
91 } 91 }
92 92
93 constructor1() { 93 constructor1() {
94 String generated = compileAll(TEST_FIVE); 94 String generated = compileAll(TEST_FIVE);
95 Expect.isTrue(generated.contains(@"new $.A(x);")); 95 Expect.isTrue(generated.contains(@"new $.A(x);"));
96 } 96 }
97 97
98 main() { 98 main() {
99 twoClasses(); 99 twoClasses();
100 subClass(); 100 subClass();
101 fieldTest(); 101 fieldTest();
102 constructor1(); 102 constructor1();
103 } 103 }
OLDNEW
« no previous file with comments | « no previous file | lib/compiler/implementation/emitter.dart » ('j') | lib/compiler/implementation/namer.dart » ('J')

Powered by Google App Engine
This is Rietveld 408576698