toLocaleString now supports reading object property

Author: Dachande663

lib/Runtime/Language/JavascriptConversion.cpp

@@ -701,8 +701,16 @@ using namespace Js;
         case TypeIds_Number:
             return JavascriptNumber::ToLocaleString(JavascriptNumber::GetValue(aValue), scriptContext);
 
-        case TypeIds_String:
-            return UnsafeVarTo<JavascriptString>(aValue);
+        case TypeIds_String: {
+            RecyclableObject* object = VarTo<RecyclableObject>(aValue);
+            Var value = JavascriptOperators::GetProperty(object, PropertyIds::toLocaleString, scriptContext, NULL);
+            RecyclableObject* toLocaleStringFunction = VarTo<RecyclableObject>(value);
+            Var aResult = scriptContext->GetThreadContext()->ExecuteImplicitCall(toLocaleStringFunction, Js::ImplicitCall_ToPrimitive, [=]()->Js::Var
+            {
+                return CALL_FUNCTION(scriptContext->GetThreadContext(), toLocaleStringFunction, CallInfo(1), aValue);
+            });
+            return UnsafeVarTo<JavascriptString>(aResult);
+        }
 
         case TypeIds_VariantDate:
             // Legacy behavior was to create an empty object and call toLocaleString on it, which would result in this value