PDF: Handle hit tests for form controls and links consistently.

PDF: Handle hit tests for form controls and links consistently.

BUG=434064
Committed: https://crrev.com/0ad537cdce686af79f130f4a9cb30d40572e7aef
Cr-Commit-Position: refs/heads/master@{#318748}

[Lei Zhang, 2015-02-28 01:16:14]

thestig@chromium.org changed reviewers:
+ gene@chromium.org, jam@chromium.org

[Lei Zhang, 2015-02-28 01:16:14]

https://codereview.chromium.org/969513002/diff/1/pdf/pdfium/pdfium_page.cc
File pdf/pdfium/pdfium_page.cc (right):

https://codereview.chromium.org/969513002/diff/1/pdf/pdfium/pdfium_page.cc#ne...
pdf/pdfium/pdfium_page.cc:275: FPDPage_HasFormFieldAtPoint(engine_->form(),
GetPage(), new_x, new_y);
FPDPage_HasFormFieldAtPoint() and FPDFLink_GetLinkAtPoint() can both return
valid data if there exists one object of each type and they overlap. We'll need
to fix that corner case in PDFium separately.

I'm just putting FPDPage_HasFormFieldAtPoint() here first because it happens to
fix the bug, and eventually this will all work out.

[jam, 2015-03-02 16:24:01]

lgtm

[Lei Zhang, 2015-03-02 20:02:37]

The CQ bit was checked by thestig@chromium.org

[commit-bot: I haz the power, 2015-03-02 20:04:11]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/969513002/1

[commit-bot: I haz the power, 2015-03-02 21:03:27]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-03-02 21:04:17]

Patchset 1 (id:??) landed as
https://crrev.com/0ad537cdce686af79f130f4a9cb30d40572e7aef
Cr-Commit-Position: refs/heads/master@{#318748}