Comment #9 on issue 8809 by [email protected]: Crash in  
WaitableEventWatcher::StopWatching()
http://code.google.com/p/chromium/issues/detail?id=8809

The following revision refers to this bug:
     http://src.chromium.org/viewvc/chrome?view=rev&revision=12459

------------------------------------------------------------------------
r12459 | [email protected] | 2009-03-25 10:08:41 -0700 (Wed, 25 Mar 2009) |  
16 lines
Changed paths:
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/base/waitable_event.h?r1=12459&r2=12458
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/base/waitable_event_posix.cc?r1=12459&r2=12458
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/base/waitable_event_watcher.h?r1=12459&r2=12458
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/base/waitable_event_watcher_posix.cc?r1=12459&r2=12458
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/base/waitable_event_watcher_unittest.cc?r1=12459&r2=12458

POSIX: allow WaitableEvents to be deleted while watching them.

On Windows, one can close a HANDLE which is currently being waited on. The  
MSDN
documentation says that the resulting behaviour is 'undefined', but it  
doesn't
crash. Currently, on POSIX, one couldn't use WaitableEventWatcher to watch  
an
event which gets deleted. This mismatch has bitten us several times now.

This patch allows WaitableEvents to be deleted while a WaitableEventWatcher  
is
still watching them. It applies only to watchers, the usual Wait() and
WaitMany() calls still require that all their target be valid until the end  
of
the call.

http://crbug.com/8809

Review URL: http://codereview.chromium.org/53026

------------------------------------------------------------------------


--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

--~--~---------~--~----~------------~-------~--~----~
Automated mail from issue updates at http://crbug.com/
Subscription options: http://groups.google.com/group/chromium-bugs
-~----------~----~----~----~------~----~------~--~---

Reply via email to