dart2js: Fast fix for deferred globals.

dart2js: Fast fix for deferred globals.

The fix is not perfect: it uses "arguments[1]" to avoid coming up with a fresh name.

Global variables are still allocated in the main-unit. If they are initialized with a constant value, they drag in that value into the main unit. (see http://dartbug.com/21301). That bug should, however, affect very few users.

BUG= http://dartbug.com/21293
R=cbracken@google.com

Committed: https://code.google.com/p/dart/source/detail?r=41056

[floitsch, 2014-10-10 22:10:21]

floitsch@google.com changed reviewers:
+ cbracken@google.com, sigurdm@google.com

[floitsch, 2014-10-10 22:12:21]

https://codereview.chromium.org/646083002/diff/70001/sdk/lib/_internal/compil...
File
sdk/lib/_internal/compiler/implementation/js_emitter/old_emitter/emitter.dart
(right):

https://codereview.chromium.org/646083002/diff/70001/sdk/lib/_internal/compil...
sdk/lib/_internal/compiler/implementation/js_emitter/old_emitter/emitter.dart:1839:
..write('var ${namer.currentIsolate}$_=$_$isolatePropertiesName$N')
This is, where we set the namer.currentIsolate variable to the
isolate-properties (reusing the short name we reserve for the current isolate).

[Chris Bracken, 2014-10-10 23:34:47]

lgtm - thanks.

[floitsch, 2014-10-10 23:53:17]

Committed patchset #2 (id:70001) manually as 41056 (presubmit successful).