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

Unified Diff: third_party/mojo/src/mojo/public/go/system/shared_buffer.go

Issue 890843003: Revert of Update mojo sdk to rev 8af2ccff2eee4bfca1043015abee30482a030b30 (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 10 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: third_party/mojo/src/mojo/public/go/system/shared_buffer.go
diff --git a/third_party/mojo/src/mojo/public/go/system/shared_buffer.go b/third_party/mojo/src/mojo/public/go/system/shared_buffer.go
index 927c7ea3d6433a2ba3a79e2520437822d196fe63..a8bdbfa380529b8171a7a18655e6b0f78e067db3 100644
--- a/third_party/mojo/src/mojo/public/go/system/shared_buffer.go
+++ b/third_party/mojo/src/mojo/public/go/system/shared_buffer.go
@@ -31,9 +31,6 @@
}
func (h *sharedBuffer) DuplicateBufferHandle(opts *DuplicateBufferHandleOptions) (MojoResult, SharedBufferHandle) {
- h.core.mu.Lock()
- defer h.core.mu.Unlock()
-
cParams := C.MallocDuplicateBufferHandleParams()
defer C.FreeDuplicateBufferHandleParams(cParams)
result := C.MojoDuplicateBufferHandle(h.mojoHandle.cValue(), opts.cValue(cParams.opts), cParams.duplicate)
@@ -41,9 +38,6 @@
}
func (h *sharedBuffer) MapBuffer(offset uint64, numBytes int, flags MojoMapBufferFlags) (MojoResult, []byte) {
- h.core.mu.Lock()
- defer h.core.mu.Unlock()
-
cParams := C.MallocMapBufferParams()
defer C.FreeMapBufferParams(cParams)
result := C.MojoMapBuffer(h.mojoHandle.cValue(), C.uint64_t(offset), C.uint64_t(numBytes), cParams.buffer, flags.cValue())
@@ -54,8 +48,5 @@
}
func (h *sharedBuffer) UnmapBuffer(buffer []byte) MojoResult {
- h.core.mu.Lock()
- defer h.core.mu.Unlock()
-
return MojoResult(C.MojoUnmapBuffer(unsafe.Pointer(&buffer[0])))
}

Powered by Google App Engine
This is Rietveld 408576698