Moved the fake input stream's processing onto the audio worker thread.

media/audio/fake_audio_worker.h

 // Copyright (c) 2013 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.
 
-#ifndef MEDIA_AUDIO_FAKE_AUDIO_CONSUMER_H_
-#define MEDIA_AUDIO_FAKE_AUDIO_CONSUMER_H_
+#ifndef MEDIA_AUDIO_FAKE_AUDIO_WORKER_H_
+#define MEDIA_AUDIO_FAKE_AUDIO_WORKER_H_
 
 #include "base/callback_forward.h"
 #include "base/memory/ref_counted.h"
 #include "media/base/media_export.h"
 
 namespace base {
 class SingleThreadTaskRunner;
 }
 
 namespace media {
 class AudioBus;
 class AudioParameters;
 
-// A fake audio consumer.  Using a provided message loop, FakeAudioConsumer will
-// simulate a real time consumer of audio data.
-class MEDIA_EXPORT FakeAudioConsumer {
+// A fake audio worker.  Using a provided message loop, FakeAudioWorker will
+// call back the provided callback like a real audio consumer or producer would.
+class MEDIA_EXPORT FakeAudioWorker {
  public:
-  // |worker_task_runner| is the task runner on which the ReadCB provided to
+  // |worker_task_runner| is the task runner on which the WorkerCB provided to
   // Start() will be executed on.  This may or may not be the be for the same
   // thread that invokes the Start/Stop methods.
   // |params| is used to determine the frequency of callbacks.
-  FakeAudioConsumer(
+  FakeAudioWorker(
       const scoped_refptr<base::SingleThreadTaskRunner>& worker_task_runner,
       const AudioParameters& params);
-  ~FakeAudioConsumer();
+  ~FakeAudioWorker();
 
-  // Start executing |read_cb| at a regular intervals.  Stop() must be called by
-  // the same thread before destroying FakeAudioConsumer.
-  typedef base::Callback<void(AudioBus* audio_bus)> ReadCB;
-  void Start(const ReadCB& read_cb);
+  // Start executing |worker_cb| at a regular intervals.  Stop() must be called
+  // by the same thread before destroying FakeAudioWorker.
+  typedef base::Callback<void()> WorkerCB;

[DaleCurtis, 2015/02/18 19:04:49]

Delete typedef in favor of base::Closure.

[phoglund_chromium, 2015/02/19 15:44:11]

Done.

+  void Start(const WorkerCB& worker_cb);
 
-  // Stop executing the ReadCB provided to Start().  Blocks until the worker
-  // loop is not inside a ReadCB invocation.  Safe to call multiple times.  Must
-  // be called on the same thread that called Start().
+  // Stop executing the WorkerCB provided to Start(). Blocks until the worker
+  // loop is not inside a WorkerCB invocation. Safe to call multiple times.
+  // Must be called on the same thread that called Start().
   void Stop();
 
  private:
   // All state and implementation is kept within this ref-counted class because
   // cancellation of posted tasks must happen on the worker thread some time
   // after the call to Stop() (on the main thread) returns.
   class Worker;
   const scoped_refptr<Worker> worker_;
 
-  DISALLOW_COPY_AND_ASSIGN(FakeAudioConsumer);
+  DISALLOW_COPY_AND_ASSIGN(FakeAudioWorker);
 };
 
 }  // namespace media
 
-#endif  // MEDIA_AUDIO_FAKE_AUDIO_CONSUMER_H_
+#endif  // MEDIA_AUDIO_FAKE_AUDIO_WORKER_H_