chakrashim: handle Symbol as property name

Also remove some unused code, cache a few propertyIds, refactor a few code
for better perf.

Reviewed-by: @kunalspathak

deps/chakrashim/include/v8.h

@@ -836,7 +836,6 @@ class EXPORT Object : public Value {
   Local<Value> GetPrototype();
   bool SetPrototype(Handle<Value> prototype);
   Maybe<bool> SetPrototype(Local<Context> context, Local<Value> prototype);
-  Local<Value> GetConstructor();
   Local<Array> GetPropertyNames();
   Local<Array> GetOwnPropertyNames();
   bool HasOwnProperty(Handle<String> key);

deps/chakrashim/src/jsrtcachedpropertyidref.inc

@@ -28,6 +28,7 @@
 
 DEF(apply)
 DEF(construct)
+DEF(constructor)
 DEF(defineProperty)
 DEF(deleteProperty)
 DEF(enumerate)
@@ -36,6 +37,7 @@ DEF(getOwnPropertyDescriptor)
 DEF(getPrototypeOf)
 DEF(has)
 DEF(isExtensible)
+DEF(name)
 DEF(ownKeys)
 DEF(preventExtensions)
 DEF(set)
@@ -72,6 +74,8 @@ DEF(byteOffset)
 DEF(byteLength)
 
 DEFSYMBOL(self)
+DEFSYMBOL(__external__)
+DEFSYMBOL(__hiddenvalues__)
 DEFSYMBOL(__isexternal__)
 
 #undef DEF

deps/chakrashim/src/jsrtproxyutils.cc

@@ -135,9 +135,9 @@ JsErrorCode TryParseUInt32(
   size_t strLength;
 
   error = JsStringToPointer(strRef, &strPtr, &strLength);
-
   if (error != JsNoError) {
-    return error;
+    // If strRef is not a string, just return not Uint32
+    return error == JsErrorInvalidArgument ? JsNoError : error;
   }
 
   // empty string