Android Video Capture: use tightest fps Range available on allocate()

Android Video Capture: use tightest fps Range available on allocate()

From the bug descr:

> N7 camera list capabilities are like (extracted):
>
> D/VideoCaptureAndroid(30916): Format 640x480 @ [15000-15000]
> D/VideoCaptureAndroid(30916): Format 640x480 @ [24000-24000]
> D/VideoCaptureAndroid(30916): Format 640x480 @ [15000-30000]
> D/VideoCaptureAndroid(30916): Format 640x480 @ [30000-30000]
>
> where the second numbers are the frame rate _ranges_. The algorithm for
> selecting FPS range in [1] would select the 3rd enumerated range when
> fed with a desired frame rate of 30, while the 4th range would be more
> appropriate.

[1] https://code.google.com/p/chromium/codesearch#chromium/src/media/base/android/java/src/org/chromium/media/VideoCapture.java&sq=package:chromium&type=cs&q=videocapture.java&l=130

TBR=qinmin@chromium.org
BUG=418911
Committed: https://crrev.com/9f73252eb27914b282df24c60549f0581cf0645d
Cr-Commit-Position: refs/heads/master@{#297414}

[mcasas, 2014-09-30 10:10:15]

mcasas@chromium.org changed reviewers:
+ perkj@chromium.org

[mcasas, 2014-09-30 10:10:15]

perkj@ brush off your Java, PTAL :)

[perkj_chrome, 2014-09-30 11:18:06]

lgtm

https://codereview.chromium.org/614003002/diff/1/media/base/android/java/src/...
File media/base/android/java/src/org/chromium/media/VideoCapture.java (right):

https://codereview.chromium.org/614003002/diff/1/media/base/android/java/src/...
media/base/android/java/src/org/chromium/media/VideoCapture.java:125: int
frameRateInMs = frameRate * 1000;
nit: a frameRate can never be in MS since since its the inverse.

[mcasas, 2014-09-30 12:31:52]

The CQ bit was checked by mcasas@chromium.org

[commit-bot: I haz the power, 2014-09-30 12:32:54]

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

[mcasas, 2014-09-30 12:33:07]

https://codereview.chromium.org/614003002/diff/1/media/base/android/java/src/...
File media/base/android/java/src/org/chromium/media/VideoCapture.java (right):

https://codereview.chromium.org/614003002/diff/1/media/base/android/java/src/...
media/base/android/java/src/org/chromium/media/VideoCapture.java:125: int
frameRateInMs = frameRate * 1000;
On 2014/09/30 11:18:06, perkj wrote:
> nit: a frameRate can never be in MS since since its the inverse.

Done.

[commit-bot: I haz the power, 2014-09-30 12:39:49]

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

[commit-bot: I haz the power, 2014-09-30 12:39:49]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[mcasas, 2014-09-30 12:56:40]

The CQ bit was checked by mcasas@chromium.org

[commit-bot: I haz the power, 2014-09-30 12:56:47]

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

[commit-bot: I haz the power, 2014-09-30 13:25:07]

Committed patchset #2 (id:20001) as 6d186a1eecac08eff58a0533f2e9d9e8877554b8

[commit-bot: I haz the power, 2014-09-30 13:25:45]

Patchset 2 (id:??) landed as
https://crrev.com/9f73252eb27914b282df24c60549f0581cf0645d
Cr-Commit-Position: refs/heads/master@{#297414}