Merge to M50: Re-land "Only place primitive objects on the V8 global template."

fpdfsdk/src/jsapi/fxjs_v8.cpp

Index: fpdfsdk/src/jsapi/fxjs_v8.cpp
diff --git a/fpdfsdk/src/jsapi/fxjs_v8.cpp b/fpdfsdk/src/jsapi/fxjs_v8.cpp
index 4e139ca37b01166b7fbb3bc70cdbdbb5540aa307..e0e26deac81accdde7345054c80da69998c6db94 100644
--- a/fpdfsdk/src/jsapi/fxjs_v8.cpp
+++ b/fpdfsdk/src/jsapi/fxjs_v8.cpp
@@ -292,14 +292,15 @@ void FXJS_DefineGlobalMethod(v8::Isolate* pIsolate,
 
 void FXJS_DefineGlobalConst(v8::Isolate* pIsolate,
                             const wchar_t* sConstName,
-                            v8::Local<v8::Value> pDefault) {
+                            v8::FunctionCallback pConstGetter) {
   v8::Isolate::Scope isolate_scope(pIsolate);
   v8::HandleScope handle_scope(pIsolate);
   CFX_ByteString bsConst = CFX_WideString(sConstName).UTF8Encode();
-  GetGlobalObjectTemplate(pIsolate)->Set(
-      v8::String::NewFromUtf8(pIsolate, bsConst.c_str(),
-                              v8::NewStringType::kNormal).ToLocalChecked(),
-      pDefault, v8::ReadOnly);
+  GetGlobalObjectTemplate(pIsolate)
+      ->SetAccessorProperty(v8::String::NewFromUtf8(pIsolate, bsConst.c_str(),
+                                                    v8::NewStringType::kNormal)
+                                .ToLocalChecked(),
+                            v8::FunctionTemplate::New(pIsolate, pConstGetter));
 }
 
 void FXJS_InitializeRuntime(