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

Side by Side Diff: content/renderer/pepper/pepper_platform_audio_input.cc

Issue 1320783002: Make SharedMemoryHandle a class on windows. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@ipc_global
Patch Set: Rebase. Created 5 years, 2 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "content/renderer/pepper/pepper_platform_audio_input.h" 5 #include "content/renderer/pepper/pepper_platform_audio_input.h"
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "base/location.h" 8 #include "base/location.h"
9 #include "base/logging.h" 9 #include "base/logging.h"
10 #include "base/single_thread_task_runner.h" 10 #include "base/single_thread_task_runner.h"
(...skipping 66 matching lines...) Expand 10 before | Expand all | Expand 10 after
77 FROM_HERE, 77 FROM_HERE,
78 base::Bind(&PepperPlatformAudioInput::ShutDownOnIOThread, this)); 78 base::Bind(&PepperPlatformAudioInput::ShutDownOnIOThread, this));
79 } 79 }
80 80
81 void PepperPlatformAudioInput::OnStreamCreated( 81 void PepperPlatformAudioInput::OnStreamCreated(
82 base::SharedMemoryHandle handle, 82 base::SharedMemoryHandle handle,
83 base::SyncSocket::Handle socket_handle, 83 base::SyncSocket::Handle socket_handle,
84 int length, 84 int length,
85 int total_segments) { 85 int total_segments) {
86 #if defined(OS_WIN) 86 #if defined(OS_WIN)
87 DCHECK(handle); 87 DCHECK(handle.IsValid());
88 DCHECK(socket_handle); 88 DCHECK(socket_handle);
89 #else 89 #else
90 DCHECK(base::SharedMemory::IsHandleValid(handle)); 90 DCHECK(base::SharedMemory::IsHandleValid(handle));
91 DCHECK_NE(-1, socket_handle); 91 DCHECK_NE(-1, socket_handle);
92 #endif 92 #endif
93 DCHECK(length); 93 DCHECK(length);
94 // TODO(yzshen): Make use of circular buffer scheme. crbug.com/181449. 94 // TODO(yzshen): Make use of circular buffer scheme. crbug.com/181449.
95 DCHECK_EQ(1, total_segments); 95 DCHECK_EQ(1, total_segments);
96 96
97 if (base::ThreadTaskRunnerHandle::Get().get() != main_task_runner_.get()) { 97 if (base::ThreadTaskRunnerHandle::Get().get() != main_task_runner_.get()) {
(...skipping 182 matching lines...) Expand 10 before | Expand all | Expand 10 after
280 PepperMediaDeviceManager* PepperPlatformAudioInput::GetMediaDeviceManager() { 280 PepperMediaDeviceManager* PepperPlatformAudioInput::GetMediaDeviceManager() {
281 DCHECK(main_task_runner_->BelongsToCurrentThread()); 281 DCHECK(main_task_runner_->BelongsToCurrentThread());
282 282
283 RenderFrameImpl* const render_frame = 283 RenderFrameImpl* const render_frame =
284 RenderFrameImpl::FromRoutingID(render_frame_id_); 284 RenderFrameImpl::FromRoutingID(render_frame_id_);
285 return render_frame ? 285 return render_frame ?
286 PepperMediaDeviceManager::GetForRenderFrame(render_frame).get() : NULL; 286 PepperMediaDeviceManager::GetForRenderFrame(render_frame).get() : NULL;
287 } 287 }
288 288
289 } // namespace content 289 } // namespace content
OLDNEW
« no previous file with comments | « content/renderer/npapi/webplugin_delegate_proxy.cc ('k') | content/renderer/pepper/pepper_platform_audio_output.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698