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

Unified Diff: content/renderer/media/audio_input_device.h

Issue 9121045: Switch AudioDevice classes from SyncSocket to CancelableSyncSocket. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Address comments, add temporary ScopedAllowIO for the audio thread cleanup+TODO for next cl. Created 8 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
Index: content/renderer/media/audio_input_device.h
diff --git a/content/renderer/media/audio_input_device.h b/content/renderer/media/audio_input_device.h
index ec7452559c409b20396392ac92bd7f2cb9e474ac..2529f79c2f8517f1bc1aa4357c61b5925cd07d63 100644
--- a/content/renderer/media/audio_input_device.h
+++ b/content/renderer/media/audio_input_device.h
@@ -164,6 +164,8 @@ class CONTENT_EXPORT AudioInputDevice
void StartOnIOThread();
void ShutDownOnIOThread(base::WaitableEvent* completion);
void SetVolumeOnIOThread(double volume);
+ // Closes socket and joins with the audio thread.
+ void ShutDownAudioThread();
void Send(IPC::Message* message);
@@ -209,7 +211,7 @@ class CONTENT_EXPORT AudioInputDevice
bool pending_device_ready_;
base::SharedMemoryHandle shared_memory_handle_;
- base::SyncSocket::Handle socket_handle_;
+ scoped_ptr<base::CancelableSyncSocket> audio_socket_;
int memory_length_;
DISALLOW_IMPLICIT_CONSTRUCTORS(AudioInputDevice);

Powered by Google App Engine
This is Rietveld 408576698