diff options
| author | Tom Sepez <tsepez@chromium.org> | 2018-08-01 19:13:06 +0000 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2018-08-01 19:13:06 +0000 |
| commit | 2563fc3895f2a6a1faf74b46387b99c61be7370a (patch) | |
| tree | bc809136c0daa416387c9a2044474969171796ca /fxjs | |
| parent | 6214ce98801c1f7a226781701ed290079b2a4031 (diff) | |
| download | pdfium-2563fc3895f2a6a1faf74b46387b99c61be7370a.tar.xz | |
Make FPDF_FormHandle be represented as an incomplete type.chromium/3510
Make consistent with other public API types.
Introduce CPDFSDKFormFillEnvironmentFromFPDFFormHandle() and
FPDFFormHandleFromCPDFSDKFormFillEnvironment() helper functions.
Use these to kill off some casts in the process.
Change-Id: I6230ecdb4cecd03076f5e24c8cc49c45ad694da7
Reviewed-on: https://pdfium-review.googlesource.com/39250
Reviewed-by: Lei Zhang <thestig@chromium.org>
Commit-Queue: Tom Sepez <tsepez@chromium.org>
Diffstat (limited to 'fxjs')
| -rw-r--r-- | fxjs/cjs_publicmethods_embeddertest.cpp | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/fxjs/cjs_publicmethods_embeddertest.cpp b/fxjs/cjs_publicmethods_embeddertest.cpp index 38604555ae..623ce6801e 100644 --- a/fxjs/cjs_publicmethods_embeddertest.cpp +++ b/fxjs/cjs_publicmethods_embeddertest.cpp @@ -180,7 +180,8 @@ TEST_F(CJS_PublicMethodsEmbedderTest, AFSimple_CalculateSum) { auto* page = LoadPage(0); ASSERT_TRUE(page); - CJS_Runtime runtime(static_cast<CPDFSDK_FormFillEnvironment*>(form_handle())); + CJS_Runtime runtime( + CPDFSDKFormFillEnvironmentFromFPDFFormHandle(form_handle())); runtime.NewEventContext(); WideString result; @@ -214,7 +215,8 @@ TEST_F(CJS_PublicMethodsEmbedderTest, AFNumber_Keystroke) { auto* page = LoadPage(0); ASSERT_TRUE(page); - CJS_Runtime runtime(static_cast<CPDFSDK_FormFillEnvironment*>(form_handle())); + CJS_Runtime runtime( + CPDFSDKFormFillEnvironmentFromFPDFFormHandle(form_handle())); runtime.NewEventContext(); auto* handler = runtime.GetCurrentEventContext()->GetEventHandler(); |