summaryrefslogtreecommitdiff
path: root/fxjs
diff options
context:
space:
mode:
authorTom Sepez <tsepez@chromium.org>2018-07-23 20:41:04 +0000
committerChromium commit bot <commit-bot@chromium.org>2018-07-23 20:41:04 +0000
commit4c358488d0a8daf3a625cae842efcf27a3bb62f5 (patch)
treeb830f9e75a27f4e3c6a381d01e0932edfd551177 /fxjs
parent2bfa78540c375916ec9973f0ae11271b098180bd (diff)
downloadpdfium-4c358488d0a8daf3a625cae842efcf27a3bb62f5.tar.xz
Remove argument from CFXJSE_Value::ToHostObject()
Like the other cases, it is always nullptr. Change-Id: I280f25899ffbe5e35f4ef3342aec7896edf3e1f2 Reviewed-on: https://pdfium-review.googlesource.com/38592 Reviewed-by: Lei Zhang <thestig@chromium.org> Commit-Queue: Tom Sepez <tsepez@chromium.org>
Diffstat (limited to 'fxjs')
-rw-r--r--fxjs/cfxjse_engine.cpp8
-rw-r--r--fxjs/cfxjse_formcalc_context.cpp5
-rw-r--r--fxjs/cfxjse_value.cpp7
-rw-r--r--fxjs/cfxjse_value.h2
4 files changed, 9 insertions, 13 deletions
diff --git a/fxjs/cfxjse_engine.cpp b/fxjs/cfxjse_engine.cpp
index dd907271eb..3efe335d67 100644
--- a/fxjs/cfxjse_engine.cpp
+++ b/fxjs/cfxjse_engine.cpp
@@ -65,8 +65,8 @@ namespace {
const char kFormCalcRuntime[] = "pfm_rt";
-CXFA_ThisProxy* ToThisProxy(CFXJSE_Value* pValue, CFXJSE_Class* pClass) {
- return static_cast<CXFA_ThisProxy*>(pValue->ToHostObject(pClass));
+CXFA_ThisProxy* ToThisProxy(CFXJSE_Value* pValue) {
+ return static_cast<CXFA_ThisProxy*>(pValue->ToHostObject());
}
} // namespace
@@ -84,7 +84,7 @@ CXFA_Object* CFXJSE_Engine::ToObject(
// static.
CXFA_Object* CFXJSE_Engine::ToObject(CFXJSE_Value* pValue) {
- CFXJSE_HostObject* pHostObj = pValue->ToHostObject(nullptr);
+ CFXJSE_HostObject* pHostObj = pValue->ToHostObject();
return pHostObj ? pHostObj->AsCXFAObject() : nullptr;
}
@@ -113,7 +113,7 @@ CFXJSE_Engine::CFXJSE_Engine(CXFA_Document* pDocument,
CFXJSE_Engine::~CFXJSE_Engine() {
for (const auto& pair : m_mapVariableToContext)
- delete ToThisProxy(pair.second->GetGlobalObject().get(), nullptr);
+ delete ToThisProxy(pair.second->GetGlobalObject().get());
}
bool CFXJSE_Engine::RunScript(CXFA_Script::Type eScriptType,
diff --git a/fxjs/cfxjse_formcalc_context.cpp b/fxjs/cfxjse_formcalc_context.cpp
index 6ca57bb649..5319466cfa 100644
--- a/fxjs/cfxjse_formcalc_context.cpp
+++ b/fxjs/cfxjse_formcalc_context.cpp
@@ -449,7 +449,7 @@ bool PatternStringType(const ByteStringView& szPattern, uint32_t& patternType) {
}
CFXJSE_FormCalcContext* ToFormCalcContext(CFXJSE_Value* pValue) {
- CFXJSE_HostObject* pHostObj = pValue->ToHostObject(nullptr);
+ CFXJSE_HostObject* pHostObj = pValue->ToHostObject();
return pHostObj ? pHostObj->AsFormCalcContext() : nullptr;
}
@@ -4944,8 +4944,7 @@ bool CFXJSE_FormCalcContext::fm_ref_equal(CFXJSE_Value* pThis,
if (firstJSObject->IsNull() || secondJSObject->IsNull())
return false;
- return (firstJSObject->ToHostObject(nullptr) ==
- secondJSObject->ToHostObject(nullptr));
+ return firstJSObject->ToHostObject() == secondJSObject->ToHostObject();
}
// static
diff --git a/fxjs/cfxjse_value.cpp b/fxjs/cfxjse_value.cpp
index 5d7f885746..915fefc119 100644
--- a/fxjs/cfxjse_value.cpp
+++ b/fxjs/cfxjse_value.cpp
@@ -68,18 +68,15 @@ CFXJSE_Value::CFXJSE_Value(v8::Isolate* pIsolate) : m_pIsolate(pIsolate) {}
CFXJSE_Value::~CFXJSE_Value() {}
-CFXJSE_HostObject* CFXJSE_Value::ToHostObject(CFXJSE_Class* lpClass) const {
- ASSERT(!m_hValue.IsEmpty());
-
+CFXJSE_HostObject* CFXJSE_Value::ToHostObject() const {
CFXJSE_ScopeUtil_IsolateHandleRootContext scope(GetIsolate());
v8::Local<v8::Value> pValue =
v8::Local<v8::Value>::New(GetIsolate(), m_hValue);
ASSERT(!pValue.IsEmpty());
-
if (!pValue->IsObject())
return nullptr;
- return FXJSE_RetrieveObjectBinding(pValue.As<v8::Object>(), lpClass);
+ return FXJSE_RetrieveObjectBinding(pValue.As<v8::Object>(), nullptr);
}
void CFXJSE_Value::SetObject(CFXJSE_HostObject* lpObject,
diff --git a/fxjs/cfxjse_value.h b/fxjs/cfxjse_value.h
index 5fbac4d58a..dd01843990 100644
--- a/fxjs/cfxjse_value.h
+++ b/fxjs/cfxjse_value.h
@@ -43,7 +43,7 @@ class CFXJSE_Value {
WideString ToWideString() const {
return WideString::FromUTF8(ToString().AsStringView());
}
- CFXJSE_HostObject* ToHostObject(CFXJSE_Class* lpClass) const;
+ CFXJSE_HostObject* ToHostObject() const;
void SetUndefined();
void SetNull();