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

Side by Side Diff: LayoutTests/fast/js/script-tests/with-scope-gc.js

Issue 211933008: Remove multiple GC calls and replace GCController.collect calls with GCController.collectAll (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: revert change to worker-event-listener Created 6 years, 9 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 description( 1 description(
2 'Tests to make sure that dynamic scope objects are correctly protected from GC. To pass we need to not crash.' 2 'Tests to make sure that dynamic scope objects are correctly protected from GC. To pass we need to not crash.'
3 ); 3 );
4 4
5 function gc() 5 function gc()
6 { 6 {
7 if (this.GCController) 7 if (this.GCController)
8 GCController.collect(); 8 GCController.collectAll();
9 else 9 else
10 for (var i = 0; i < 10000; ++i) // Allocate a sufficient number of objec ts to force a GC. 10 for (var i = 0; i < 10000; ++i) // Allocate a sufficient number of objec ts to force a GC.
11 ({}); 11 ({});
12 } 12 }
13 13
14 (function() { 14 (function() {
15 try { 15 try {
16 // Immediate value for scope 16 // Immediate value for scope
17 with(1) { gc(); a; } 17 with(1) { gc(); a; }
18 } catch(e) { 18 } catch(e) {
(...skipping 19 matching lines...) Expand all
38 // Test catch blocks for the heck of it 38 // Test catch blocks for the heck of it
39 try { 39 try {
40 throw 1; 40 throw 1;
41 } catch(e) { 41 } catch(e) {
42 gc(); 42 gc();
43 b; 43 b;
44 } 44 }
45 } catch (e) { 45 } catch (e) {
46 } 46 }
47 })(); 47 })();
OLDNEW
« no previous file with comments | « LayoutTests/fast/forms/radio/radio-group.html ('k') | LayoutTests/fast/workers/worker-messageport-gc.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698