XFA: Restore early return if no FPDF_Page in FORM_DoPageAAction()

XFA: Restore early return if no FPDF_Page in FORM_DoPageAAction()

Present in original XFA code drop.

BUG=pdfium:259
R=thestig@chromium.org

Committed: https://pdfium.googlesource.com/pdfium/+/de4791d291b4252045d7922fb838c64d9b711fdf

[Tom Sepez, 2015-10-30 18:41:04]

Description was changed from

==========
XFA: Restore early return if no FPDF_Page in FORM_DoPageAAction()

Present in original XFA code drop.
==========

to

==========
XFA: Restore early return if no FPDF_Page in FORM_DoPageAAction()

Present in original XFA code drop.

BUG=pdfium:259
==========

[Tom Sepez, 2015-10-30 18:41:04]

tsepez@chromium.org changed reviewers:
+ thestig@chromium.org

[Tom Sepez, 2015-10-30 18:41:15]

Lei, last one.

[Lei Zhang, 2015-10-30 18:50:08]

https://codereview.chromium.org/1425223002/diff/1/fpdfsdk/src/fpdfformfill.cpp
File fpdfsdk/src/fpdfformfill.cpp (left):

https://codereview.chromium.org/1425223002/diff/1/fpdfsdk/src/fpdfformfill.cp...
fpdfsdk/src/fpdfformfill.cpp:666: FX_BOOL bExistOAAction = FALSE;
Remember to merge this to master.

https://codereview.chromium.org/1425223002/diff/1/fpdfsdk/src/fpdfformfill.cpp
File fpdfsdk/src/fpdfformfill.cpp (right):

https://codereview.chromium.org/1425223002/diff/1/fpdfsdk/src/fpdfformfill.cp...
fpdfsdk/src/fpdfformfill.cpp:658: CPDFXFA_Page* pPage = (CPDFXFA_Page*)page;
Just do the pPage->GetPDFPage() check here?

[Tom Sepez, 2015-10-30 18:58:27]

https://codereview.chromium.org/1425223002/diff/1/fpdfsdk/src/fpdfformfill.cpp
File fpdfsdk/src/fpdfformfill.cpp (right):

https://codereview.chromium.org/1425223002/diff/1/fpdfsdk/src/fpdfformfill.cp...
fpdfsdk/src/fpdfformfill.cpp:658: CPDFXFA_Page* pPage = (CPDFXFA_Page*)page;
On 2015/10/30 18:50:08, Lei Zhang wrote:
> Just do the pPage->GetPDFPage() check here?

Done.

[Tom Sepez, 2015-10-30 18:58:42]

https://codereview.chromium.org/1425223002/diff/1/fpdfsdk/src/fpdfformfill.cpp
File fpdfsdk/src/fpdfformfill.cpp (left):

https://codereview.chromium.org/1425223002/diff/1/fpdfsdk/src/fpdfformfill.cp...
fpdfsdk/src/fpdfformfill.cpp:666: FX_BOOL bExistOAAction = FALSE;
On 2015/10/30 18:50:08, Lei Zhang wrote:
> Remember to merge this to master.

Acknowledged.

[Lei Zhang, 2015-10-30 19:12:27]

lgtm

[Tom Sepez, 2015-10-30 19:13:15]

Committed patchset #2 (id:20001) manually as
de4791d291b4252045d7922fb838c64d9b711fdf (presubmit successful).