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

Unified Diff: mojo/public/platform/native/platform_handle_private.h

Issue 2250183003: Make the fuchsia mojo/public repo the source of truth. (Closed) Base URL: https://github.com/domokit/mojo.git@master
Patch Set: Created 4 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: mojo/public/platform/native/platform_handle_private.h
diff --git a/mojo/public/platform/native/platform_handle_private.h b/mojo/public/platform/native/platform_handle_private.h
deleted file mode 100644
index f3397b62d5dbe9dc9a1f0582775f7e4fd5960410..0000000000000000000000000000000000000000
--- a/mojo/public/platform/native/platform_handle_private.h
+++ /dev/null
@@ -1,72 +0,0 @@
-// Copyright 2016 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 MOJO_PUBLIC_PLATFORM_NATIVE_PLATFORM_HANDLE_PRIVATE_H_
-#define MOJO_PUBLIC_PLATFORM_NATIVE_PLATFORM_HANDLE_PRIVATE_H_
-
-#include <mojo/result.h>
-#include <mojo/system/handle.h>
-
-// |MojoPlatformHandle|: Type for "platform handles", i.e., the underlying OS's
-// handles. Currently this is always just a Unix file descriptor.
-
-typedef int MojoPlatformHandle;
-
-#ifdef __cplusplus
-extern "C" {
-#endif
-
-// |MojoCreatePlatformHandleWrapper()|: Creates a |MojoHandle| that wraps (and
-// takes ownership of) the platform handle |platform_handle|, which must be
-// valid.
-//
-// On success, |*platform_handle_wrapper_handle| will be set to the wrapper
-// handle. It will have (at least) the |MOJO_HANDLE_RIGHT_TRANSFER|,
-// |MOJO_HANDLE_RIGHT_READ|, and |MOJO_HANDLE_RIGHT_WRITE| rights. Warning: No
-// validation of |platform_handle| is done. (TODO(vtl): This has poor/annoying
-// implications, since we may detect this when we transfer the wrapper handle.)
-//
-// Warning: On failure, this will still take ownership of |platform_handle|
-// (which just means that |platform_handle| will be closed).
-//
-// Returns:
-// |MOJO_RESULT_OK| on success.
-// |MOJO_RESULT_RESOURCE_EXHAUSTED| if a process/system/quota/etc. limit has
-// been reached (e.g., if the maximum number of handles was exceeded).
-MojoResult MojoCreatePlatformHandleWrapper(
- MojoPlatformHandle platform_handle,
- MojoHandle* platform_handle_wrapper_handle);
-
-// |MojoExtractPlatformHandle()|: Extracts the wrapped platform handle from
-// |platform_handle_wrapper_handle| (which must have both the
-// |MOJO_HANDLE_RIGHT_READ| and |MOJO_HANDLE_RIGHT_WRITE| rights).
-//
-// On success, |*platform_handle| will be set to the wrapped platform handle and
-// ownership of the wrapped platform handle will be passed to the caller (i.e.,
-// closing |platform_handle_wrapper_handle| will no longer close the platform
-// handle).
-//
-// Warnings:
-// - Even though |platform_handle_wrapper_handle| is then basically useless
-// (it no longer "contains" a platform handle), it must still be closed as
-// usual.
-// - If the wrapped platform handle has already been extracted from
-// |platform_handle_wrapper_handle|, then this will still succeed, but
-// |*platform_handle| will be set to -1.
-//
-// Returns:
-// |MOJO_RESULT_OK| on success.
-// |MOJO_RESULT_INVALID_ARGUMENT| if some argument was invalid (e.g.,
-// |platform_handle_wrapper_handle| is not a valid wrapper handle).
-// |MOJO_RESULT_PERMISSION_DENIED| if |platform_handle_wrapper_handle| does
-// not have the both the |MOJO_HANDLE_RIGHT_READ| and
-// |MOJO_HANDLE_RIGHT_WRITE| rights.
-MojoResult MojoExtractPlatformHandle(MojoHandle platform_handle_wrapper_handle,
- MojoPlatformHandle* platform_handle);
-
-#ifdef __cplusplus
-} // extern "C"
-#endif
-
-#endif // MOJO_PUBLIC_PLATFORM_NATIVE_PLATFORM_HANDLE_PRIVATE_H_
« no previous file with comments | « mojo/public/platform/native/mgl_thunks.c ('k') | mojo/public/platform/native/platform_handle_private_apptest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698