diff options
| author | Dan Sinclair <dsinclair@chromium.org> | 2017-12-12 01:53:28 +0000 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2017-12-12 01:53:28 +0000 |
| commit | 909fa2daa49e5439c595e3d17a76f0e05d7934b0 (patch) | |
| tree | 7dac1ee220b8e0968ff42cedf53775b1ca5a62ac /fxjs/cjs_app.cpp | |
| parent | 5e99d62f4d92531833daeb062073d805bf363df3 (diff) | |
| download | pdfium-909fa2daa49e5439c595e3d17a76f0e05d7934b0.tar.xz | |
[js] Convert to using size instead of sentinels
This CL changes the DefineMethod, DefineProps and DefineConsts methods
to pass a size instead of depending on a sentinel value in the
definition arrays.
Change-Id: Ie054544124290c0833a8b21af175a203ca99591a
Reviewed-on: https://pdfium-review.googlesource.com/20551
Commit-Queue: dsinclair <dsinclair@chromium.org>
Reviewed-by: Lei Zhang <thestig@chromium.org>
Diffstat (limited to 'fxjs/cjs_app.cpp')
| -rw-r--r-- | fxjs/cjs_app.cpp | 10 |
1 files changed, 4 insertions, 6 deletions
diff --git a/fxjs/cjs_app.cpp b/fxjs/cjs_app.cpp index e89fb2a2c7..32cac22153 100644 --- a/fxjs/cjs_app.cpp +++ b/fxjs/cjs_app.cpp @@ -48,8 +48,7 @@ const JSPropertySpec CJS_App::PropertySpecs[] = { {"viewerType", get_viewer_type_static, set_viewer_type_static}, {"viewerVariation", get_viewer_variation_static, set_viewer_variation_static}, - {"viewerVersion", get_viewer_version_static, set_viewer_version_static}, - {0, 0, 0}}; + {"viewerVersion", get_viewer_version_static, set_viewer_version_static}}; const JSMethodSpec CJS_App::MethodSpecs[] = { {"alert", alert_static}, @@ -72,8 +71,7 @@ const JSMethodSpec CJS_App::MethodSpecs[] = { {"popUpMenu", popUpMenu_static}, {"response", response_static}, {"setInterval", setInterval_static}, - {"setTimeOut", setTimeOut_static}, - {0, 0}}; + {"setTimeOut", setTimeOut_static}}; int CJS_App::ObjDefnID = -1; @@ -82,8 +80,8 @@ void CJS_App::DefineJSObjects(CFXJS_Engine* pEngine) { ObjDefnID = pEngine->DefineObj("app", FXJSOBJTYPE_STATIC, JSConstructor<CJS_App, app>, JSDestructor<CJS_App>); - DefineProps(pEngine, ObjDefnID, PropertySpecs); - DefineMethods(pEngine, ObjDefnID, MethodSpecs); + DefineProps(pEngine, ObjDefnID, PropertySpecs, FX_ArraySize(PropertySpecs)); + DefineMethods(pEngine, ObjDefnID, MethodSpecs, FX_ArraySize(MethodSpecs)); } app::app(CJS_Object* pJSObject) |