Don't use 'new Function' for lazily initialized getter.

Don't use 'new Function' for lazily initialized getter.
Committed: https://code.google.com/p/dart/source/detail?r=15249

[floitsch, 2012-11-21 16:58:13]

https://codereview.chromium.org/11419119/diff/1/sdk/lib/_internal/compiler/im...
File sdk/lib/_internal/compiler/implementation/js_backend/emitter.dart (left):

https://codereview.chromium.org/11419119/diff/1/sdk/lib/_internal/compiler/im...
sdk/lib/_internal/compiler/implementation/js_backend/emitter.dart:332: var
getter = new Function("{ return $isolate." + fieldName + ";}");
Unfortunately no.
I'm pretty sure this would make lazily initialized functions much slower.
You need to special case this in the csp-emitter.
2 alternatives: what you just did (but only when emitting csp), or (faster but
more code) passing in the getter as argument to the lazyInitializerFunction.

[ngeoffray, 2012-11-22 11:19:28]

Thanks Florian for the feedback. PTAL

[floitsch, 2012-11-22 11:51:25]

LGTM.