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

Unified Diff: base/memory/generic_shared_memory_id.h

Issue 1280513002: Add GenericSharedMemoryId and use w/ GpuMemoryBuffer (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@trackpools
Patch Set: Comments and tweaks. Created 5 years, 4 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: base/memory/generic_shared_memory_id.h
diff --git a/base/memory/generic_shared_memory_id.h b/base/memory/generic_shared_memory_id.h
new file mode 100644
index 0000000000000000000000000000000000000000..a55c71098fdbfb696be930b50d7fd9ff496776be
--- /dev/null
+++ b/base/memory/generic_shared_memory_id.h
@@ -0,0 +1,31 @@
+// Copyright 2015 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef BASE_MEMORY_GENERIC_SHARED_MEMORY_ID_H_
+#define BASE_MEMORY_GENERIC_SHARED_MEMORY_ID_H_
+
+#include "base/base_export.h"
+#include "base/trace_event/memory_allocator_dump.h"
+
+namespace base {
+
+namespace memory {
+
+// Defines an ID type which is used accross all types of shared memory
+// allocations. This includes other types (IOSurfaces, SharedBitmaps), not
+// just base::SharedMemory.
+using GenericSharedMemoryId = int;
Tom Sepez 2015/08/06 00:11:08 do we want to use a specific-sized unsigned type l
reveman 2015/08/06 04:25:06 +1
ericrk 2015/08/06 18:28:08 The underlying code uses base::StaticAtomicSequenc
reveman 2015/08/06 19:19:36 I think it's fine to keep this as 'int' then unles
+
+// Generates GUID which can be used to trace shared memory using its
+// GenericSharedMemoryId.
+BASE_EXPORT base::trace_event::MemoryAllocatorDumpGuid
+GetGenericSharedMemoryGUIDForTracing(
+ uint64_t tracing_process_id,
+ GenericSharedMemoryId generic_shared_memory_id);
+
+} // namespace memory
+
+} // namespace base
+
+#endif // BASE_MEMORY_GENERIC_SHARED_MEMORY_ID_H_

Powered by Google App Engine
This is Rietveld 408576698