Keep the declaration of FPDF_HasXFAField consistent with other XFA APIs

Keep the declaration of FPDF_HasXFAField consistent with other XFA APIs
It's an enhancement requested in issue 452794.

BUG=433994
R=tsepez@chromium.org

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

[jun_fang, 2015-02-17 22:10:09]

jun_fang@foxitsoftware.com changed reviewers:
+ kai_jing@foxitsoftware.com, tsepez@chromium.org

[jun_fang, 2015-02-17 22:10:09]

Hi Tom, please review this code change.

[Tom Sepez, 2015-02-17 22:16:34]

LGTM once you fix the one comment.

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

https://codereview.chromium.org/939483003/diff/1/fpdfsdk/src/fpdfview.cpp#new...
fpdfsdk/src/fpdfview.cpp:327: FX_BOOL bDymasticXFA = FALSE;
nit: Dymastic isn't a word as far as I know.  Did you mean dynamic?

[jun_fang, 2015-02-17 22:32:41]

New patchsets have been uploaded after l-g-t-m from tsepez@chromium.org

[Tom Sepez, 2015-02-17 22:34:29]

https://codereview.chromium.org/939483003/diff/20001/fpdfsdk/src/fpdfview.cpp
File fpdfsdk/src/fpdfview.cpp (right):

https://codereview.chromium.org/939483003/diff/20001/fpdfsdk/src/fpdfview.cpp...
fpdfsdk/src/fpdfview.cpp:323: FX_BOOL bDynamicXFA = FALSE;
nit: assigning this to false and then immediately re-assiging can be combined
into a single statement

[jun_fang, 2015-02-17 22:53:12]

Committed patchset #3 (id:40001) manually as
e118ce99c52a21a1093badc9d180539a9a6e973f (presubmit successful).