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

Unified Diff: content/renderer/media/audio_message_filter_unittest.cc

Issue 2852803002: Remove base::SharedMemory::ShareToProcess. (Closed)
Patch Set: Compile error. Created 3 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: content/renderer/media/audio_message_filter_unittest.cc
diff --git a/content/renderer/media/audio_message_filter_unittest.cc b/content/renderer/media/audio_message_filter_unittest.cc
index 6bd6a81721b8de29075d9e1923b26d85a9d03360..161970624b467abfdecd9aabcfbb680569025803 100644
--- a/content/renderer/media/audio_message_filter_unittest.cc
+++ b/content/renderer/media/audio_message_filter_unittest.cc
@@ -24,6 +24,10 @@ class MockAudioDelegate : public media::AudioOutputIPCDelegate {
MockAudioDelegate() {
Reset();
}
+ ~MockAudioDelegate() override {
+ if (handle_.IsValid())
+ handle_.Close();
+ }
void OnError() override { error_received_ = true; }
@@ -54,6 +58,8 @@ class MockAudioDelegate : public media::AudioOutputIPCDelegate {
device_status_ = media::OUTPUT_DEVICE_STATUS_ERROR_INTERNAL;
created_received_ = false;
+ if (handle_.IsValid())
+ handle_.Close();
handle_ = base::SharedMemoryHandle();
length_ = 0;
« no previous file with comments | « content/common/sandbox_mac_fontloading_unittest.mm ('k') | content/renderer/media/speech_recognition_audio_sink_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698