Handlify factory methods for typed array, ArrayBuffer and DataView.

Handlify factory methods for typed array, ArrayBuffer and DataView.

These factory methods used pointers for constructor fucntions, therefore
those pointers could corrupt if allocation triggered gc.

R=mstarzinger@chromium.org

Committed: https://code.google.com/p/v8/source/detail?r=16084

[Dmitry Lomov (no reviews), 2013-08-06 17:05:42]

Daniel/Michael, please take a look.
This is a cause of WebAudio test failures after I tried to land my Blink patch.
I validated that after this, all webaudio tests pass at all times.

In related news, asan is awesome.

[Michael Starzinger, 2013-08-06 18:05:45]

LGTM (with a nit).

https://codereview.chromium.org/22426003/diff/1/src/factory.cc
File src/factory.cc (right):

https://codereview.chromium.org/22426003/diff/1/src/factory.cc#newcode1159
src/factory.cc:1159: GetTypedArrayFun(type, native_context));
nit: Instead of passing "Context*" into GetTypedArrayFun() can we instead pass
in "Isolate*"? Just to prevent accidental use of the potentially stale
"native_context" variable here.

[Dmitry Lomov (no reviews), 2013-08-06 19:14:56]

Committed patchset #1 manually as r16084 (presubmit successful).

[Dmitry Lomov (no reviews), 2013-08-06 19:15:09]

https://codereview.chromium.org/22426003/diff/1/src/factory.cc
File src/factory.cc (right):

https://codereview.chromium.org/22426003/diff/1/src/factory.cc#newcode1159
src/factory.cc:1159: GetTypedArrayFun(type, native_context));
On 2013/08/06 18:05:46, Michael Starzinger wrote:
> nit: Instead of passing "Context*" into GetTypedArrayFun() can we instead pass
> in "Isolate*"? Just to prevent accidental use of the potentially stale
> "native_context" variable here.

Done, great idea - thanks!