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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: base/memory/shared_memory_handle.h

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
« no previous file with comments | « base/memory/shared_memory.h ('k') | base/memory/shared_memory_mac.cc » ('j') | chrome/renderer/pepper/pepper_shared_memory_message_filter.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: base/memory/shared_memory_handle.h
diff --git a/base/memory/shared_memory_handle.h b/base/memory/shared_memory_handle.h
index 228e8216c9f39354b8f978361b8a28722e32faa2..b293054ee86c5e00029430a50edc61429c82a2b2 100644
--- a/base/memory/shared_memory_handle.h
+++ b/base/memory/shared_memory_handle.h
@@ -56,7 +56,9 @@ class BASE_EXPORT SharedMemoryHandle {
// Whether the underlying OS resource is valid.
bool IsValid() const;
- // Duplicates the underlying OS resource.
+ // Duplicates the underlying OS resource. Using the return value as a
+ // parameter to an IPC message will cause the IPC subsystem to consume the OS
Nico 2017/05/02 20:08:00 Feels like a bit strange place to document this (I
+ // resource.
SharedMemoryHandle Duplicate() const;
#if defined(OS_MACOSX) && !defined(OS_IOS)
« no previous file with comments | « base/memory/shared_memory.h ('k') | base/memory/shared_memory_mac.cc » ('j') | chrome/renderer/pepper/pepper_shared_memory_message_filter.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698