One more minor mixin fix

One more minor mixin fix

We need to preserve the Mixin constructor to make sure metadata (e.g.,
implements) is injected in the right place.

R=jacobr@google.com

Committed: https://github.com/dart-lang/sdk/commit/92d39a4e0819adf4fed9c69fe9886884e753c449

[vsm, 2017-03-28 15:07:24]

Description was changed from

==========
One more minor mixin fix

We need to preserve the Mixin constructor to make sure metadata (e.g.,
implements) is injected in the right place.
==========

to

==========
One more minor mixin fix

We need to preserve the Mixin constructor to make sure metadata (e.g.,
implements) is injected in the right place.
==========

[vsm, 2017-03-28 15:07:24]

vsm@google.com changed reviewers:
+ jmesserly@google.com

[Jennifer Messerly, 2017-03-28 20:47:48]

lgtm but a suggestion

https://codereview.chromium.org/2779063002/diff/1/pkg/dev_compiler/tool/input...
File pkg/dev_compiler/tool/input_sdk/private/ddc_runtime/classes.dart (right):

https://codereview.chromium.org/2779063002/diff/1/pkg/dev_compiler/tool/input...
pkg/dev_compiler/tool/input_sdk/private/ddc_runtime/classes.dart:38:
$copyProperties(Mixin.prototype, m.prototype);
hmm, maybe just skip "constructor" when copying properties on this line, rather
than copy and copy back?

[vsm, 2017-05-18 23:58:25]

vsm@google.com changed reviewers:
+ leafp@google.com
- jmesserly@google.com

[vsm, 2017-05-19 14:27:43]

vsm@google.com changed reviewers:
+ jacobr@google.com

[Jacob, 2017-05-19 14:31:27]

lgtm

https://codereview.chromium.org/2779063002/diff/20001/pkg/dev_compiler/tool/i...
File pkg/dev_compiler/tool/input_sdk/private/ddc_runtime/classes.dart (right):

https://codereview.chromium.org/2779063002/diff/20001/pkg/dev_compiler/tool/i...
pkg/dev_compiler/tool/input_sdk/private/ddc_runtime/classes.dart:35: var
constructor = Mixin.prototype.constructor;
nit: let

[vsm, 2017-05-19 14:41:37]

Description was changed from

==========
One more minor mixin fix

We need to preserve the Mixin constructor to make sure metadata (e.g.,
implements) is injected in the right place.
==========

to

==========
One more minor mixin fix

We need to preserve the Mixin constructor to make sure metadata (e.g.,
implements) is injected in the right place.

R=jacobr@google.com

Committed:
https://github.com/dart-lang/sdk/commit/92d39a4e0819adf4fed9c69fe9886884e753c449
==========

[vsm, 2017-05-19 14:41:38]

Committed patchset #3 (id:20001) manually as
92d39a4e0819adf4fed9c69fe9886884e753c449 (presubmit successful).