Revert of Removed unused cricket::VideoCapturer methods

Revert of Removed unused cricket::VideoCapturer methods (patchset #2 id:30001 of https://codereview.webrtc.org/1733673002/ )

Reason for revert:
Breaks remoting::protocol::WebrtcVideoCapturerAdapter::Pause'

See https://build.chromium.org/p/chromium.webrtc.fyi/builders/Win%20Builder/builds/3689/steps/compile/logs/stdio

Original issue's description:
> Removed unused cricket::VideoCapturer methods:
>
> void UpdateAspectRatio(int ratio_w, int ratio_h);
> void ClearAspectRatio();
> ool Pause(bool paused);
> Restart(const VideoFormat& capture_format);
> MuteToBlackThenPause(bool muted);
> IsMuted() const
> set_square_pixel_aspect_ratio
> bool square_pixel_aspect_ratio()
>
> This cl also remove the use of messages and posting of state change.
> Further more - a thread checker is added to make sure methods are called on only one thread. Construction can happen on a separate thred.
> It does not add restrictions on what thread frames are delivered on though.
>
> There is more features in VideoCapturer::Onframe related to screen share in ARGB that probably can be cleaned up in a follow up cl.
>
> BUG=webrtc:5426
>
> Committed: https://crrev.com/e9c0cdff2dad2553b6ff6820c0c7429cb2854861
> Cr-Commit-Position: refs/heads/master@{#11773}

TBR=magjed@webrtc.org,pthatcher@webrtc.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=webrtc:5426
Committed: https://crrev.com/74622e061320d9fab4388c260b2d806de4b34e2f
Cr-Commit-Position: refs/heads/master@{#11777}

[perkj_webrtc, 2016-02-26 10:53:48]

Created Revert of Removed unused cricket::VideoCapturer methods

[commit-bot: I haz the power, 2016-02-26 10:54:03]

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

[commit-bot: I haz the power, 2016-02-26 10:54:41]

Description was changed from

==========
Revert of Removed unused cricket::VideoCapturer methods (patchset #2 id:30001 of
https://codereview.webrtc.org/1733673002/ )

Reason for revert:
Breaks remoting::protocol::WebrtcVideoCapturerAdapter::Pause'

See
https://build.chromium.org/p/chromium.webrtc.fyi/builders/Win%20Builder/build...

Original issue's description:
> Removed unused cricket::VideoCapturer methods:
> 
> void UpdateAspectRatio(int ratio_w, int ratio_h);
> void ClearAspectRatio();
> ool Pause(bool paused);
> Restart(const VideoFormat& capture_format);
> MuteToBlackThenPause(bool muted);
> IsMuted() const
> set_square_pixel_aspect_ratio
> bool square_pixel_aspect_ratio()
> 
> This cl also remove the use of messages and posting of state change.
> Further more - a thread checker is added to make sure methods are called on
only one thread. Construction can happen on a separate thred.
> It does not add restrictions on what thread frames are delivered on though.
> 
> There is more features in VideoCapturer::Onframe related to screen share in
ARGB that probably can be cleaned up in a follow up cl.
> 
> BUG=webrtc:5426
> 
> Committed: https://crrev.com/e9c0cdff2dad2553b6ff6820c0c7429cb2854861
> Cr-Commit-Position: refs/heads/master@{#11773}

TBR=magjed@webrtc.org,pthatcher@webrtc.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=webrtc:5426
==========

to

==========
Revert of Removed unused cricket::VideoCapturer methods (patchset #2 id:30001 of
https://codereview.webrtc.org/1733673002/ )

Reason for revert:
Breaks remoting::protocol::WebrtcVideoCapturerAdapter::Pause'

See
https://build.chromium.org/p/chromium.webrtc.fyi/builders/Win%20Builder/build...

Original issue's description:
> Removed unused cricket::VideoCapturer methods:
> 
> void UpdateAspectRatio(int ratio_w, int ratio_h);
> void ClearAspectRatio();
> ool Pause(bool paused);
> Restart(const VideoFormat& capture_format);
> MuteToBlackThenPause(bool muted);
> IsMuted() const
> set_square_pixel_aspect_ratio
> bool square_pixel_aspect_ratio()
> 
> This cl also remove the use of messages and posting of state change.
> Further more - a thread checker is added to make sure methods are called on
only one thread. Construction can happen on a separate thred.
> It does not add restrictions on what thread frames are delivered on though.
> 
> There is more features in VideoCapturer::Onframe related to screen share in
ARGB that probably can be cleaned up in a follow up cl.
> 
> BUG=webrtc:5426
> 
> Committed: https://crrev.com/e9c0cdff2dad2553b6ff6820c0c7429cb2854861
> Cr-Commit-Position: refs/heads/master@{#11773}

TBR=magjed@webrtc.org,pthatcher@webrtc.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=webrtc:5426
==========

[commit-bot: I haz the power, 2016-02-26 10:54:42]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-02-26 10:54:48]

Description was changed from

==========
Revert of Removed unused cricket::VideoCapturer methods (patchset #2 id:30001 of
https://codereview.webrtc.org/1733673002/ )

Reason for revert:
Breaks remoting::protocol::WebrtcVideoCapturerAdapter::Pause'

See
https://build.chromium.org/p/chromium.webrtc.fyi/builders/Win%20Builder/build...

Original issue's description:
> Removed unused cricket::VideoCapturer methods:
> 
> void UpdateAspectRatio(int ratio_w, int ratio_h);
> void ClearAspectRatio();
> ool Pause(bool paused);
> Restart(const VideoFormat& capture_format);
> MuteToBlackThenPause(bool muted);
> IsMuted() const
> set_square_pixel_aspect_ratio
> bool square_pixel_aspect_ratio()
> 
> This cl also remove the use of messages and posting of state change.
> Further more - a thread checker is added to make sure methods are called on
only one thread. Construction can happen on a separate thred.
> It does not add restrictions on what thread frames are delivered on though.
> 
> There is more features in VideoCapturer::Onframe related to screen share in
ARGB that probably can be cleaned up in a follow up cl.
> 
> BUG=webrtc:5426
> 
> Committed: https://crrev.com/e9c0cdff2dad2553b6ff6820c0c7429cb2854861
> Cr-Commit-Position: refs/heads/master@{#11773}

TBR=magjed@webrtc.org,pthatcher@webrtc.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=webrtc:5426
==========

to

==========
Revert of Removed unused cricket::VideoCapturer methods (patchset #2 id:30001 of
https://codereview.webrtc.org/1733673002/ )

Reason for revert:
Breaks remoting::protocol::WebrtcVideoCapturerAdapter::Pause'

See
https://build.chromium.org/p/chromium.webrtc.fyi/builders/Win%20Builder/build...

Original issue's description:
> Removed unused cricket::VideoCapturer methods:
>
> void UpdateAspectRatio(int ratio_w, int ratio_h);
> void ClearAspectRatio();
> ool Pause(bool paused);
> Restart(const VideoFormat& capture_format);
> MuteToBlackThenPause(bool muted);
> IsMuted() const
> set_square_pixel_aspect_ratio
> bool square_pixel_aspect_ratio()
>
> This cl also remove the use of messages and posting of state change.
> Further more - a thread checker is added to make sure methods are called on
only one thread. Construction can happen on a separate thred.
> It does not add restrictions on what thread frames are delivered on though.
>
> There is more features in VideoCapturer::Onframe related to screen share in
ARGB that probably can be cleaned up in a follow up cl.
>
> BUG=webrtc:5426
>
> Committed: https://crrev.com/e9c0cdff2dad2553b6ff6820c0c7429cb2854861
> Cr-Commit-Position: refs/heads/master@{#11773}

TBR=magjed@webrtc.org,pthatcher@webrtc.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=webrtc:5426

Committed: https://crrev.com/74622e061320d9fab4388c260b2d806de4b34e2f
Cr-Commit-Position: refs/heads/master@{#11777}
==========

[commit-bot: I haz the power, 2016-02-26 10:54:49]

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