From fb2a824f87f7096382681fd258906823ef93ebec Mon Sep 17 00:00:00 2001 From: tsepez Date: Wed, 1 Jun 2016 16:10:41 -0700 Subject: 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 --- xfa/fxfa/parser/xfa_script_imp.cpp | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) (limited to 'xfa/fxfa/parser/xfa_script_imp.cpp') 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; } -- cgit v1.2.3