From d780735def22c4c2ec3a0cc89856c86486c25494 Mon Sep 17 00:00:00 2001 From: tsepez Date: Mon, 25 Jul 2016 07:26:47 -0700 Subject: Remove unused CJS_Value::Attach() variant Review-Url: https://codereview.chromium.org/2174513002 --- fpdfsdk/javascript/JS_Value.cpp | 5 ----- fpdfsdk/javascript/JS_Value.h | 5 ++--- 2 files changed, 2 insertions(+), 8 deletions(-) diff --git a/fpdfsdk/javascript/JS_Value.cpp b/fpdfsdk/javascript/JS_Value.cpp index 5c1ba2a432..c13d1d405e 100644 --- a/fpdfsdk/javascript/JS_Value.cpp +++ b/fpdfsdk/javascript/JS_Value.cpp @@ -75,11 +75,6 @@ void CJS_Value::Attach(v8::Local pValue) { m_pValue = pValue; } -void CJS_Value::Attach(CJS_Value* pValue) { - if (pValue) - Attach(pValue->ToV8Value()); -} - void CJS_Value::Detach() { m_pValue = v8::Local(); } diff --git a/fpdfsdk/javascript/JS_Value.h b/fpdfsdk/javascript/JS_Value.h index 8e791253c6..2abc42521d 100644 --- a/fpdfsdk/javascript/JS_Value.h +++ b/fpdfsdk/javascript/JS_Value.h @@ -31,7 +31,7 @@ class CJS_Value { VT_undefined }; - CJS_Value(CJS_Runtime* pRuntime); + explicit CJS_Value(CJS_Runtime* pRuntime); CJS_Value(CJS_Runtime* pRuntime, v8::Local pValue); CJS_Value(CJS_Runtime* pRuntime, const int& iValue); CJS_Value(CJS_Runtime* pRuntime, const double& dValue); @@ -41,13 +41,12 @@ class CJS_Value { CJS_Value(CJS_Runtime* pRuntime, const FX_CHAR* pStr); CJS_Value(CJS_Runtime* pRuntime, const FX_WCHAR* pWstr); CJS_Value(CJS_Runtime* pRuntime, CJS_Array& array); + CJS_Value(const CJS_Value& other); ~CJS_Value(); - CJS_Value(const CJS_Value& other); void SetNull(); void Attach(v8::Local pValue); - void Attach(CJS_Value* pValue); void Detach(); static Type GetValueType(v8::Local value); -- cgit v1.2.3