dart2js: Give the globals holder for deferred loading a fixed name.

dart2js: Give the globals holder for deferred loading a fixed name.

R=floitsch@google.com, sigurdm@google.com
BUG=

Committed: https://github.com/dart-lang/sdk/commit/5c4749ac92fbeed58c06112a56860df36a0a4b01

[herhut, 2015-07-03 08:40:14]

The globalsHolder is not a global name in the sense of the namer. It is a
top-level variable in the setup code but those are not tracked by naming.
Globals that come from the namer should always end up in a holder to avoid
clashes with local variables.

[herhut, 2015-07-03 08:45:03]

herhut@google.com changed reviewers:
+ floitsch@google.com

[herhut, 2015-07-03 08:45:23]

On 2015/07/03 08:40:14, herhut wrote:
> The globalsHolder is not a global name in the sense of the namer. It is a
> top-level variable in the setup code but those are not tracked by naming.
> Globals that come from the namer should always end up in a holder to avoid
> clashes with local variables.

[+floitsch]

[floitsch, 2015-07-03 10:26:21]

LGTM.

[herhut, 2015-07-03 10:46:59]

Committed patchset #1 (id:1) manually as
5c4749ac92fbeed58c06112a56860df36a0a4b01 (presubmit successful).

[sigurdm, 2015-07-08 06:58:11]

lgtm