Fix Array constructor bug impacting dart2js and ddc running on the same page.

Fix Array constructor bug impacting dart2js and ddc running on the same page.

BUG=
R=jmesserly@google.com

Committed: https://github.com/dart-lang/dev_compiler/commit/c1824da0666989637d4b016d0e56f0cefdd296c1

[Jacob, 2015-04-23 19:20:02]

jacobr@google.com changed reviewers:
+ jmesserly@google.comq

[Jennifer Messerly, 2015-04-23 19:33:53]

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

[Jennifer Messerly, 2015-04-23 19:33:54]

lgtm

https://codereview.chromium.org/1094243007/diff/1/lib/runtime/dart/core.js
File lib/runtime/dart/core.js (right):

https://codereview.chromium.org/1094243007/diff/1/lib/runtime/dart/core.js#ne...
lib/runtime/dart/core.js:1619:
dart.copyPropertySymbols(dart.global.Array.prototype, List.prototype);
dart.registerExtension(dart.global.Array, List)?

we might need this for the dynamic ops too. Seems nice to express the intent
here, rather than specifically how to implement it?

https://codereview.chromium.org/1094243007/diff/1/lib/runtime/dart_runtime.js
File lib/runtime/dart_runtime.js (right):

https://codereview.chromium.org/1094243007/diff/1/lib/runtime/dart_runtime.js...
lib/runtime/dart_runtime.js:614: function copyPropertiesHelper(to, from, names)
{
hmm, not sure this is really worth a helper.

https://codereview.chromium.org/1094243007/diff/1/lib/runtime/dart_runtime.js...
lib/runtime/dart_runtime.js:627: getOwnNamesAndSymbols(from));
short line? it looks like it would fit

[Jacob, 2015-04-23 20:19:43]

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