Implement Field::SetHidden using Field::SetDisplay.

Implement Field::SetHidden using Field::SetDisplay.

Following up on [1], where the duplicated logic in
Field::SetDisplay was factored out into a helper function,
CL further cleans up a related method: ::SetHidden.

Field::SetHidden(true), for instance, is equivalent
to calling Field::SetDisplay(1), whereas Field::SetHidden(false)
is equivalent to Field::SetDisplay(0);

No behavior change is expected.

[1] https://codereview.chromium.org/2255843002

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

[tonikitoo, 2016-08-22 20:49:10]

Description was changed from

==========
Implement Field::SetHidden using Field::SetDisplay.

Following up on [1], where the duplicated logic in
Field::SetDisplay was factored out into a helper function,
CL further cleans up a related method: ::SetHidden.

Field::SetHidden(true), for instance, is equivalent
to calling Field::SetDisplay(1), whereas Field::SetHidden(false)
is equivalent to Field::SetDisplay(0);

No behavior change is expected.

[1] https://codereview.chromium.org/2255843002
==========

to

==========
Implement Field::SetHidden using Field::SetDisplay.

Following up on [1], where the duplicated logic in
Field::SetDisplay was factored out into a helper function,
CL further cleans up a related method: ::SetHidden.

Field::SetHidden(true), for instance, is equivalent
to calling Field::SetDisplay(1), whereas Field::SetHidden(false)
is equivalent to Field::SetDisplay(0);

No behavior change is expected.

[1] https://codereview.chromium.org/2255843002
==========

[tonikitoo, 2016-08-22 20:49:10]

tonikitoo@igalia.com changed reviewers:
+ dsinclair@chromium.org, tsepez@chromium.org

[dsinclair, 2016-08-22 20:55:42]

The CQ bit was checked by dsinclair@chromium.org

[dsinclair, 2016-08-22 20:55:42]

lgtm

[commit-bot: I haz the power, 2016-08-22 20:55:47]

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-08-22 21:06:51]

Description was changed from

==========
Implement Field::SetHidden using Field::SetDisplay.

Following up on [1], where the duplicated logic in
Field::SetDisplay was factored out into a helper function,
CL further cleans up a related method: ::SetHidden.

Field::SetHidden(true), for instance, is equivalent
to calling Field::SetDisplay(1), whereas Field::SetHidden(false)
is equivalent to Field::SetDisplay(0);

No behavior change is expected.

[1] https://codereview.chromium.org/2255843002
==========

to

==========
Implement Field::SetHidden using Field::SetDisplay.

Following up on [1], where the duplicated logic in
Field::SetDisplay was factored out into a helper function,
CL further cleans up a related method: ::SetHidden.

Field::SetHidden(true), for instance, is equivalent
to calling Field::SetDisplay(1), whereas Field::SetHidden(false)
is equivalent to Field::SetDisplay(0);

No behavior change is expected.

[1] https://codereview.chromium.org/2255843002

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

[commit-bot: I haz the power, 2016-08-22 21:06:52]

Committed patchset #1 (id:1) as
https://pdfium.googlesource.com/pdfium/+/a73b8fee8751dae443af9437007261e4a182...