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

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

Issue 2849633002: Get rid of base::SharedMemory::NULLHandle(); (Closed)
Patch Set: Rebase. 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 33eef882b08c42f5944a28388cf50921e9cc02e4..6bd6a81721b8de29075d9e1923b26d85a9d03360 100644
--- a/content/renderer/media/audio_message_filter_unittest.cc
+++ b/content/renderer/media/audio_message_filter_unittest.cc
@@ -54,7 +54,7 @@ class MockAudioDelegate : public media::AudioOutputIPCDelegate {
device_status_ = media::OUTPUT_DEVICE_STATUS_ERROR_INTERNAL;
created_received_ = false;
- handle_ = base::SharedMemory::NULLHandle();
+ handle_ = base::SharedMemoryHandle();
length_ = 0;
volume_received_ = false;
@@ -130,7 +130,7 @@ TEST(AudioMessageFilterTest, Basic) {
const uint32_t kLength = 1024;
EXPECT_FALSE(delegate.created_received());
filter->OnMessageReceived(AudioMsg_NotifyStreamCreated(
- kStreamId, base::SharedMemory::NULLHandle(), socket_descriptor, kLength));
+ kStreamId, base::SharedMemoryHandle(), socket_descriptor, kLength));
EXPECT_TRUE(delegate.created_received());
EXPECT_FALSE(base::SharedMemory::IsHandleValid(delegate.handle()));
EXPECT_EQ(kLength, delegate.length());

Powered by Google App Engine
This is Rietveld 408576698