actually enable caching for CreateKeys enumerator

Author: MikeHolman

lib/Runtime/Library/JavascriptObject.cpp

@@ -1209,7 +1209,7 @@ JavascriptArray* JavascriptObject::CreateKeysHelper(RecyclableObject* object, Sc
     AssertMsg(includeStringProperties || includeSymbolProperties, "Should either get string or symbol properties.");
 
     JavascriptStaticEnumerator enumerator;
-    EnumeratorFlags flags = EnumeratorFlags::UseCache;
+    EnumeratorFlags flags = EnumeratorFlags::SnapShotSemantics | EnumeratorFlags::UseCache;
     JavascriptArray* newArr = scriptContext->GetLibrary()->CreateArray(0);
     if (includeNonEnumerable)
     {

lib/Runtime/Types/DynamicObjectPropertyEnumerator.cpp

@@ -73,7 +73,7 @@ namespace Js
             Assert(data != nullptr);
             Assert(data->scriptContext == this->scriptContext); // The cache data script context should be the same as request context
 
-            if (data->enumNonEnumerable == GetEnumNonEnumerable())
+            if (data->enumNonEnumerable == GetEnumNonEnumerable() && data->enumSymbols == GetEnumSymbols())
             {
                 Initialize(type, data, data->propertyCount);
                 return true;