diff options
| author | tsepez <tsepez@chromium.org> | 2016-06-01 16:10:41 -0700 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2016-06-01 16:10:41 -0700 |
| commit | fb2a824f87f7096382681fd258906823ef93ebec (patch) | |
| tree | 1754bc611306c55ffe02e971db11349de530f491 /xfa/fxfa/parser | |
| parent | 44f8faf9852a37df1edda34f99e1de4c9d222921 (diff) | |
| download | pdfium-fb2a824f87f7096382681fd258906823ef93ebec.tar.xz | |
Don't use array for only one compatible mode script
Also, don't invent an enum that has one possible value that is
always set.
Review-Url: https://codereview.chromium.org/2028343002
Diffstat (limited to 'xfa/fxfa/parser')
| -rw-r--r-- | xfa/fxfa/parser/xfa_script_imp.cpp | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/xfa/fxfa/parser/xfa_script_imp.cpp b/xfa/fxfa/parser/xfa_script_imp.cpp index 8f579b5f26..357566ed9c 100644 --- a/xfa/fxfa/parser/xfa_script_imp.cpp +++ b/xfa/fxfa/parser/xfa_script_imp.cpp @@ -424,8 +424,7 @@ void CXFA_ScriptContext::DefineJsContext() { m_pJsContext = FXJSE_Context_Create(m_pIsolate, &GlobalClassDescriptor, m_pDocument->GetRoot()); RemoveBuiltInObjs(m_pJsContext); - FXJSE_Context_EnableCompatibleMode( - m_pJsContext, FXJSE_COMPATIBLEMODEFLAG_CONSTRUCTOREXTRAMETHODS); + FXJSE_Context_EnableCompatibleMode(m_pJsContext); } CFXJSE_Context* CXFA_ScriptContext::CreateVariablesContext( CXFA_Node* pScriptNode, @@ -437,8 +436,7 @@ CFXJSE_Context* CXFA_ScriptContext::CreateVariablesContext( FXJSE_Context_Create(m_pIsolate, &VariablesClassDescriptor, new CXFA_ThisProxy(pSubform, pScriptNode)); RemoveBuiltInObjs(pVariablesContext); - FXJSE_Context_EnableCompatibleMode( - pVariablesContext, FXJSE_COMPATIBLEMODEFLAG_CONSTRUCTOREXTRAMETHODS); + FXJSE_Context_EnableCompatibleMode(pVariablesContext); m_mapVariableToContext.SetAt(pScriptNode, pVariablesContext); return pVariablesContext; } |