| OLD | NEW | 
|---|
| 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 // Based on dartbug.com/7681 | 4 // Based on dartbug.com/7681 | 
| 5 // Verify that context chains do not lead to unintended memory being held. | 5 // Verify that context chains do not lead to unintended memory being held. | 
| 6 | 6 | 
| 7 library closure_cycles_test; | 7 library closure_cycles_test; | 
| 8 | 8 | 
| 9 import "dart:async"; | 9 import "dart:async"; | 
| 10 | 10 | 
| 11 class X { | 11 class X { | 
| 12   Function onX; | 12   Function onX; | 
| 13   X() { Timer.run(() => onX(new Y())); } | 13   X() { | 
|  | 14     Timer.run(() => onX(new Y())); | 
|  | 15   } | 
| 14 } | 16 } | 
| 15 | 17 | 
| 16 class Y { | 18 class Y { | 
| 17   Function onY; | 19   Function onY; | 
| 18   var heavyMemory; | 20   var heavyMemory; | 
| 19   static var count = 0; | 21   static var count = 0; | 
| 20   Y() { | 22   Y() { | 
| 21     // Consume large amounts of memory per iteration to fail/succeed quicker. | 23     // Consume large amounts of memory per iteration to fail/succeed quicker. | 
| 22     heavyMemory = new List(10*1024*1024); | 24     heavyMemory = new List(10 * 1024 * 1024); | 
| 23     // Terminate the test if we allocated enough memory without running out. | 25     // Terminate the test if we allocated enough memory without running out. | 
| 24     if (count++ > 100) return; | 26     if (count++ > 100) return; | 
| 25     Timer.run(() => onY()); | 27     Timer.run(() => onY()); | 
| 26   } | 28   } | 
| 27 } | 29 } | 
| 28 | 30 | 
| 29 void doIt() { | 31 void doIt() { | 
| 30   var x = new X(); | 32   var x = new X(); | 
| 31   x.onX = (y) { | 33   x.onX = (y) { | 
| 32     y.onY = () { | 34     y.onY = () { | 
| 33       y;  // Capturing y can lead to endless context chains! | 35       y; // Capturing y can lead to endless context chains! | 
| 34       doIt(); | 36       doIt(); | 
| 35     }; | 37     }; | 
| 36   }; | 38   }; | 
| 37 } | 39 } | 
| 38 | 40 | 
| 39 void main() { | 41 void main() { | 
| 40   doIt(); | 42   doIt(); | 
| 41 } | 43 } | 
| OLD | NEW | 
|---|