diff options
| author | Dan Sinclair <dsinclair@chromium.org> | 2017-11-06 21:35:11 +0000 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2017-11-06 21:35:11 +0000 |
| commit | 67687896d2959494d2e689ffbbc9c86098e54280 (patch) | |
| tree | df8e1d598a973ad7f3ccbf3e7bec0e4ca68085b2 /fxjs/cfxjse_engine.h | |
| parent | 98c6b5e57932cc212fcd08056e43e1c11c18d926 (diff) | |
| download | pdfium-67687896d2959494d2e689ffbbc9c86098e54280.tar.xz | |
Consolidate CFXJSE_Engine construction
This CL removes the Initialize method from CFXJSE_Engine and moves the
code to the constuctor. The DefineJsContext and DefineJSClass methods
are also removed and done directly in the constructor.
Change-Id: I3f849509a17a6bda22b520c640f23ee110e939af
Reviewed-on: https://pdfium-review.googlesource.com/17857
Commit-Queue: dsinclair <dsinclair@chromium.org>
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Diffstat (limited to 'fxjs/cfxjse_engine.h')
| -rw-r--r-- | fxjs/cfxjse_engine.h | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/fxjs/cfxjse_engine.h b/fxjs/cfxjse_engine.h index f901850203..bfc2cc1fd9 100644 --- a/fxjs/cfxjse_engine.h +++ b/fxjs/cfxjse_engine.h @@ -23,10 +23,9 @@ class CFXJSE_ResolveProcessor; class CFXJSE_Engine { public: - explicit CFXJSE_Engine(CXFA_Document* pDocument); + explicit CFXJSE_Engine(CXFA_Document* pDocument, v8::Isolate* pIsolate); ~CFXJSE_Engine(); - void Initialize(v8::Isolate* pIsolate); void SetEventParam(CXFA_EventParam param) { m_eventParam = param; } CXFA_EventParam* GetEventParam() { return &m_eventParam; } bool RunScript(XFA_SCRIPTLANGTYPE eScriptType, @@ -97,10 +96,8 @@ class CFXJSE_Engine { static CXFA_Object* ToObject(CFXJSE_Value* pValue, CFXJSE_Class* pClass); private: - void DefineJsContext(); CFXJSE_Context* CreateVariablesContext(CXFA_Node* pScriptNode, CXFA_Node* pSubform); - void DefineJsClass(); void RemoveBuiltInObjs(CFXJSE_Context* pContext) const; UnownedPtr<CXFA_Document> const m_pDocument; |