win: make CrashpadInfo retrievable

util/win/address_types.h

 // 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.
 
-#include "test/thread.h"
+#ifndef CRASHPAD_UTIL_WIN_ADDRESS_TYPES_H_
+#define CRASHPAD_UTIL_WIN_ADDRESS_TYPES_H_
 
-#include "gtest/gtest.h"
+#include <stdint.h>
 
 namespace crashpad {
-namespace test {
 
-Thread::Thread() : platform_thread_(0) {
-}
+using win_vm_address_t = uint64_t;

[Mark Mentovai, 2015/05/01 19:05:31]

Provide a quick //! \brief for each of these saying what it’s intended for, and
perhaps they “why” also.

[Mark Mentovai, 2015/05/01 19:05:31]

I initially wrote a comment here suggesting WinVMAddress and WinVMSize to match
Google style for type names, but then I saw your last comment about considering
and rejecting those, so I came back and fixed this comment to acknowledge. OK,
if you want to keep using these names, it’s fine, and you can read any other
comments I’ve written with the cappy names as though they had the underscorey
ones instead.

[scottmg, 2015/05/01 19:46:32]

Done.

[scottmg, 2015/05/01 19:46:32]

Switched to WinVMAddress/Size.

+using win_vm_size_t = uint64_t;
 
-Thread::~Thread() {
-  EXPECT_FALSE(platform_thread_);
-}
+}  // namespace crashpad
 
-}  // namespace test
-}  // namespace crashpad
+#endif  // CRASHPAD_UTIL_WIN_ADDRESS_TYPES_H_