Faster YUV conversion in WebrtcVideoCapturerAdapter.

Faster YUV conversion in WebrtcVideoCapturerAdapter.

WebrtcVideoCapturerAdapter was previously relying on
cricket::VideoCapturer for YUV conversion, which was also resizing the
frames, but that's not desired. Now it's responsible for YUV conversion,
which also allows to optimize it by coverting only parts of the screen
that are changing.

Also fixed WebrtcVideoCapturerAdapter to avoid multiple simultaneous capture
requests, which was causing assert failure in DesktopCapturer.

BUG=547158
Committed: https://crrev.com/5b105ebfe8dc4a5e11bfa8fe56aa3f544eeae902
Cr-Commit-Position: refs/heads/master@{#368508}

[Sergey Ulanov, 2016-01-08 19:34:58]

Description was changed from

==========
Bugfixes in WebrtcVideoCapturerAdapter.

WebrtcVideoCapturerAdapter was previously relying on
cricket::VideoCapturer for YUV conversion, which was also resizing the
frames, but that's not desired.

Alos fixed WebrtcVideoCapturerAdapter to avoid multiple simultaneous capture
requests, which was causing assert failure in DesktopCapturer.

BUG=547158
==========

to

==========
Faster YUV conversion in WebrtcVideoCapturerAdapter.

WebrtcVideoCapturerAdapter was previously relying on
cricket::VideoCapturer for YUV conversion, which was also resizing the
frames, but that's not desired. Now it's responsible for YUV conversion,
which also allows to optimize it by coverting only parts of the screen
that are changing.

Also fixed WebrtcVideoCapturerAdapter to avoid multiple simultaneous capture
requests, which was causing assert failure in DesktopCapturer.

BUG=547158
==========

[Sergey Ulanov, 2016-01-08 19:36:20]

sergeyu@chromium.org changed reviewers:
+ jamiewalch@chromium.org

[Sergey Ulanov, 2016-01-08 19:36:21]

Moved half of https://codereview.chromium.org/1571543002/

[Jamie, 2016-01-08 23:33:53]

lgtm

https://codereview.chromium.org/1574543002/diff/20001/remoting/protocol/webrt...
File remoting/protocol/webrtc_video_capturer_adapter.cc (right):

https://codereview.chromium.org/1574543002/diff/20001/remoting/protocol/webrt...
remoting/protocol/webrtc_video_capturer_adapter.cc:182: int left = 
i.rect().left();
Nit: Unnecessary extra space after =

https://codereview.chromium.org/1574543002/diff/20001/remoting/protocol/webrt...
remoting/protocol/webrtc_video_capturer_adapter.cc:194: }
Is it only the top-left corner that must be on a 2px boundary, or do you also
need to make sure that with and height are even?

[Sergey Ulanov, 2016-01-09 01:33:29]

https://codereview.chromium.org/1574543002/diff/20001/remoting/protocol/webrt...
File remoting/protocol/webrtc_video_capturer_adapter.cc (right):

https://codereview.chromium.org/1574543002/diff/20001/remoting/protocol/webrt...
remoting/protocol/webrtc_video_capturer_adapter.cc:182: int left = 
i.rect().left();
On 2016/01/08 23:33:52, Jamie wrote:
> Nit: Unnecessary extra space after =

Done.

https://codereview.chromium.org/1574543002/diff/20001/remoting/protocol/webrt...
remoting/protocol/webrtc_video_capturer_adapter.cc:194: }
On 2016/01/08 23:33:52, Jamie wrote:
> Is it only the top-left corner that must be on a 2px boundary, or do you also
> need to make sure that with and height are even?

ARGBToI420() supports odd width and height.

[Sergey Ulanov, 2016-01-09 01:33:40]

The CQ bit was checked by sergeyu@chromium.org

[Sergey Ulanov, 2016-01-09 01:33:41]

The patchset sent to the CQ was uploaded after l-g-t-m from
jamiewalch@chromium.org
Link to the patchset: https://codereview.chromium.org/1574543002/#ps40001
(title: " ")

[commit-bot: I haz the power, 2016-01-09 01:34:43]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1574543002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1574543002/40001

[commit-bot: I haz the power, 2016-01-09 02:11:55]

Description was changed from

==========
Faster YUV conversion in WebrtcVideoCapturerAdapter.

WebrtcVideoCapturerAdapter was previously relying on
cricket::VideoCapturer for YUV conversion, which was also resizing the
frames, but that's not desired. Now it's responsible for YUV conversion,
which also allows to optimize it by coverting only parts of the screen
that are changing.

Also fixed WebrtcVideoCapturerAdapter to avoid multiple simultaneous capture
requests, which was causing assert failure in DesktopCapturer.

BUG=547158
==========

to

==========
Faster YUV conversion in WebrtcVideoCapturerAdapter.

WebrtcVideoCapturerAdapter was previously relying on
cricket::VideoCapturer for YUV conversion, which was also resizing the
frames, but that's not desired. Now it's responsible for YUV conversion,
which also allows to optimize it by coverting only parts of the screen
that are changing.

Also fixed WebrtcVideoCapturerAdapter to avoid multiple simultaneous capture
requests, which was causing assert failure in DesktopCapturer.

BUG=547158
==========

[commit-bot: I haz the power, 2016-01-09 02:11:55]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-01-09 02:12:53]

Description was changed from

==========
Faster YUV conversion in WebrtcVideoCapturerAdapter.

WebrtcVideoCapturerAdapter was previously relying on
cricket::VideoCapturer for YUV conversion, which was also resizing the
frames, but that's not desired. Now it's responsible for YUV conversion,
which also allows to optimize it by coverting only parts of the screen
that are changing.

Also fixed WebrtcVideoCapturerAdapter to avoid multiple simultaneous capture
requests, which was causing assert failure in DesktopCapturer.

BUG=547158
==========

to

==========
Faster YUV conversion in WebrtcVideoCapturerAdapter.

WebrtcVideoCapturerAdapter was previously relying on
cricket::VideoCapturer for YUV conversion, which was also resizing the
frames, but that's not desired. Now it's responsible for YUV conversion,
which also allows to optimize it by coverting only parts of the screen
that are changing.

Also fixed WebrtcVideoCapturerAdapter to avoid multiple simultaneous capture
requests, which was causing assert failure in DesktopCapturer.

BUG=547158

Committed: https://crrev.com/5b105ebfe8dc4a5e11bfa8fe56aa3f544eeae902
Cr-Commit-Position: refs/heads/master@{#368508}
==========

[commit-bot: I haz the power, 2016-01-09 02:12:54]

Patchset 3 (id:??) landed as
https://crrev.com/5b105ebfe8dc4a5e11bfa8fe56aa3f544eeae902
Cr-Commit-Position: refs/heads/master@{#368508}