Tiny little refactoring of DesktopEnvironment.

remoting/host/audio_scheduler.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "remoting/host/audio_scheduler.h"
 
 #include "base/bind.h"
-#include "base/callback.h"
 #include "base/location.h"
 #include "base/logging.h"
 #include "base/single_thread_task_runner.h"
 #include "remoting/codec/audio_encoder.h"
 #include "remoting/host/audio_capturer.h"
 #include "remoting/proto/audio.pb.h"
 #include "remoting/protocol/audio_stub.h"
 
 namespace remoting {
 
 // static
 scoped_refptr<AudioScheduler> AudioScheduler::Create(
     scoped_refptr<base::SingleThreadTaskRunner> audio_task_runner,
     scoped_refptr<base::SingleThreadTaskRunner> network_task_runner,
-    AudioCapturer* audio_capturer,
+    scoped_ptr<AudioCapturer> audio_capturer,
     scoped_ptr<AudioEncoder> audio_encoder,
     protocol::AudioStub* audio_stub) {
   DCHECK(network_task_runner->BelongsToCurrentThread());
   DCHECK(audio_capturer);
   DCHECK(audio_encoder);
   DCHECK(audio_stub);
 
   scoped_refptr<AudioScheduler> scheduler = new AudioScheduler(
       audio_task_runner, network_task_runner,
-      audio_capturer, audio_encoder.Pass(), audio_stub);
+      audio_capturer.Pass(), audio_encoder.Pass(), audio_stub);
   audio_task_runner->PostTask(
       FROM_HERE, base::Bind(&AudioScheduler::StartOnAudioThread, scheduler));
 
   return scheduler;
 }
 
-void AudioScheduler::Stop(const base::Closure& done_task) {
+void AudioScheduler::Stop() {
   DCHECK(network_task_runner_->BelongsToCurrentThread());
-  DCHECK(!done_task.is_null());
   DCHECK(audio_stub_);
 
   // Clear |audio_stub_| to prevent audio packets being delivered to the client.
   audio_stub_ = NULL;
 
   audio_task_runner_->PostTask(
       FROM_HERE,
-      base::Bind(&AudioScheduler::StopOnAudioThread, this, done_task));
+      base::Bind(&AudioScheduler::StopOnAudioThread, this));
 }
 
 AudioScheduler::AudioScheduler(
     scoped_refptr<base::SingleThreadTaskRunner> audio_task_runner,
     scoped_refptr<base::SingleThreadTaskRunner> network_task_runner,
-    AudioCapturer* audio_capturer,
+    scoped_ptr<AudioCapturer> audio_capturer,
     scoped_ptr<AudioEncoder> audio_encoder,
     protocol::AudioStub* audio_stub)
     : audio_task_runner_(audio_task_runner),
       network_task_runner_(network_task_runner),
-      audio_capturer_(audio_capturer),
+      audio_capturer_(audio_capturer.Pass()),
       audio_encoder_(audio_encoder.Pass()),
       audio_stub_(audio_stub),
       network_stopped_(false),
       enabled_(true) {
 }
 
 AudioScheduler::~AudioScheduler() {
 }
 
 void AudioScheduler::StartOnAudioThread() {
   DCHECK(audio_task_runner_->BelongsToCurrentThread());
 
   // TODO(kxing): Do something with the return value.
   audio_capturer_->Start(
       base::Bind(&AudioScheduler::EncodeAudioPacket, this));
 }
 
-void AudioScheduler::StopOnAudioThread(const base::Closure& done_task) {
+void AudioScheduler::StopOnAudioThread() {
   DCHECK(audio_task_runner_->BelongsToCurrentThread());
   audio_capturer_->Stop();
-
-  network_task_runner_->PostTask(FROM_HERE, done_task);
 }
 
-void AudioScheduler::SetEnabled(bool enabled) {
+void AudioScheduler::Pause(bool pause) {
   if (!audio_task_runner_->BelongsToCurrentThread()) {
     audio_task_runner_->PostTask(
-        FROM_HERE, base::Bind(&AudioScheduler::SetEnabled, this, enabled));
+        FROM_HERE, base::Bind(&AudioScheduler::Pause, this, pause));
     return;
   }
 
-  enabled_ = enabled;
+  enabled_ = !pause;
 }
 
 void AudioScheduler::EncodeAudioPacket(scoped_ptr<AudioPacket> packet) {
   DCHECK(audio_task_runner_->BelongsToCurrentThread());
   DCHECK(packet.get());
 
   if (!enabled_)
     return;
 
   scoped_ptr<AudioPacket> encoded_packet =
@@ skipping 11 matching lines @@
   DCHECK(network_task_runner_->BelongsToCurrentThread());
   DCHECK(packet.get());
 
   if (!audio_stub_)
     return;
 
   audio_stub_->ProcessAudioPacket(packet.Pass(), base::Closure());
 }
 
 }  // namespace remoting