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

Unified Diff: device/gamepad/gamepad_provider.cc

Issue 2852803002: Remove base::SharedMemory::ShareToProcess. (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: device/gamepad/gamepad_provider.cc
diff --git a/device/gamepad/gamepad_provider.cc b/device/gamepad/gamepad_provider.cc
index a560aebc9328820560b5bbb789ca28069490475c..6e1a30798770fabf50825026a9e428d060136475 100644
--- a/device/gamepad/gamepad_provider.cc
+++ b/device/gamepad/gamepad_provider.cc
@@ -84,10 +84,7 @@ GamepadProvider::~GamepadProvider() {
base::SharedMemoryHandle GamepadProvider::GetSharedMemoryHandleForProcess(
Nico 2017/05/02 16:06:12 This whole function can probably go away now?
erikchen 2017/05/02 19:00:51 The function is still needed to access the private
base::ProcessHandle process) {
- base::SharedMemoryHandle renderer_handle;
- gamepad_shared_buffer_->shared_memory()->ShareToProcess(process,
- &renderer_handle);
- return renderer_handle;
+ return gamepad_shared_buffer_->shared_memory()->handle().Duplicate();
}
mojo::ScopedSharedBufferHandle GamepadProvider::GetSharedBufferHandle() {

Powered by Google App Engine
This is Rietveld 408576698