Move reflection support out of model.

Move reflection support out of model.

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

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

[floitsch, 2015-01-29 15:01:22]

LGTM.

https://codereview.chromium.org/891473002/diff/1/pkg/compiler/lib/src/js_emit...
File pkg/compiler/lib/src/js_emitter/old_emitter/container_builder.dart (right):

https://codereview.chromium.org/891473002/diff/1/pkg/compiler/lib/src/js_emit...
pkg/compiler/lib/src/js_emitter/old_emitter/container_builder.dart:257: final
bool canBeReflected = backend.isAccessibleByReflection(member) ||
I prefer if the comment is not on a sub-expression.
Maybe simply:

bool canBeReflected = backend.isAccessibleByReflection(member);

// During incremental compilation...
if (compiler.hasIncrementalSupport) canBeReflected = true;

[herhut, 2015-01-29 15:04:21]

https://codereview.chromium.org/891473002/diff/1/pkg/compiler/lib/src/js_emit...
File pkg/compiler/lib/src/js_emitter/old_emitter/container_builder.dart (right):

https://codereview.chromium.org/891473002/diff/1/pkg/compiler/lib/src/js_emit...
pkg/compiler/lib/src/js_emitter/old_emitter/container_builder.dart:257: final
bool canBeReflected = backend.isAccessibleByReflection(member) ||
On 2015/01/29 15:01:22, floitsch wrote:
> I prefer if the comment is not on a sub-expression.
> Maybe simply:
> 
> bool canBeReflected = backend.isAccessibleByReflection(member);
> 
> // During incremental compilation...
> if (compiler.hasIncrementalSupport) canBeReflected = true;

Done.

[herhut, 2015-01-29 15:05:12]

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