Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(428)

Side by Side Diff: chrome/browser/file_path_watcher/file_path_watcher.cc

Issue 6697020: Fixed shutdown concurrency issues in FilePathWatcher. (Closed) Base URL: http://git.chromium.org/git/chromium.git@trunk
Patch Set: Created 9 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2010 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2010 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 // Cross platform methods for FilePathWatcher. See the various platform 5 // Cross platform methods for FilePathWatcher. See the various platform
6 // specific implementation files, too. 6 // specific implementation files, too.
7 7
8 #include "chrome/browser/file_path_watcher/file_path_watcher.h" 8 #include "chrome/browser/file_path_watcher/file_path_watcher.h"
9 9
10 #include "content/browser/browser_thread.h"
11
10 FilePathWatcher::~FilePathWatcher() { 12 FilePathWatcher::~FilePathWatcher() {
11 impl_->Cancel(); 13 impl_->Cancel();
12 } 14 }
13 15
14 bool FilePathWatcher::Watch(const FilePath& path, 16 bool FilePathWatcher::Watch(const FilePath& path,
15 Delegate* delegate) { 17 Delegate* delegate) {
16 DCHECK(BrowserThread::CurrentlyOn(BrowserThread::FILE)); 18 DCHECK(BrowserThread::CurrentlyOn(BrowserThread::FILE));
17 DCHECK(path.IsAbsolute()); 19 DCHECK(path.IsAbsolute());
18 return impl_->Watch(path, delegate); 20 return impl_->Watch(path, delegate);
19 } 21 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698