Fix the way to check terminal field in interactive form

Fix the way to check terminal field in interactive form

Originally the key "T" was checked which was not correct since that
key is optional. Now change to check "FT" as well as its parent's
key since this key is also inheritable.

BUG=chromium:440132
Committed: https://pdfium.googlesource.com/pdfium/+/ab5a20d9de8c801b8376bfe2d6f7cf9ac7d26028

[Wei Li, 2016-12-07 01:23:21]

The CQ bit was checked by weili@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-12-07 01:23:28]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-12-07 01:33:03]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-12-07 01:33:03]

Dry run: This issue passed the CQ dry run.

[Wei Li, 2016-12-07 01:35:33]

weili@chromium.org changed reviewers:
+ dsinclair@chromium.org, tsepez@chromium.org

[Wei Li, 2016-12-07 01:35:34]

pls review, thanks

[dsinclair, 2016-12-07 14:02:40]

The CQ bit was checked by dsinclair@chromium.org

[dsinclair, 2016-12-07 14:02:41]

lgtm

[dsinclair, 2016-12-07 14:02:49]

The CQ bit was unchecked by dsinclair@chromium.org

[dsinclair, 2016-12-07 14:03:13]

On 2016/12/07 14:02:49, dsinclair wrote:
> The CQ bit was unchecked by mailto:dsinclair@chromium.org

Didn't CQ because I wasn't sure if you wanted to roll DEPS for the test addition
in this CL or not.

[Tom Sepez, 2016-12-07 17:16:28]

Let's put the corpus test DEPS roll into this CL once it lands.  Looks good
otherwise.

[Wei Li, 2016-12-07 18:37:13]

The CQ bit was checked by weili@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-12-07 18:37:16]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-12-07 18:52:06]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-12-07 18:52:07]

Dry run: This issue passed the CQ dry run.

[Wei Li, 2016-12-07 19:02:13]

test added, suppress it on mac since the rendered image is slightly different
although the appearance is same.

[Wei Li, 2016-12-07 19:32:28]

The CQ bit was checked by weili@chromium.org

[Wei Li, 2016-12-07 19:32:28]

The patchset sent to the CQ was uploaded after l-g-t-m from
dsinclair@chromium.org
Link to the patchset: https://codereview.chromium.org/2554223002/#ps20001
(title: "add test")

[commit-bot: I haz the power, 2016-12-07 19:32:37]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-12-07 19:32:53]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1481139148077510,
"parent_rev": "833619b4441915c7c55085d44b3221eaef0d9800", "commit_rev":
"ab5a20d9de8c801b8376bfe2d6f7cf9ac7d26028"}

[commit-bot: I haz the power, 2016-12-07 19:32:56]

Description was changed from

==========
Fix the way to check terminal field in interactive form

Originally the key "T" was checked which was not correct since that
key is optional. Now change to check "FT" as well as its parent's
key since this key is also inheritable.

BUG=chromium:440132
==========

to

==========
Fix the way to check terminal field in interactive form

Originally the key "T" was checked which was not correct since that
key is optional. Now change to check "FT" as well as its parent's
key since this key is also inheritable.

BUG=chromium:440132

Committed:
https://pdfium.googlesource.com/pdfium/+/ab5a20d9de8c801b8376bfe2d6f7cf9ac7d2...
==========

[commit-bot: I haz the power, 2016-12-07 19:32:57]

Committed patchset #2 (id:20001) as
https://pdfium.googlesource.com/pdfium/+/ab5a20d9de8c801b8376bfe2d6f7cf9ac7d2...