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

Unified Diff: mojo/edk/embedder/platform_handle.h

Issue 1352913002: Remove Windows support from the EDK. (Closed) Base URL: https://github.com/domokit/mojo.git@master
Patch Set: Created 5 years, 3 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 | « mojo/edk/embedder/platform_channel_pair_win.cc ('k') | mojo/edk/embedder/platform_handle.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: mojo/edk/embedder/platform_handle.h
diff --git a/mojo/edk/embedder/platform_handle.h b/mojo/edk/embedder/platform_handle.h
index 743d02cc49ad167904b8f9663443cf3a513700c0..658d27c6d162383369e17dee74dc87afcdaa4bea 100644
--- a/mojo/edk/embedder/platform_handle.h
+++ b/mojo/edk/embedder/platform_handle.h
@@ -5,18 +5,12 @@
#ifndef MOJO_EDK_EMBEDDER_PLATFORM_HANDLE_H_
#define MOJO_EDK_EMBEDDER_PLATFORM_HANDLE_H_
-#include "build/build_config.h"
#include "mojo/edk/system/system_impl_export.h"
-#if defined(OS_WIN)
-#include <windows.h>
-#endif
-
namespace mojo {
namespace embedder {
-// A |PlatformHandle| is a file descriptor on POSIX and a |HANDLE| on Windows.
-#if defined(OS_POSIX)
+// A |PlatformHandle| is just a file descriptor on POSIX.
struct MOJO_SYSTEM_IMPL_EXPORT PlatformHandle {
PlatformHandle() : fd(-1) {}
explicit PlatformHandle(int fd) : fd(fd) {}
@@ -27,20 +21,6 @@ struct MOJO_SYSTEM_IMPL_EXPORT PlatformHandle {
int fd;
};
-#elif defined(OS_WIN)
-struct MOJO_SYSTEM_IMPL_EXPORT PlatformHandle {
- PlatformHandle() : handle(INVALID_HANDLE_VALUE) {}
- explicit PlatformHandle(HANDLE handle) : handle(handle) {}
-
- void CloseIfNecessary();
-
- bool is_valid() const { return handle != INVALID_HANDLE_VALUE; }
-
- HANDLE handle;
-};
-#else
-#error "Platform not yet supported."
-#endif
} // namespace embedder
} // namespace mojo
« no previous file with comments | « mojo/edk/embedder/platform_channel_pair_win.cc ('k') | mojo/edk/embedder/platform_handle.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698