Log supported camera preview resolutions.

Log supported camera preview resolutions.

BUG=b/29935437
R=sakal@webrtc.org

Committed: https://crrev.com/772bd0d40b3b4c18d391c943b92c9c2714a8c027
Cr-Commit-Position: refs/heads/master@{#14336}

[AlexG, 2016-09-19 21:28:33]

glaznev@webrtc.org changed reviewers:
+ magjed@webrtc.org

[AlexG, 2016-09-19 21:28:33]

PTAL

[sakal, 2016-09-20 07:38:32]

sakal@webrtc.org changed reviewers:
+ sakal@webrtc.org

[sakal, 2016-09-20 07:38:33]

I think changing the resolution selection logic is the major change here and
should be mentioned in the title of the CL.

https://codereview.webrtc.org/2356563002/diff/1/webrtc/api/android/java/src/o...
File webrtc/api/android/java/src/org/webrtc/VideoCapturerAndroid.java (right):

https://codereview.webrtc.org/2356563002/diff/1/webrtc/api/android/java/src/o...
webrtc/api/android/java/src/org/webrtc/VideoCapturerAndroid.java:398:
Logging.d(TAG, "Available preview sizes: " + supportedPreviewSizes);
Also add this statement to Camera2Session?

[AlexG, 2016-09-20 18:39:06]

On 2016/09/20 07:38:33, sakal wrote:
> I think changing the resolution selection logic is the major change here and
> should be mentioned in the title of the CL.
> 
>
https://codereview.webrtc.org/2356563002/diff/1/webrtc/api/android/java/src/o...
> File webrtc/api/android/java/src/org/webrtc/VideoCapturerAndroid.java (right):
> 
>
https://codereview.webrtc.org/2356563002/diff/1/webrtc/api/android/java/src/o...
> webrtc/api/android/java/src/org/webrtc/VideoCapturerAndroid.java:398:
> Logging.d(TAG, "Available preview sizes: " + supportedPreviewSizes);
> Also add this statement to Camera2Session?

Done

[AlexG, 2016-09-20 18:47:27]

Description was changed from

==========
Log supported camera preview resolutions.

Also update camera preview size selection to prefer
target camera width to camera height.

BUG=b/29935437
==========

to

==========
Log supported camera preview resolutions.

BUG=b/29935437
==========

[sakal, 2016-09-21 08:25:49]

lgtm

[AlexG, 2016-09-21 17:14:36]

The CQ bit was checked by glaznev@webrtc.org

[commit-bot: I haz the power, 2016-09-21 17:14:48]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...

[commit-bot: I haz the power, 2016-09-21 19:15:21]

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

[commit-bot: I haz the power, 2016-09-21 19:15:22]

Try jobs failed on following builders:
  android_arm64_rel on master.tryserver.webrtc (JOB_TIMED_OUT, no build URL)

[AlexG, 2016-09-21 19:17:18]

Description was changed from

==========
Log supported camera preview resolutions.

BUG=b/29935437
==========

to

==========
Log supported camera preview resolutions.

BUG=b/29935437
R=sakal@webrtc.org

Committed:
https://chromium.googlesource.com/external/webrtc/+/772bd0d40b3b4c18d391c943b...
==========

[AlexG, 2016-09-21 19:17:20]

Committed patchset #3 (id:40001) manually as
772bd0d40b3b4c18d391c943b92c9c2714a8c027 (presubmit successful).

[commit-bot: I haz the power, 2016-09-21 19:17:20]

Description was changed from

==========
Log supported camera preview resolutions.

BUG=b/29935437
R=sakal@webrtc.org

Committed:
https://chromium.googlesource.com/external/webrtc/+/772bd0d40b3b4c18d391c943b...
==========

to

==========
Log supported camera preview resolutions.

BUG=b/29935437
R=sakal@webrtc.org

Committed: https://crrev.com/772bd0d40b3b4c18d391c943b92c9c2714a8c027
Cr-Commit-Position: refs/heads/master@{#14336}
==========