chakra-core
diff --git a/‎bin/ch/DbgController.js
Copy file name to clipboard
+2Lines changed: 2 additions & 0 deletions b/‎bin/ch/DbgController.js
Copy file name to clipboard
+2Lines changed: 2 additions & 0 deletions
diff --git a/‎lib/Common/ConfigFlagsList.h
Copy file name to clipboard
+2-5Lines changed: 2 additions & 5 deletions b/‎lib/Common/ConfigFlagsList.h
Copy file name to clipboard
+2-5Lines changed: 2 additions & 5 deletions
diff --git a/‎lib/Common/Memory/Allocator.h
Copy file name to clipboard
+10-5Lines changed: 10 additions & 5 deletions b/‎lib/Common/Memory/Allocator.h
Copy file name to clipboard
+10-5Lines changed: 10 additions & 5 deletions
diff --git a/‎lib/Common/Memory/Recycler.h
Copy file name to clipboard
+15-3Lines changed: 15 additions & 3 deletions b/‎lib/Common/Memory/Recycler.h
Copy file name to clipboard
+15-3Lines changed: 15 additions & 3 deletions
diff --git a/‎lib/Runtime/Language/JavascriptOperators.cpp
Copy file name to clipboard
+2-10Lines changed: 2 additions & 10 deletions b/‎lib/Runtime/Language/JavascriptOperators.cpp
Copy file name to clipboard
+2-10Lines changed: 2 additions & 10 deletions
diff --git a/‎lib/Runtime/Library/ArrayBuffer.cpp
Copy file name to clipboard
+2-2Lines changed: 2 additions & 2 deletions b/‎lib/Runtime/Library/ArrayBuffer.cpp
Copy file name to clipboard
+2-2Lines changed: 2 additions & 2 deletions
diff --git a/‎lib/Runtime/Library/ArrayBuffer.h
Copy file name to clipboard
+3-3Lines changed: 3 additions & 3 deletions b/‎lib/Runtime/Library/ArrayBuffer.h
Copy file name to clipboard
+3-3Lines changed: 3 additions & 3 deletions
@@ -64,6 +64,7 @@ var controllerObj = (function () {
             "#__proto__",
             "Array",
             "ArrayBuffer",
+            "Atomics",
             "Boolean",
             "CollectGarbage",
             "console",
@@ -105,6 +106,7 @@ var controllerObj = (function () {
             "Reflect",
             "RegExp",
             "Set",
+            "SharedArrayBuffer",
             "String",
             "Symbol",
             "SyntaxError",
 
@@ -587,11 +587,8 @@ PHASE(All)
 #define DEFAULT_CONFIG_ES7ValuesEntries        (true)
 #define DEFAULT_CONFIG_ESObjectGetOwnPropertyDescriptors (true)
 
-#ifdef COMPILE_DISABLE_ESSharedArrayBuffer
-#define DEFAULT_CONFIG_ESSharedArrayBuffer     (false)
-#else
-#define DEFAULT_CONFIG_ESSharedArrayBuffer     (false)
-#endif
+#define DEFAULT_CONFIG_ESSharedArrayBuffer     (true)
+
 #define DEFAULT_CONFIG_ES6Verbose              (false)
 #define DEFAULT_CONFIG_ES6All                  (false)
 // ES6 DEFAULT BEHAVIOR
 
@@ -227,6 +227,7 @@ struct AllocatorInfo
 template <typename TAllocator>
 struct ForceNonLeafAllocator
 {
+    static const bool FakeZeroLengthArray = true;
     typedef TAllocator AllocatorType;
 };
 
@@ -235,6 +236,7 @@ template <typename TAllocator>
 struct ForceLeafAllocator
 {
     typedef TAllocator AllocatorType;
+    static const bool FakeZeroLengthArray = true;
 };
 
 // Optional AllocatorDelete flags
@@ -382,16 +384,19 @@ void DestructArray(size_t count, T* obj)
 template <typename TAllocator, typename T>
 void DeleteArray(typename AllocatorInfo<TAllocator, T>::AllocatorType * allocator, size_t count, T * obj)
 {
-    if (count == 0)
+    if (count == 0 && AllocatorInfo<TAllocator, T>::AllocatorType::FakeZeroLengthArray)
     {
         return;
     }
 
-    DestructArray(count, obj);
+    if (count != 0)
+    {
+        DestructArray(count, obj);
 
-    // DeleteArray can only be called when an array is allocated successfully.
-    // So the add should never overflow
-    Assert(count * sizeof(T) / count == sizeof(T));
+        // DeleteArray can only be called when an array is allocated successfully.
+        // So the add should never overflow
+        Assert(count * sizeof(T) / count == sizeof(T));
+    }
 
     auto freeFunc = AllocatorInfo<TAllocator, T>::AllocatorFunc::GetFreeFunc();
     (allocator->*freeFunc)((void *)obj, sizeof(T) * count);
 
@@ -2318,9 +2318,21 @@ class TypeAllocatorFunc<Recycler, T *> : public _RecyclerAllocatorFunc<_Recycler
 #endif
 
 // Dummy class to choose the allocation function
-class RecyclerLeafAllocator;
-class RecyclerNonLeafAllocator;
-class RecyclerWriteBarrierAllocator;
+class RecyclerLeafAllocator
+{
+public:
+    static const bool FakeZeroLengthArray = true;
+};
+class RecyclerNonLeafAllocator
+{
+public:
+    static const bool FakeZeroLengthArray = true;
+};
+class RecyclerWriteBarrierAllocator
+{
+public:
+    static const bool FakeZeroLengthArray = true;
+};
 
 // Choose RecyclerLeafAllocator / RecyclerNonLeafAllocator based on "bool isLeaf"
 template <bool isLeaf>
 
@@ -5431,16 +5431,8 @@ namespace Js
 
     Var JavascriptOperators::NewVarFromDetachedState(DetachedStateBase* state, JavascriptLibrary *library)
     {
-        switch (state->GetTypeId())
-        {
-        case TypeIds_SharedArrayBuffer:
-            return Js::SharedArrayBuffer::NewFromSharedState(state, library);
-        case TypeIds_ArrayBuffer:
-            return Js::ArrayBuffer::NewFromDetachedState(state, library);
-        default:
-            AssertMsg(false, "We should explicitly have a case statement for each object which has detached state.");
-            return nullptr;
-        }
+        AssertOrFailFastMsg(state->GetTypeId() == TypeIds_ArrayBuffer, "We should only re-activate detached ArrayBuffer");
+        return Js::ArrayBuffer::NewFromDetachedState(state, library);
     }
 
     DynamicType *
 
@@ -736,7 +736,7 @@ namespace Js
         return IsValidAsmJsBufferLengthAlgo(length, forceCheck);
     }
 
-    bool JavascriptArrayBuffer::IsValidVirtualBufferLength(uint length)
+    bool JavascriptArrayBuffer::IsValidVirtualBufferLength(uint length) const
     {
 #if ENABLE_FAST_ARRAYBUFFER
         return !PHASE_OFF1(Js::TypedArrayVirtualPhase) && IsValidAsmJsBufferLengthAlgo(length, true);
@@ -995,7 +995,7 @@ namespace Js
         return result;
     }
 
-    bool WebAssemblyArrayBuffer::IsValidVirtualBufferLength(uint length)
+    bool WebAssemblyArrayBuffer::IsValidVirtualBufferLength(uint length) const
     {
 #if ENABLE_FAST_ARRAYBUFFER
         return true;
 
@@ -77,7 +77,7 @@ namespace Js
         virtual void AddParent(ArrayBufferParent* parent) { }
         virtual uint32 GetByteLength() const = 0;
         virtual BYTE* GetBuffer() const = 0;
-        virtual bool IsValidVirtualBufferLength(uint length) { return false; }
+        virtual bool IsValidVirtualBufferLength(uint length) const { return false; };
 
         static bool Is(Var value);
         static ArrayBufferBase* FromVar(Var value);
@@ -255,7 +255,7 @@ namespace Js
 
         static bool IsValidAsmJsBufferLengthAlgo(uint length, bool forceCheck);
         virtual bool IsValidAsmJsBufferLength(uint length, bool forceCheck = false) override;
-        virtual bool IsValidVirtualBufferLength(uint length) override;
+        virtual bool IsValidVirtualBufferLength(uint length) const override;
 
         virtual ArrayBuffer * TransferInternal(DECLSPEC_GUARD_OVERFLOW uint32 newBufferLength) override;
 
@@ -286,7 +286,7 @@ namespace Js
         DEFINE_MARSHAL_OBJECT_TO_SCRIPT_CONTEXT(WebAssemblyArrayBuffer);
     public:
         static WebAssemblyArrayBuffer* Create(byte* buffer, DECLSPEC_GUARD_OVERFLOW uint32 length, DynamicType * type);
-        virtual bool IsValidVirtualBufferLength(uint length) override;
+        virtual bool IsValidVirtualBufferLength(uint length) const override;
         virtual ArrayBuffer * TransferInternal(DECLSPEC_GUARD_OVERFLOW uint32 newBufferLength) override;
     };
Original file line number	Diff line number	Diff line change
`@@ -736,7 +736,7 @@ namespace Js`
`736`	`736`	`return IsValidAsmJsBufferLengthAlgo(length, forceCheck);`
`737`	`737`	`}`
`738`	`738`
`739`		`- bool JavascriptArrayBuffer::IsValidVirtualBufferLength(uint length)`
	`739`	`+ bool JavascriptArrayBuffer::IsValidVirtualBufferLength(uint length) const`
`740`	`740`	`{`
`741`	`741`	`#if ENABLE_FAST_ARRAYBUFFER`
`742`	`742`	`return !PHASE_OFF1(Js::TypedArrayVirtualPhase) && IsValidAsmJsBufferLengthAlgo(length, true);`
`@@ -995,7 +995,7 @@ namespace Js`
`995`	`995`	`return result;`
`996`	`996`	`}`
`997`	`997`
`998`		`- bool WebAssemblyArrayBuffer::IsValidVirtualBufferLength(uint length)`
	`998`	`+ bool WebAssemblyArrayBuffer::IsValidVirtualBufferLength(uint length) const`
`999`	`999`	`{`
`1000`	`1000`	`#if ENABLE_FAST_ARRAYBUFFER`
`1001`	`1001`	`return true;`