Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(813)

Unified Diff: remoting/host/audio_scheduler.h

Issue 11778049: Making DesktopEnvironment a factory class used by ClientSession to create audio/video capturers and… (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | remoting/host/audio_scheduler.cc » ('j') | remoting/host/client_session.h » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/host/audio_scheduler.h
diff --git a/remoting/host/audio_scheduler.h b/remoting/host/audio_scheduler.h
index 43fb3976a8af9c750f32361b41dbf17125c320a1..4f9a22ee6fd554f9761d90c545d0e89e6282ebb2 100644
--- a/remoting/host/audio_scheduler.h
+++ b/remoting/host/audio_scheduler.h
@@ -5,7 +5,6 @@
#ifndef REMOTING_HOST_AUDIO_SCHEDULER_H_
#define REMOTING_HOST_AUDIO_SCHEDULER_H_
-#include "base/callback_forward.h"
#include "base/memory/ref_counted.h"
#include "base/memory/scoped_ptr.h"
@@ -36,17 +35,17 @@ class AudioScheduler : public base::RefCountedThreadSafe<AudioScheduler> {
static scoped_refptr<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);
// Stop the recording session.
- void Stop(const base::Closure& done_task);
+ void Stop();
// Enable or disable audio on a running session.
// This leaves the audio capturer running, and only affects whether or not the
// captured audio is encoded and sent on the wire.
- void SetEnabled(bool enabled);
+ void Pause(bool pause);
private:
friend class base::RefCountedThreadSafe<AudioScheduler>;
@@ -54,7 +53,7 @@ class AudioScheduler : public base::RefCountedThreadSafe<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);
virtual ~AudioScheduler();
@@ -63,7 +62,7 @@ class AudioScheduler : public base::RefCountedThreadSafe<AudioScheduler> {
void StartOnAudioThread();
// Called on the audio thread to stop capturing.
- void StopOnAudioThread(const base::Closure& done_task);
+ void StopOnAudioThread();
// Called on the audio thread when a new audio packet is available.
void EncodeAudioPacket(scoped_ptr<AudioPacket> packet);
@@ -74,7 +73,7 @@ class AudioScheduler : public base::RefCountedThreadSafe<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_;
« no previous file with comments | « no previous file | remoting/host/audio_scheduler.cc » ('j') | remoting/host/client_session.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698