Remove unreachable code in mirrors.

Remove unreachable code in mirrors.

The only case where we do not set the name in the constructor is for anonymous
mixin applications.

BUG=
R=asiva@google.com, rmacnak@google.com

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

[rmacnak, 2013-09-10 23:38:51]

LGTM with comment

https://codereview.chromium.org/23531037/diff/1/runtime/lib/mirrors_impl.dart
File runtime/lib/mirrors_impl.dart (right):

https://codereview.chromium.org/23531037/diff/1/runtime/lib/mirrors_impl.dart...
runtime/lib/mirrors_impl.dart:463: // dynamic, void and the function types have
their names set eagerly in the
Comment out-of-sync.  All but anonymous mixin applications have their name set
at construction.

[Michael Lippautz (Google), 2013-09-11 01:01:15]

https://codereview.chromium.org/23531037/diff/1/runtime/lib/mirrors_impl.dart
File runtime/lib/mirrors_impl.dart (right):

https://codereview.chromium.org/23531037/diff/1/runtime/lib/mirrors_impl.dart...
runtime/lib/mirrors_impl.dart:463: // dynamic, void and the function types have
their names set eagerly in the
On 2013/09/10 23:38:51, Ryan Macnak wrote:
> Comment out-of-sync.  All but anonymous mixin applications have their name set
> at construction.

Done.

[siva, 2013-09-11 01:06:52]

lgtm

[Michael Lippautz (Google), 2013-09-11 16:10:49]

Committed patchset #2 manually as r27390 (presubmit successful).