Update CaptureVideoDecoder to handle DeviceInfoReceived() calls.

Update CaptureVideoDecoder to handle DeviceInfoReceived() calls.

Previously the natural size output by the video decoder was not updated based on the device information reported by the capture device.

Also cleaned up logging statements and unit tests.

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=108685

[wjia(left Chromium), 2011-11-04 03:31:16]

Thanks for the patch, Andrew!

http://codereview.chromium.org/8466001/diff/2001/content/renderer/media/captu...
File content/renderer/media/capture_video_decoder_unittest.cc (right):

http://codereview.chromium.org/8466001/diff/2001/content/renderer/media/captu...
content/renderer/media/capture_video_decoder_unittest.cc:159: void Read() {
Read() alone will fail to receive FrameReady(). It has to be combined with
Seek(). How about using:
void Start() {
  // put Read() code here;
  // put Seek() code here;
}

[scherkus (not reviewing), 2011-11-04 16:32:57]

http://codereview.chromium.org/8466001/diff/2001/content/renderer/media/captu...
File content/renderer/media/capture_video_decoder_unittest.cc (right):

http://codereview.chromium.org/8466001/diff/2001/content/renderer/media/captu...
content/renderer/media/capture_video_decoder_unittest.cc:159: void Read() {
On 2011/11/04 03:31:16, wjia wrote:
> Read() alone will fail to receive FrameReady(). It has to be combined with
> Seek(). How about using:
> void Start() {
>   // put Read() code here;
>   // put Seek() code here;
> }

Done.

[wjia(left Chromium), 2011-11-04 16:36:44]

lgtm

[commit-bot: I haz the power, 2011-11-04 16:37:14]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/scherkus@chromium.org/8466001/6006

[commit-bot: I haz the power, 2011-11-04 18:32:25]

Change committed as 108685