Chromium Code Reviews| 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 cdd8288850674277c447e4f3160958eef1247295..81ba88e2b432556cdeee4b42d049e718f0fb3339 100644 |
| --- a/content/renderer/media/audio_message_filter_unittest.cc |
| +++ b/content/renderer/media/audio_message_filter_unittest.cc |
| @@ -86,17 +86,11 @@ TEST(AudioMessageFilterTest, Basic) { |
| EXPECT_EQ(&delegate, filter->delegates_.Lookup(kStreamId)); |
| // AudioMsg_NotifyStreamCreated |
|
henrika (OOO until Aug 14)
2014/09/02 07:46:06
What does this comment mean really?
burnik
2014/09/02 10:02:54
Not my doing. I guess it means that a message name
|
| -#if defined(OS_WIN) |
| - base::SyncSocket::Handle socket_handle; |
| -#else |
| - base::FileDescriptor socket_handle; |
| -#endif |
| + base::SyncSocket::TransitDescriptor socket_descriptor; |
| const uint32 kLength = 1024; |
| EXPECT_FALSE(delegate.created_received()); |
| - filter->OnMessageReceived( |
| - AudioMsg_NotifyStreamCreated( |
| - kStreamId, base::SharedMemory::NULLHandle(), |
| - socket_handle, kLength)); |
| + filter->OnMessageReceived(AudioMsg_NotifyStreamCreated( |
| + kStreamId, base::SharedMemory::NULLHandle(), socket_descriptor, kLength)); |
| EXPECT_TRUE(delegate.created_received()); |
| EXPECT_FALSE(base::SharedMemory::IsHandleValid(delegate.handle())); |
| EXPECT_EQ(kLength, delegate.length()); |