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

Side by Side Diff: tests/compiler/dart2js_native/native_field_name_test.dart

Issue 11453032: Reapply class/method/field minification (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Review feedback Created 8 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) 2011, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2011, 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 // Check that native fields are not incorrectly renamed. 5 // Check that native fields are not incorrectly renamed.
6 6
7 class A native "*A" { 7 class A native "*A" {
8 int myLongPropertyName; 8 int myLongPropertyName;
9 int getValue; 9 int getValue;
10 10
11 int method(int z) => myLongPropertyName; 11 int method(int z) => myLongPropertyName;
12 } 12 }
13 13
14 14
15 // This code is inside the setup function, so the function names are not 15 // This code is inside the setup function, so the function names are not
16 // accessible, but the makeA variable is global through the magic of JS scoping. 16 // accessible, but the makeA variable is global through the magic of JS scoping.
17 // The contents of this are of course not analyzable by the compiler. 17 // The contents of this are of course not analyzable by the compiler.
18 void setup() native r""" 18 void setup() native r"""
19 function getter() { 19 function getter() {
20 return ++this.getValue; 20 return ++this.getValue;
21 } 21 }
22 22
23 function setter(x) { 23 function setter(x) {
24 this.getValue += 10; 24 this.getValue += 10;
25 } 25 }
(...skipping 28 matching lines...) Expand all
54 // Inside this 'if' the compiler knows that a2 is an A, so it is tempted 54 // Inside this 'if' the compiler knows that a2 is an A, so it is tempted
55 // to access myLongPropertyName directly, using its minified name. But 55 // to access myLongPropertyName directly, using its minified name. But
56 // renaming of native properties can only work using getters and setters 56 // renaming of native properties can only work using getters and setters
57 // that access the original name. 57 // that access the original name.
58 a2.myLongPropertyName = 21; 58 a2.myLongPropertyName = 21;
59 int gotten = a2.myLongPropertyName; 59 int gotten = a2.myLongPropertyName;
60 Expect.equals(11, gotten); 60 Expect.equals(11, gotten);
61 } 61 }
62 } 62 }
63 63
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698