Re-enable support for --disallow-unsafe-eval.

Re-enable support for --disallow-unsafe-eval.

R=erikcorry@google.com,ahe@google.com
BUG=

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

[kasperl, 2013-02-28 12:59:14]

Notice that this can now be tested using:

   $ test.py --csp ...

and if you're running the tests in a browser the Content-Security-Policy HTTP
header will be sent.

[erikcorry, 2013-02-28 13:06:08]

LGTM

[ngeoffray, 2013-02-28 13:11:31]

https://codereview.chromium.org/12388018/diff/3001/sdk/lib/_internal/compiler...
File sdk/lib/_internal/compiler/implementation/js_backend/emitter_no_eval.dart
(right):

https://codereview.chromium.org/12388018/diff/3001/sdk/lib/_internal/compiler...
sdk/lib/_internal/compiler/implementation/js_backend/emitter_no_eval.dart:34: //
Class fields are dynamically generated so they are have
are have -> have

https://codereview.chromium.org/12388018/diff/3001/sdk/lib/_internal/compiler...
sdk/lib/_internal/compiler/implementation/js_backend/emitter_no_eval.dart:98: //
function(prototype, staticName, fieldName, getterName, lazyValue, getter) {
line too long

https://codereview.chromium.org/12388018/diff/3001/sdk/lib/_internal/compiler...
sdk/lib/_internal/compiler/implementation/js_backend/emitter_no_eval.dart:106:
// all its field with the initial (and often final) value of all globals.
field -> fields

[kasperl, 2013-02-28 13:18:48]

https://codereview.chromium.org/12388018/diff/3001/sdk/lib/_internal/compiler...
File sdk/lib/_internal/compiler/implementation/js_backend/emitter_no_eval.dart
(right):

https://codereview.chromium.org/12388018/diff/3001/sdk/lib/_internal/compiler...
sdk/lib/_internal/compiler/implementation/js_backend/emitter_no_eval.dart:34: //
Class fields are dynamically generated so they are have
On 2013/02/28 13:11:31, ngeoffray wrote:
> are have -> have

Done.

https://codereview.chromium.org/12388018/diff/3001/sdk/lib/_internal/compiler...
sdk/lib/_internal/compiler/implementation/js_backend/emitter_no_eval.dart:98: //
function(prototype, staticName, fieldName, getterName, lazyValue, getter) {
On 2013/02/28 13:11:31, ngeoffray wrote:
> line too long

Done.

https://codereview.chromium.org/12388018/diff/3001/sdk/lib/_internal/compiler...
sdk/lib/_internal/compiler/implementation/js_backend/emitter_no_eval.dart:106:
// all its field with the initial (and often final) value of all globals.
On 2013/02/28 13:11:31, ngeoffray wrote:
> field -> fields

Done.

[kasperl, 2013-02-28 13:21:44]

Committed patchset #4 manually as r19219 (presubmit successful).