From bef4d3e97bba3274add9738d6b199f7760c88f53 Mon Sep 17 00:00:00 2001 From: Dan Sinclair Date: Thu, 26 Oct 2017 16:49:38 -0400 Subject: Remove unneeded DefineJSObject param The second parameter to DefineJSObject is always set the same for each object type. Remove and put the value directly into the DefineObj calls. Change-Id: If939c57fe6ea3b985ab2337eb30f472c668cdc42 Reviewed-on: https://pdfium-review.googlesource.com/16911 Commit-Queue: dsinclair Reviewed-by: Tom Sepez --- fpdfsdk/javascript/cjs_runtime.cpp | 42 +++++++++++++++++++------------------- 1 file changed, 21 insertions(+), 21 deletions(-) (limited to 'fpdfsdk/javascript/cjs_runtime.cpp') diff --git a/fpdfsdk/javascript/cjs_runtime.cpp b/fpdfsdk/javascript/cjs_runtime.cpp index 1b566b121c..57dfffe716 100644 --- a/fpdfsdk/javascript/cjs_runtime.cpp +++ b/fpdfsdk/javascript/cjs_runtime.cpp @@ -116,34 +116,34 @@ void CJS_Runtime::DefineJSObjects() { // The call order determines the "ObjDefID" assigned to each class. // ObjDefIDs 0 - 2 - CJS_Border::DefineJSObjects(this, FXJSOBJTYPE_STATIC); - CJS_Display::DefineJSObjects(this, FXJSOBJTYPE_STATIC); - CJS_Font::DefineJSObjects(this, FXJSOBJTYPE_STATIC); + CJS_Border::DefineJSObjects(this); + CJS_Display::DefineJSObjects(this); + CJS_Font::DefineJSObjects(this); // ObjDefIDs 3 - 5 - CJS_Highlight::DefineJSObjects(this, FXJSOBJTYPE_STATIC); - CJS_Position::DefineJSObjects(this, FXJSOBJTYPE_STATIC); - CJS_ScaleHow::DefineJSObjects(this, FXJSOBJTYPE_STATIC); + CJS_Highlight::DefineJSObjects(this); + CJS_Position::DefineJSObjects(this); + CJS_ScaleHow::DefineJSObjects(this); // ObjDefIDs 6 - 8 - CJS_ScaleWhen::DefineJSObjects(this, FXJSOBJTYPE_STATIC); - CJS_Style::DefineJSObjects(this, FXJSOBJTYPE_STATIC); - CJS_Zoomtype::DefineJSObjects(this, FXJSOBJTYPE_STATIC); + CJS_ScaleWhen::DefineJSObjects(this); + CJS_Style::DefineJSObjects(this); + CJS_Zoomtype::DefineJSObjects(this); // ObjDefIDs 9 - 11 - CJS_App::DefineJSObjects(this, FXJSOBJTYPE_STATIC); - CJS_Color::DefineJSObjects(this, FXJSOBJTYPE_STATIC); - CJS_Console::DefineJSObjects(this, FXJSOBJTYPE_STATIC); + CJS_App::DefineJSObjects(this); + CJS_Color::DefineJSObjects(this); + CJS_Console::DefineJSObjects(this); // ObjDefIDs 12 - 14 - CJS_Document::DefineJSObjects(this, FXJSOBJTYPE_GLOBAL); - CJS_Event::DefineJSObjects(this, FXJSOBJTYPE_STATIC); - CJS_Field::DefineJSObjects(this, FXJSOBJTYPE_DYNAMIC); + CJS_Document::DefineJSObjects(this); + CJS_Event::DefineJSObjects(this); + CJS_Field::DefineJSObjects(this); // ObjDefIDs 15 - 17 - CJS_Global::DefineJSObjects(this, FXJSOBJTYPE_STATIC); - CJS_Icon::DefineJSObjects(this, FXJSOBJTYPE_DYNAMIC); - CJS_Util::DefineJSObjects(this, FXJSOBJTYPE_STATIC); + CJS_Global::DefineJSObjects(this); + CJS_Icon::DefineJSObjects(this); + CJS_Util::DefineJSObjects(this); // ObjDefIDs 18 - 20 (these can't fail, return void). CJS_PublicMethods::DefineJSObjects(this); @@ -151,9 +151,9 @@ void CJS_Runtime::DefineJSObjects() { CJS_GlobalArrays::DefineJSObjects(this); // ObjDefIDs 21 - 23. - CJS_TimerObj::DefineJSObjects(this, FXJSOBJTYPE_DYNAMIC); - CJS_PrintParamsObj::DefineJSObjects(this, FXJSOBJTYPE_DYNAMIC); - CJS_Annot::DefineJSObjects(this, FXJSOBJTYPE_DYNAMIC); + CJS_TimerObj::DefineJSObjects(this); + CJS_PrintParamsObj::DefineJSObjects(this); + CJS_Annot::DefineJSObjects(this); } IJS_EventContext* CJS_Runtime::NewEventContext() { -- cgit v1.2.3