diff options
| -rw-r--r-- | chrome/browser/renderer_host/audio_sync_reader.cc | 14 |
1 files changed, 4 insertions, 10 deletions
diff --git a/chrome/browser/renderer_host/audio_sync_reader.cc b/chrome/browser/renderer_host/audio_sync_reader.cc index 1cccfbe..48787e7 100644 --- a/chrome/browser/renderer_host/audio_sync_reader.cc +++ b/chrome/browser/renderer_host/audio_sync_reader.cc @@ -47,11 +47,8 @@ bool AudioSyncReader::PrepareForeignSocketHandle( ::DuplicateHandle(GetCurrentProcess(), foreign_socket_->handle(), process_handle, foreign_handle, 0, FALSE, DUPLICATE_SAME_ACCESS); - if (*foreign_handle != 0) { - // Note that this is just a way to get away with the unused variable - // warning. We want to return true here. - return foreign_socket_.release() != NULL; - } + if (*foreign_handle != 0) + return true; return false; } #else @@ -60,11 +57,8 @@ bool AudioSyncReader::PrepareForeignSocketHandle( base::FileDescriptor* foreign_handle) { foreign_handle->fd = foreign_socket_->handle(); foreign_handle->auto_close = false; - if (foreign_handle->fd != -1) { - // Note that this is just a way to get away with the unused variable - // warning. We want to return true here. - return foreign_socket_.release() != NULL; - } + if (foreign_handle->fd != -1) + return true; return false; } #endif |