dart2Js: bailout early when using the new emitter with mirrors.

dart2Js: bailout early when using the new emitter with mirrors.

R=floitsch@google.com

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

[zarah, 2015-02-27 13:11:17]

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

[floitsch, 2015-02-27 14:05:51]

LGTM.

https://codereview.chromium.org/963753002/diff/1/pkg/compiler/lib/src/compile...
File pkg/compiler/lib/src/compiler.dart (right):

https://codereview.chromium.org/963753002/diff/1/pkg/compiler/lib/src/compile...
pkg/compiler/lib/src/compiler.dart:1224: internalError(library,
This should not be an internal error, but a "reportError" or similar. The string
should be in "warnings.dart".

I would put this close to the already existing warning in line 1325.

[zarah, 2015-03-02 10:42:37]

https://codereview.chromium.org/963753002/diff/1/pkg/compiler/lib/src/compile...
File pkg/compiler/lib/src/compiler.dart (right):

https://codereview.chromium.org/963753002/diff/1/pkg/compiler/lib/src/compile...
pkg/compiler/lib/src/compiler.dart:1224: internalError(library,
On 2015/02/27 14:05:50, floitsch wrote:
> This should not be an internal error, but a "reportError" or similar. The
string
> should be in "warnings.dart".
> 
> I would put this close to the already existing warning in line 1325.

Done.

[zarah, 2015-03-03 08:59:37]

Committed patchset #2 (id:20001) manually as 44160 (presubmit successful).