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

Unified Diff: media/audio/audio_output_dispatcher.h

Issue 10067035: RefCounted types should not have public destructors, media/ and gpu/ edition (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 8 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: media/audio/audio_output_dispatcher.h
diff --git a/media/audio/audio_output_dispatcher.h b/media/audio/audio_output_dispatcher.h
index 79474a4cff318ebcda0a29f247313026ed522a58..8cc8f8d047303228921733673fd44832b0124d40 100644
--- a/media/audio/audio_output_dispatcher.h
+++ b/media/audio/audio_output_dispatcher.h
@@ -46,7 +46,6 @@ class MEDIA_EXPORT AudioOutputDispatcher
AudioOutputDispatcher(AudioManager* audio_manager,
const AudioParameters& params,
base::TimeDelta close_delay);
- ~AudioOutputDispatcher();
// Called by AudioOutputProxy when the stream is closed. Opens a new
// physical stream if there are no pending streams in |idle_streams_|.
@@ -73,7 +72,9 @@ class MEDIA_EXPORT AudioOutputDispatcher
void Shutdown();
private:
+ friend base::RefCountedThreadSafe<AudioOutputDispatcher>;
friend class AudioOutputProxyTest;
+ ~AudioOutputDispatcher();
scherkus (not reviewing) 2012/04/13 18:06:53 +virtual
// Creates a new physical output stream, opens it and pushes to
// |idle_streams_|. Returns false if the stream couldn't be created or

Powered by Google App Engine
This is Rietveld 408576698