Fix deferred loading with multiple apps on same page.

Fix deferred loading with multiple apps on same page.

Even though two apps don't have to load a file twice, they still both have to initialize it.

Still missing this support for the new emitter.

BUG=
R=floitsch@google.com

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

[sigurdm, 2015-01-07 12:09:08]

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

[floitsch, 2015-01-07 13:04:32]

LGTM.

https://codereview.chromium.org/837903002/diff/20001/pkg/compiler/lib/src/js_...
File pkg/compiler/lib/src/js_emitter/old_emitter/emitter.dart (right):

https://codereview.chromium.org/837903002/diff/20001/pkg/compiler/lib/src/js_...
pkg/compiler/lib/src/js_emitter/old_emitter/emitter.dart:1832:
buffer.write(jsAst.prettyPrint(
Put all these statements together into one string.

buffer.write(jsAst.prettyPrint(js.statement('''
#isHunkLoaded = ...
#... = ...
#...

https://codereview.chromium.org/837903002/diff/20001/pkg/compiler/lib/src/js_...
pkg/compiler/lib/src/js_emitter/old_emitter/emitter.dart:1838:
js.statement('$initName.$deferredInitializedName = new Object(null);'),
this looks like a bad "initName" of an embedded global. These are deprecated.

[sigurdm, 2015-01-07 13:37:13]

https://codereview.chromium.org/837903002/diff/20001/pkg/compiler/lib/src/js_...
File pkg/compiler/lib/src/js_emitter/old_emitter/emitter.dart (right):

https://codereview.chromium.org/837903002/diff/20001/pkg/compiler/lib/src/js_...
pkg/compiler/lib/src/js_emitter/old_emitter/emitter.dart:1832:
buffer.write(jsAst.prettyPrint(
On 2015/01/07 13:04:31, floitsch wrote:
> Put all these statements together into one string.
> 
> buffer.write(jsAst.prettyPrint(js.statement('''
> #isHunkLoaded = ...
> #... = ...
> #...

Done.

https://codereview.chromium.org/837903002/diff/20001/pkg/compiler/lib/src/js_...
pkg/compiler/lib/src/js_emitter/old_emitter/emitter.dart:1838:
js.statement('$initName.$deferredInitializedName = new Object(null);'),
On 2015/01/07 13:04:31, floitsch wrote:
> this looks like a bad "initName" of an embedded global. These are deprecated.

Changed it to an embedded global.

[sigurdm, 2015-01-07 14:06:01]

Committed patchset #3 (id:40001) manually as 42660 (presubmit successful).