Convert search for most visible page in the document to a binary search.

Convert search for most visible page in the document to a binary search.

The PDF extension needs to compute the most visible page frequently so we use
a binary search.

BUG=303491
R=arv@chromium.org, koz@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=252177

[koz (OOO until 15th September), 2014-02-18 07:04:33]

lgtm

https://codereview.chromium.org/170163003/diff/1/chrome/browser/resources/pdf...
File chrome/browser/resources/pdf/viewport.js (right):

https://codereview.chromium.org/170163003/diff/1/chrome/browser/resources/pdf...
chrome/browser/resources/pdf/viewport.js:192: if (area == 0)
nit: Maybe move this before the if statement and add a comment why it's ok to
break (i.e. zero visible area means we have gone past what's visible in the
viewport).

[raymes, 2014-02-19 05:39:36]

+arv for OWNERS
Note I'll be adding a unittest for this file soon.

https://codereview.chromium.org/170163003/diff/1/chrome/browser/resources/pdf...
File chrome/browser/resources/pdf/viewport.js (right):

https://codereview.chromium.org/170163003/diff/1/chrome/browser/resources/pdf...
chrome/browser/resources/pdf/viewport.js:192: if (area == 0)
On 2014/02/18 07:04:34, koz wrote:
> nit: Maybe move this before the if statement and add a comment why it's ok to
> break (i.e. zero visible area means we have gone past what's visible in the
> viewport).

Done.

[arv (Not doing code reviews), 2014-02-19 21:33:36]

LGTM

[raymes, 2014-02-19 23:49:36]

The CQ bit was checked by raymes@chromium.org

[commit-bot: I haz the power, 2014-02-20 01:20:19]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/raymes@chromium.org/170163003/80001

[raymes, 2014-02-20 04:31:33]

Committed patchset #3 manually as r252177 (presubmit successful).