diff options
author | amistry <amistry@chromium.org> | 2016-03-17 13:18:00 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2016-03-17 20:18:55 +0000 |
commit | 7fffb56f8566329795e8fc744198b119c24460ab (patch) | |
tree | 09b2722f2d5a989d1e9d1f4f2321275dfb52fda8 /mojo | |
parent | 9d74e771fce6cace39eb0ad7243036b857af483e (diff) | |
download | chromium_src-7fffb56f8566329795e8fc744198b119c24460ab.zip chromium_src-7fffb56f8566329795e8fc744198b119c24460ab.tar.gz chromium_src-7fffb56f8566329795e8fc744198b119c24460ab.tar.bz2 |
[mojo-edk] Switch shared buffers to use Mach on OSX.
BUG=582468
Review URL: https://codereview.chromium.org/1804353002
Cr-Commit-Position: refs/heads/master@{#381785}
Diffstat (limited to 'mojo')
-rw-r--r-- | mojo/edk/embedder/platform_shared_buffer.cc | 3 | ||||
-rw-r--r-- | mojo/edk/system/shared_buffer_unittest.cc | 4 |
2 files changed, 4 insertions, 3 deletions
diff --git a/mojo/edk/embedder/platform_shared_buffer.cc b/mojo/edk/embedder/platform_shared_buffer.cc index f3f9b70..614b19c 100644 --- a/mojo/edk/embedder/platform_shared_buffer.cc +++ b/mojo/edk/embedder/platform_shared_buffer.cc @@ -175,8 +175,7 @@ bool PlatformSharedBuffer::Init() { base::SharedMemoryCreateOptions options; options.size = num_bytes_; #if defined(OS_MACOSX) && !defined(OS_IOS) - // TODO(crbug.com/582468): Support Mach shared memory. - options.type = base::SharedMemoryHandle::POSIX; + options.type = base::SharedMemoryHandle::MACH; #endif shared_memory_.reset(new base::SharedMemory); diff --git a/mojo/edk/system/shared_buffer_unittest.cc b/mojo/edk/system/shared_buffer_unittest.cc index c699d9d..210c2f5 100644 --- a/mojo/edk/system/shared_buffer_unittest.cc +++ b/mojo/edk/system/shared_buffer_unittest.cc @@ -216,8 +216,10 @@ DEFINE_TEST_CLIENT_TEST_WITH_PIPE(ReceiveAndEditBufferParent, SharedBufferTest, END_CHILD() } -#if defined(OS_ANDROID) +#if defined(OS_ANDROID) || defined(OS_MACOSX) // Android multi-process tests are not executing the new process. This is flaky. +// Passing shared memory handles between cousins is not currently supported on +// OSX. #define MAYBE_PassHandleBetweenCousins DISABLED_PassHandleBetweenCousins #else #define MAYBE_PassHandleBetweenCousins PassHandleBetweenCousins |