Use MediaFormat's crop rectangle when available instead of width/height.

Use MediaFormat's crop rectangle when available instead of width/height.

Per Android media team, width/height is not always reliable. See:
https://github.com/google/ExoPlayer/blob/a7b88cd6a9bac4de5ef47b23e0d7fb50c570b3a6/library/src/main/java/com/google/android/exoplayer/MediaCodecVideoTrackRenderer.java#L319

R=qinmin@chromium.org
BUG=internal b/18224769

Committed: https://crrev.com/357d1f8cc2c53eebafa512b258acd6b18d0c44ab
Cr-Commit-Position: refs/heads/master@{#302739}

[gunsch, 2014-11-05 00:05:51]

Verified this change on both:

1) https://www.youtube.com/watch?v=MJ62hh0a9U4
2) https://www.youtube.com/watch?v=tmeOjFno6Do

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

https://codereview.chromium.org/698273003/diff/1/media/base/android/java/src/...
media/base/android/java/src/org/chromium/media/MediaCodecBridge.java:253:
MediaCodecInfo.CodecCapabilities capabilities =
info.getCapabilitiesForType(mime);
note: this change was due to a presubmit error, prevented me from uploading

[qinmin, 2014-11-05 00:28:00]

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

https://codereview.chromium.org/698273003/diff/1/media/base/android/java/src/...
media/base/android/java/src/org/chromium/media/MediaCodecBridge.java:253:
MediaCodecInfo.CodecCapabilities capabilities =
info.getCapabilitiesForType(mime);
On 2014/11/05 00:05:51, gunsch wrote:
> note: this change was due to a presubmit error, prevented me from uploading

this line is now more than 100 chars long, what is the presubmit error? I think
the previous presubmit error is probably 8 char spacing against 4 char spacing

[gunsch, 2014-11-05 00:31:44]

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

https://codereview.chromium.org/698273003/diff/1/media/base/android/java/src/...
media/base/android/java/src/org/chromium/media/MediaCodecBridge.java:253:
MediaCodecInfo.CodecCapabilities capabilities =
info.getCapabilitiesForType(mime);
On 2014/11/05 00:28:00, qinmin wrote:
> On 2014/11/05 00:05:51, gunsch wrote:
> > note: this change was due to a presubmit error, prevented me from uploading
> 
> this line is now more than 100 chars long, what is the presubmit error? I
think
> the previous presubmit error is probably 8 char spacing against 4 char spacing
Yes, indentation was the warning. This is only 98 characters, however.

[qinmin, 2014-11-05 00:36:06]

On 2014/11/05 00:31:44, gunsch wrote:
>
https://codereview.chromium.org/698273003/diff/1/media/base/android/java/src/...
> File media/base/android/java/src/org/chromium/media/MediaCodecBridge.java
> (right):
> 
>
https://codereview.chromium.org/698273003/diff/1/media/base/android/java/src/...
> media/base/android/java/src/org/chromium/media/MediaCodecBridge.java:253:
> MediaCodecInfo.CodecCapabilities capabilities =
> info.getCapabilitiesForType(mime);
> On 2014/11/05 00:28:00, qinmin wrote:
> > On 2014/11/05 00:05:51, gunsch wrote:
> > > note: this change was due to a presubmit error, prevented me from
uploading
> > 
> > this line is now more than 100 chars long, what is the presubmit error? I
> think
> > the previous presubmit error is probably 8 char spacing against 4 char
spacing
> Yes, indentation was the warning. This is only 98 characters, however.

lgtm, i mis-counted the chars

[gunsch, 2014-11-05 01:10:16]

The CQ bit was checked by gunsch@chromium.org

[commit-bot: I haz the power, 2014-11-05 01:12:01]

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

[commit-bot: I haz the power, 2014-11-05 01:53:45]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2014-11-05 01:55:21]

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