New issue
Advanced search Search tips

Issue 792260 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jan 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 3
Type: Bug



Sign in to add a comment

VideoCaptureDeviceFactoryAndroid::GetSupportedFormats() reports 0 supported formats

Project Member Reported by chfremer@chromium.org, Dec 5 2017

Issue description

Android video capture devices are allowed to report supported pixel formats as UNKNOWN to indicate that the format is not determined until the actual capture is started.

Currently, in VideoCaptureDeviceFactoryAndroid::GetSupportedFormats(), these reported formats get dropped, because they caused certain test timeouts in the base, see [1][2].

We need to figure out why the tests timed out, fix the problem, and then make VideoCaptureDeviceFactoryAndroid::GetSupportedFormats() correctly report the supported pixel format as UNKNOWN back to the caller.

[1]  https://crbug.com/644910 
[2] https://cs.chromium.org/chromium/src/media/capture/video/android/video_capture_device_factory_android.cc?q=video_capture_device_factory_android&dr=C&l=116
 

Comment 1 by z...@chromium.org, Jan 26 2018

I think this might already be fixed by https://chromium-review.googlesource.com/c/chromium/src/+/852450

The problem is that it was always reporting back format 0 (unknown) instead of the actual format: https://chromium-review.googlesource.com/c/chromium/src/+/852450/5/media/capture/video/android/java/src/org/chromium/media/VideoCaptureCamera2.java#b614
Status: Fixed (was: Available)
Agreed. Thanks for pointing this out!
I'll mark this as fixed.

Sign in to add a comment