Fix _wrapDartFunction to call the JS set method directly. At one point this code was previously sur…

Fix _wrapDartFunction to call the JS set method directly. At one point this code was previously surprisingly generating a direct call to set instead of a call to dsetindex.

BUG=
R=vsm@google.com

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

[Jacob, 2016-11-15 20:43:29]

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

[Jacob, 2016-11-15 20:43:30]

Strange the original code ever generated a call to set. The JS WeakMap we are
operating on was never given a type.

[vsm, 2016-11-15 20:48:46]

lgtm

[Jacob, 2016-11-15 20:52:32]

Description was changed from

==========
Fix _wrapDartFunction to call the JS set method directly. At one point this code
was previously surprisingly generating a direct call to set instead of a call to
dsetindex.

BUG=
==========

to

==========
Fix _wrapDartFunction to call the JS set method directly. At one point this code
was previously surprisingly generating a direct call to set instead of a call to
dsetindex.

BUG=
R=vsm@google.com

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

[Jacob, 2016-11-15 20:52:33]

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