Modify WaitableEvent::Wait() to return void

base/synchronization/waitable_event_win.cc

 // Copyright (c) 2011 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.
 
 #include "base/synchronization/waitable_event.h"
 
 #include <math.h>
 #include <windows.h>
 
 #include "base/logging.h"
@@ skipping 28 matching lines @@
 }
 
 void WaitableEvent::Signal() {
   SetEvent(handle_);
 }
 
 bool WaitableEvent::IsSignaled() {
   return TimedWait(TimeDelta::FromMilliseconds(0));
 }
 
-bool WaitableEvent::Wait() {
+void WaitableEvent::Wait() {
   DWORD result = WaitForSingleObject(handle_, INFINITE);
   // It is most unexpected that this should ever fail.  Help consumers learn
   // about it if it should ever fail.
   DCHECK_EQ(WAIT_OBJECT_0, result) << "WaitForSingleObject failed";
-  return result == WAIT_OBJECT_0;
 }
 
 bool WaitableEvent::TimedWait(const TimeDelta& max_time) {
   DCHECK(max_time >= TimeDelta::FromMicroseconds(0));
   // Be careful here.  TimeDelta has a precision of microseconds, but this API
   // is in milliseconds.  If there are 5.5ms left, should the delay be 5 or 6?
   // It should be 6 to avoid returning too early.
   double timeout = ceil(max_time.InMillisecondsF());
   DWORD result = WaitForSingleObject(handle_, static_cast<DWORD>(timeout));
   switch (result) {
@@ skipping 25 matching lines @@
                              INFINITE);  // no timeout
   if (result >= WAIT_OBJECT_0 + count) {
     NOTREACHED() << "WaitForMultipleObjects failed: " << GetLastError();
     return 0;
   }
 
   return result - WAIT_OBJECT_0;
 }
 
 }  // namespace base