win: implement Semaphore

win: implement Semaphore

Ref:
http://msdn.microsoft.com/en-us/library/windows/desktop/ms686946%28v=vs.85%29.aspx

R=mark@chromium.org
BUG=crashpad:1
Committed: https://chromium.googlesource.com/crashpad/crashpad/+/d1fdcf99e0026b8b87b468d6b778d1c796eaaf8a

[scottmg, 2014-12-17 22:58:15]

https://codereview.chromium.org/797173003/diff/1/util/synchronization/semapho...
File util/synchronization/semaphore.cc (right):

https://codereview.chromium.org/797173003/diff/1/util/synchronization/semapho...
util/synchronization/semaphore.cc:20: #include "base/posix/eintr_wrapper.h"
I didn't realize before, but this guards its entire contents, so is OK to
include.

[Mark Mentovai, 2014-12-18 14:38:47]

LGTM.

https://codereview.chromium.org/797173003/diff/1/util/synchronization/semapho...
File util/synchronization/semaphore.cc (right):

https://codereview.chromium.org/797173003/diff/1/util/synchronization/semapho...
util/synchronization/semaphore.cc:58: CHECK_EQ(WaitForSingleObject(semaphore_,
INFINITE), WAIT_OBJECT_0);
PCHECK this and ReleaseSemaphore()?

[scottmg, 2014-12-18 17:07:03]

https://codereview.chromium.org/797173003/diff/1/util/synchronization/semapho...
File util/synchronization/semaphore.cc (right):

https://codereview.chromium.org/797173003/diff/1/util/synchronization/semapho...
util/synchronization/semaphore.cc:58: CHECK_EQ(WaitForSingleObject(semaphore_,
INFINITE), WAIT_OBJECT_0);
On 2014/12/18 14:38:47, Mark Mentovai wrote:
> PCHECK this and ReleaseSemaphore()?

Done.

[scottmg, 2014-12-18 17:08:05]

Patchset #2 (id:20001) has been deleted

[scottmg, 2014-12-18 17:09:03]

Committed patchset #2 (id:40001) manually as
d1fdcf99e0026b8b87b468d6b778d1c796eaaf8a (presubmit successful).