Add support to Document::getAnnots method

Add support to Document::getAnnots method

Although notably, the parameters handling support is not
complete, CL intends to be the first step towards a more 
complete implementation of this API.

TEST=testing/resources/javascript/bug_492_1.in

BUG=pdfium:492
Committed: https://pdfium.googlesource.com/pdfium/+/0ee35908e906922a423fb18d7085ef80d0d8d8c8

[tonikitoo, 2016-08-26 20:51:58]

Description was changed from

==========
Implement Document::getAnnots API

CL adds an initial support Document::getAnnots API.

Notably, the parameters handling is not complete, but
intented to be a first step towards a more complete
implementation.

TEST=testing/resources/javascript/bug_492_1.in

BUG=pdfium:492
==========

to

==========
Add support to Document::getAnnots method

Although notably, the parameters handling support is not
complete, CL intends to be the first step towards a more 
complete implementation of this API.

TEST=testing/resources/javascript/bug_492_1.in

BUG=pdfium:492
==========

[tonikitoo, 2016-08-26 20:51:59]

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

[tonikitoo, 2016-08-26 20:52:22]

PTAL

[Tom Sepez, 2016-08-26 21:10:06]

Very nice. LGTM with style nit.

https://codereview.chromium.org/2281273002/diff/1/testing/resources/javascrip...
File testing/resources/javascript/bug_492_1.in (right):

https://codereview.chromium.org/2281273002/diff/1/testing/resources/javascrip...
testing/resources/javascript/bug_492_1.in:66: /QuadPoints [ 115.80264
718.9139232 157.211172 718.9139232 115.80264 706.264416 157.211172 706.264416 ]
nit: style: 80 columns, I'd write this as
  /QuadPoints [
    115.80264
    718.9139232
    157.211172
    718.9139232
    115.80264
    706.26441
    6 
    157.211172
    706.264416
  ]

since it wont fit on a single line.

[tonikitoo, 2016-08-26 21:21:35]

https://codereview.chromium.org/2281273002/diff/1/testing/resources/javascrip...
File testing/resources/javascript/bug_492_1.in (right):

https://codereview.chromium.org/2281273002/diff/1/testing/resources/javascrip...
testing/resources/javascript/bug_492_1.in:66: /QuadPoints [ 115.80264
718.9139232 157.211172 718.9139232 115.80264 706.264416 157.211172 706.264416 ]
On 2016/08/26 21:10:06, Tom Sepez wrote:
> nit: style: 80 columns, I'd write this as
>   /QuadPoints [
>     115.80264
>     718.9139232
>     157.211172
>     718.9139232
>     115.80264
>     706.26441
>     6 
>     157.211172
>     706.264416
>   ]
> 
> since it wont fit on a single line.

Done.

[tonikitoo, 2016-08-26 21:23:27]

The CQ bit was checked by tonikitoo@igalia.com

[tonikitoo, 2016-08-26 21:23:27]

The patchset sent to the CQ was uploaded after l-g-t-m from tsepez@chromium.org
Link to the patchset: https://codereview.chromium.org/2281273002/#ps20001
(title: "fixed nit as per tsepez's request")

[commit-bot: I haz the power, 2016-08-26 21:23:31]

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-26 21:41:31]

Description was changed from

==========
Add support to Document::getAnnots method

Although notably, the parameters handling support is not
complete, CL intends to be the first step towards a more 
complete implementation of this API.

TEST=testing/resources/javascript/bug_492_1.in

BUG=pdfium:492
==========

to

==========
Add support to Document::getAnnots method

Although notably, the parameters handling support is not
complete, CL intends to be the first step towards a more 
complete implementation of this API.

TEST=testing/resources/javascript/bug_492_1.in

BUG=pdfium:492

Committed:
https://pdfium.googlesource.com/pdfium/+/0ee35908e906922a423fb18d7085ef80d0d8...
==========

[commit-bot: I haz the power, 2016-08-26 21:41:32]

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