Introduce FlexibleArrayBufferView and TypedFlexibleArrayBufferView

Source/bindings/core/v8/V8BindingMacros.h

 /*
  * Copyright (C) 2010 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 28 matching lines @@
 #define TOSTRING_VOID(type, var, value) \
     type var(value);                    \
     if (UNLIKELY(!var.prepare()))       \
         return;
 
 #define TOSTRING_DEFAULT(type, var, value, retVal) \
     type var(value);                               \
     if (UNLIKELY(!var.prepare()))                  \
         return retVal;
 
+#define SMALL_ARRAY_BUFFER_VIEW_STORAGE(value) \

[haraken, 2015/07/30 14:54:57]

I'd rename this to allocateFlexibleArrayBufferViewStorage().

[haraken, 2015/07/30 14:54:57]

Add a comment about what this is doing.

[Michael Lippautz, 2015/07/30 17:06:14]

Done.

[Michael Lippautz, 2015/07/30 17:06:15]

Done.

+    (value->IsArrayBufferView() && (value.As<v8::ArrayBufferView>()->ByteLength() <= 128) ? alloca(value.As<v8::ArrayBufferView>()->ByteLength()) : nullptr)

[jochen (gone - plz use gerrit), 2015/07/30 14:30:06]

should we change the default for on-heap typed arrays in v8 to <= 128 as well?

[Michael Lippautz, 2015/07/30 17:06:14]

Changed to 64. Let's keep it in sync with V8's typed_array_max_size_heap.

+
+

[jochen (gone - plz use gerrit), 2015/07/30 14:30:06]

nit. only one empty line

[Michael Lippautz, 2015/07/30 17:06:15]

Done.

 template <typename T>
 inline bool v8Call(v8::Maybe<T> maybe, T& outVariable)
 {
     if (maybe.IsNothing())
         return false;
     outVariable = maybe.FromJust();
     return true;
 }
 
 inline bool v8CallBoolean(v8::Maybe<bool> maybe)
@@ skipping 32 matching lines @@
 
 // The last "else" is to avoid dangling else problem.
 #define V8_CALL(outVariable, handle, methodCall, failureExpression)                \
     if (handle.IsEmpty() || !v8Call(handle->methodCall, outVariable)) { \
         failureExpression;                                                         \
     } else
 
 } // namespace blink
 
 #endif // V8BindingMacros_h