Preemptive initialization of dispatch records

Preemptive initialization of dispatch records

R=blois@google.com

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

[sra1, 2013-10-23 04:35:27]

Pete,

I'm a bit nervous about patching all the prototypes like this.  Is there a
subset that should be patched if we want to patch less aggressively?

[blois, 2013-10-23 16:37:03]

Not that I can think of- what cases are you worried about?

https://codereview.chromium.org/35893008/diff/40001/sdk/lib/_internal/lib/nat...
File sdk/lib/_internal/lib/native_helper.dart (right):

https://codereview.chromium.org/35893008/diff/40001/sdk/lib/_internal/lib/nat...
sdk/lib/_internal/lib/native_helper.dart:327: if (JS('bool', 'typeof (#[#]) !=
"undefined"', context, tag)) {
Why not typeof (#[#]) == "function" and remove the later check?

[sra1, 2013-10-23 18:30:34]

https://codereview.chromium.org/35893008/diff/40001/sdk/lib/_internal/lib/nat...
File sdk/lib/_internal/lib/native_helper.dart (right):

https://codereview.chromium.org/35893008/diff/40001/sdk/lib/_internal/lib/nat...
sdk/lib/_internal/lib/native_helper.dart:327: if (JS('bool', 'typeof (#[#]) !=
"undefined"', context, tag)) {
On 2013/10/23 16:37:03, blois wrote:
> Why not typeof (#[#]) == "function" and remove the later check?

Done.

[blois, 2013-10-23 18:37:55]

lgtm

[sra1, 2013-10-23 19:11:26]

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