From 998fee395fc8a543968c7db3db9e3cf81dee57fc Mon Sep 17 00:00:00 2001 From: Dan Sinclair Date: Mon, 5 Feb 2018 21:43:19 +0000 Subject: Remove the CJS_EmbedObj template param from JSConstructor. Each of the CJS_Objects can create their CJS_EmbedObj's internally and we don't need to do it though the JSConstructor. This also removes the need for the SetEmbedObj method in CJS_Object. Change-Id: Ib0535ad922b370634fd1e622a04860a96c4f2825 Reviewed-on: https://pdfium-review.googlesource.com/25370 Reviewed-by: Tom Sepez Commit-Queue: dsinclair --- fxjs/cjs_object.h | 4 ---- 1 file changed, 4 deletions(-) (limited to 'fxjs/cjs_object.h') diff --git a/fxjs/cjs_object.h b/fxjs/cjs_object.h index f4b0d297f4..67389f59e6 100644 --- a/fxjs/cjs_object.h +++ b/fxjs/cjs_object.h @@ -57,10 +57,6 @@ class CJS_Object { v8::Local ToV8Object() { return m_pV8Object.Get(m_pIsolate); } - // Takes ownership of |pObj|. - void SetEmbedObject(std::unique_ptr pObj) { - m_pEmbedObj = std::move(pObj); - } CJS_EmbedObj* GetEmbedObject() const { return m_pEmbedObj.get(); } v8::Isolate* GetIsolate() const { return m_pIsolate; } -- cgit v1.2.3