The send done event in the old send watcher can be NULL as it can be signaled...

The send done event in the old send watcher can be NULL as it can be signaled in a nested sync channel
send scenario which yields control to the plugin before the quit task is processed. This is perfectly legit
and we should not attempt to StopWatching the send done watcher as it crashes.

I will work on a unit test for this in a subsequent CL.

TBR=jam


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=29000

[jam, 2009-10-14 20:04:02]

I don't understand what the problem is?  The cl description mentions
StopWatching but the code is changing when StartWatching is called?