Tidy FXJS_V8, backfill tests.

fxjs/fxjs_v8.cpp

 // Copyright 2014 PDFium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // Original code copyright 2014 Foxit Software Inc. http://www.foxitsoftware.com
 
 #include "fxjs/fxjs_v8.h"
 
 #include <vector>
 
@@ skipping 578 matching lines @@
   if (pObj.IsEmpty())
     return std::vector<CFX_WideString>();
 
   v8::Local<v8::Array> val;
   v8::Local<v8::Context> context = m_isolate->GetCurrentContext();
   if (!pObj->GetPropertyNames(context).ToLocal(&val))
     return std::vector<CFX_WideString>();
 
   std::vector<CFX_WideString> result;
   for (uint32_t i = 0; i < val->Length(); ++i) {
-    result.push_back(ToString(val->Get(context, i).ToLocalChecked()));
+    result.push_back(ToWideString(val->Get(context, i).ToLocalChecked()));
   }
 
   return result;
 }
 
-void CFXJS_Engine::PutObjectString(v8::Local<v8::Object> pObj,
-                                   const CFX_WideString& wsPropertyName,
-                                   const CFX_WideString& wsValue) {
-  if (pObj.IsEmpty())
-    return;
-  pObj->Set(m_isolate->GetCurrentContext(), WSToJSString(wsPropertyName),
-            WSToJSString(wsValue))
-      .FromJust();
-}
-
-void CFXJS_Engine::PutObjectNumber(v8::Local<v8::Object> pObj,
-                                   const CFX_WideString& wsPropertyName,
-                                   int nValue) {
-  if (pObj.IsEmpty())
-    return;
-  pObj->Set(m_isolate->GetCurrentContext(), WSToJSString(wsPropertyName),
-            v8::Int32::New(m_isolate, nValue))
-      .FromJust();
-}
-
-void CFXJS_Engine::PutObjectNumber(v8::Local<v8::Object> pObj,
-                                   const CFX_WideString& wsPropertyName,
-                                   float fValue) {
-  if (pObj.IsEmpty())
-    return;
-  pObj->Set(m_isolate->GetCurrentContext(), WSToJSString(wsPropertyName),
-            v8::Number::New(m_isolate, (double)fValue))
-      .FromJust();
-}
-
-void CFXJS_Engine::PutObjectNumber(v8::Local<v8::Object> pObj,
-                                   const CFX_WideString& wsPropertyName,
-                                   double dValue) {
-  if (pObj.IsEmpty())
-    return;
-  pObj->Set(m_isolate->GetCurrentContext(), WSToJSString(wsPropertyName),
-            v8::Number::New(m_isolate, (double)dValue))
-      .FromJust();
-}
-
-void CFXJS_Engine::PutObjectBoolean(v8::Local<v8::Object> pObj,
-                                    const CFX_WideString& wsPropertyName,
-                                    bool bValue) {
-  if (pObj.IsEmpty())
-    return;
-  pObj->Set(m_isolate->GetCurrentContext(), WSToJSString(wsPropertyName),
-            v8::Boolean::New(m_isolate, bValue))
-      .FromJust();
-}
-
-void CFXJS_Engine::PutObjectObject(v8::Local<v8::Object> pObj,
-                                   const CFX_WideString& wsPropertyName,
-                                   v8::Local<v8::Object> pPut) {
+void CFXJS_Engine::PutObjectProperty(v8::Local<v8::Object> pObj,
+                                     const CFX_WideString& wsPropertyName,
+                                     v8::Local<v8::Value> pPut) {
   if (pObj.IsEmpty())
     return;
   pObj->Set(m_isolate->GetCurrentContext(), WSToJSString(wsPropertyName), pPut)
       .FromJust();
 }
 
-void CFXJS_Engine::PutObjectNull(v8::Local<v8::Object> pObj,
-                                 const CFX_WideString& wsPropertyName) {
-  if (pObj.IsEmpty())
-    return;
-  pObj->Set(m_isolate->GetCurrentContext(), WSToJSString(wsPropertyName),
-            v8::Local<v8::Object>())
-      .FromJust();
-}
 
 v8::Local<v8::Array> CFXJS_Engine::NewArray() {
   return v8::Array::New(m_isolate);
 }
 
 unsigned CFXJS_Engine::PutArrayElement(v8::Local<v8::Array> pArray,
                                        unsigned index,
                                        v8::Local<v8::Value> pValue) {
   if (pArray.IsEmpty())
     return 0;
@@ skipping 35 matching lines @@
 }
 
 v8::Local<v8::Value> CFXJS_Engine::NewNumber(float number) {
   return v8::Number::New(m_isolate, (float)number);
 }
 
 v8::Local<v8::Value> CFXJS_Engine::NewBoolean(bool b) {
   return v8::Boolean::New(m_isolate, b);
 }
 
-v8::Local<v8::Value> CFXJS_Engine::NewString(const wchar_t* str) {
-  return WSToJSString(str);
+v8::Local<v8::Value> CFXJS_Engine::NewString(const CFX_WideString& str) {
+  return WSToJSString(str.c_str());
 }
 
 v8::Local<v8::Value> CFXJS_Engine::NewNull() {
   return v8::Local<v8::Value>();
 }
 
 v8::Local<v8::Date> CFXJS_Engine::NewDate(double d) {
   return v8::Date::New(m_isolate->GetCurrentContext(), d)
       .ToLocalChecked()
       .As<v8::Date>();
 }
 
 int CFXJS_Engine::ToInt32(v8::Local<v8::Value> pValue) {
   if (pValue.IsEmpty())
     return 0;
   v8::Local<v8::Context> context = m_isolate->GetCurrentContext();
   return pValue->ToInt32(context).ToLocalChecked()->Value();
 }
 
 bool CFXJS_Engine::ToBoolean(v8::Local<v8::Value> pValue) {
   if (pValue.IsEmpty())
     return false;
   v8::Local<v8::Context> context = m_isolate->GetCurrentContext();
   return pValue->ToBoolean(context).ToLocalChecked()->Value();
 }
 
-double CFXJS_Engine::ToNumber(v8::Local<v8::Value> pValue) {
+double CFXJS_Engine::ToDouble(v8::Local<v8::Value> pValue) {
   if (pValue.IsEmpty())
     return 0.0;
   v8::Local<v8::Context> context = m_isolate->GetCurrentContext();
   return pValue->ToNumber(context).ToLocalChecked()->Value();
 }
 
+CFX_WideString CFXJS_Engine::ToWideString(v8::Local<v8::Value> pValue) {
+  if (pValue.IsEmpty())
+    return CFX_WideString();
+  v8::Local<v8::Context> context = m_isolate->GetCurrentContext();
+  v8::String::Utf8Value s(pValue->ToString(context).ToLocalChecked());
+  return CFX_WideString::FromUTF8(CFX_ByteStringC(*s, s.length()));
+}
+
 v8::Local<v8::Object> CFXJS_Engine::ToObject(v8::Local<v8::Value> pValue) {
-  if (pValue.IsEmpty())
+  if (pValue.IsEmpty() || !pValue->IsObject())
     return v8::Local<v8::Object>();
   v8::Local<v8::Context> context = m_isolate->GetCurrentContext();
   return pValue->ToObject(context).ToLocalChecked();
 }
 
-CFX_WideString CFXJS_Engine::ToString(v8::Local<v8::Value> pValue) {
-  if (pValue.IsEmpty())
-    return L"";
-  v8::Local<v8::Context> context = m_isolate->GetCurrentContext();
-  v8::String::Utf8Value s(pValue->ToString(context).ToLocalChecked());
-  return CFX_WideString::FromUTF8(CFX_ByteStringC(*s, s.length()));
-}
-
 v8::Local<v8::Array> CFXJS_Engine::ToArray(v8::Local<v8::Value> pValue) {
-  if (pValue.IsEmpty())
+  if (pValue.IsEmpty() || !pValue->IsArray())
     return v8::Local<v8::Array>();
   v8::Local<v8::Context> context = m_isolate->GetCurrentContext();
   return v8::Local<v8::Array>::Cast(pValue->ToObject(context).ToLocalChecked());
 }
 
 void CFXJS_Engine::SetConstArray(const CFX_WideString& name,
                                  v8::Local<v8::Array> array) {
   m_ConstArrays[name] = v8::Global<v8::Array>(GetIsolate(), array);
 }
 
 v8::Local<v8::Array> CFXJS_Engine::GetConstArray(const CFX_WideString& name) {
   return v8::Local<v8::Array>::New(GetIsolate(), m_ConstArrays[name]);
 }