Fix two races in DirectoryWatcherInotify:

1. In DirectoryWatcherImpl::Watch we have to initialize delegate_, root_path_
   and other members before calling InotifyReader::AddWatch, because otherwise
   the following race is possible:

   1. InotifyReader::AddWatch
   2. DirectoryWatcherImpl::OnInotifyEvent, where we want to use delegate_
      and root_path_
   3. Now-late initialization of delegate_ and root_path_ in DirectoryWatcherImpl::Watch

2. In InotifyReader::OnInotifyEvent we can't just copy things under a lock
   and notify without a lock. Otherwise the following race is possible:

   1. Things get copied in InotifyReader::OnInotifyEvent, lock is released
   2. DirectoryWatcherImpl is destroyed, removing its watch
   3. But it's still in the copied set of watchers to be notified. When we get
      to notifying it, it's invalid, and we have a crash.

To fix race #2, I decided to just do everything under the lock. Notifications
shouldn't be a bottleneck, and I don't want to prematurely add more complexity.

TEST=Manually with RaceChecker, see bug.
http://crbug.com/15473

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20730 0039d316-1c4b-4281-b951-d872f2087c98

0 files changed, 0 insertions, 0 deletions