Introduce RegistrationServer.

client/registration_protocol_win.h

Index: client/registration_protocol_win.h
diff --git a/client/registration_protocol_win.h b/client/registration_protocol_win.h
new file mode 100644
index 0000000000000000000000000000000000000000..54ad1f27a3207c0af1c06d41172bb4a81e57517b
--- /dev/null
+++ b/client/registration_protocol_win.h
@@ -0,0 +1,55 @@
+// Copyright 2015 The Crashpad Authors. All rights reserved.
+//
+// Licensed under the Apache License, Version 2.0 (the "License");
+// you may not use this file except in compliance with the License.
+// You may obtain a copy of the License at
+//
+//     http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing, software
+// distributed under the License is distributed on an "AS IS" BASIS,
+// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+// See the License for the specific language governing permissions and
+// limitations under the License.
+
+#ifndef CRASHPAD_CLIENT_REGISTRATION_PROTOCOL_WIN_H_
+#define CRASHPAD_CLIENT_REGISTRATION_PROTOCOL_WIN_H_
+
+#include <windows.h>

[scottmg, 2015/05/25 18:41:54]

sort

[erikwright (departed), 2015/05/25 19:42:12]

I'm under the impression that it is a convention to place windows.h as the first
header (except for XX.h when included from XX.cc).

I actually performed the following grep to verify that this is the case in
Crashpad:

git grep -n --all-match -e windows.h -e stdint.h

[scottmg, 2015/05/25 19:53:29]

Huh! Fair enough. There are occasional broken headers that require windows
before them so I guess that makes sense.

This is good to go as far as I'm concerned, once you get whatever feedback you'd
like from Carlos.

+#include <stdint.h>
+
+#include "util/win/address_types.h"
+
+namespace crashpad {
+
+#pragma pack(push, 1)
+//! \brief A client registration request.
+struct RegistrationRequest {
+  //! \brief The PID of the client process.
+  DWORD client_process_id;
+  //! \brief The address, in the client process address space, of a CrashpadInfo
+  //!     structure.
+  crashpad::WinVMAddress crashpad_info_address;
+};
+#pragma pack(pop)
+
+#pragma pack(push, 1)
+//! \brief  A client registration response. See
+//!     https://msdn.microsoft.com/en-us/library/windows/desktop/aa384203%28v=vs.85%29.aspx

[scottmg, 2015/05/25 18:41:54]

silly, but you could use 

https://msdn.microsoft.com/en-us/library/windows/desktop/aa384203.aspx

to avoid 80 col.

[erikwright (departed), 2015/05/25 19:42:12]

Done.

+//!     for details on communicating handle values between 32-bit and 64-bit
+//!     processes.
+struct RegistrationResponse {
+  //! \brief An event `HANDLE`, valid in the client process, that should be
+  //!     signaled to request a crash report. 64-bit clients should convert the
+  //!     value to a `HANDLE` using sign-extension.
+  uint32_t request_report_event;
+  //! \brief An event `HANDLE`, valid in the client process, that will be
+  //!     signaled when the requested crash report is complete. 64-bit clients
+  //!     should convert the value to a `HANDLE` using sign-extension.
+  uint32_t report_complete_event;
+};
+#pragma pack(pop)
+
+}  // namespace crashpad
+
+#endif  // CRASHPAD_CLIENT_REGISTRATION_PROTOCOL_WIN_H_