Use a separate thread for audio capturing and encoding.

Use a separate thread for audio capturing and encoding.

Previously we were using the same thread for capturing video frames and
audio capturing and encoding. It seems suboptimal as we don't want to block 
screen capturer on audio encoding.

BUG=123046, 145891
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=154597

[Wez, 2012-08-31 04:44:51]

LGTM w/ comment nits

http://codereview.chromium.org/10914029/diff/1/remoting/host/audio_scheduler.h
File remoting/host/audio_scheduler.h (right):

http://codereview.chromium.org/10914029/diff/1/remoting/host/audio_scheduler....
remoting/host/audio_scheduler.h:28: // encoding it and then sending encoded data
over the network to the client.
nit: Suggest "... responsible for fetching audio data from the AudioCapturer and
encoding it before passing it to the AudioStub for delivery to the client."

http://codereview.chromium.org/10914029/diff/1/remoting/host/audio_scheduler....
remoting/host/audio_scheduler.h:34: // encoded. |audio_stub| is used on the
|network_task_runner| thread. The
nit: Suggest rewording "Audio capture and encoding tasks are dispatched via the
|audio_task_runner|. ..." etc

[Sergey Ulanov, 2012-08-31 20:18:49]

http://codereview.chromium.org/10914029/diff/1/remoting/host/audio_scheduler.h
File remoting/host/audio_scheduler.h (right):

http://codereview.chromium.org/10914029/diff/1/remoting/host/audio_scheduler....
remoting/host/audio_scheduler.h:28: // encoding it and then sending encoded data
over the network to the client.
On 2012/08/31 04:44:52, Wez wrote:
> nit: Suggest "... responsible for fetching audio data from the AudioCapturer
and
> encoding it before passing it to the AudioStub for delivery to the client."

Done.

http://codereview.chromium.org/10914029/diff/1/remoting/host/audio_scheduler....
remoting/host/audio_scheduler.h:34: // encoded. |audio_stub| is used on the
|network_task_runner| thread. The
On 2012/08/31 04:44:52, Wez wrote:
> nit: Suggest rewording "Audio capture and encoding tasks are dispatched via
the
> |audio_task_runner|. ..." etc

Done.

[commit-bot: I haz the power, 2012-08-31 23:32:25]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/sergeyu@chromium.org/10914029/7001

[commit-bot: I haz the power, 2012-09-01 03:55:48]

Change committed as 154597