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

Unified Diff: webkit/common/blob/scoped_file.cc

Issue 515093002: FileAPI/sync file system cleanups for scoped_refptr operator T* cleanup. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 4 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 side-by-side diff with in-line comments
Download patch
Index: webkit/common/blob/scoped_file.cc
diff --git a/webkit/common/blob/scoped_file.cc b/webkit/common/blob/scoped_file.cc
index 12edd6e412d13bfbea4f6a5e83d6b6a8dc80fb85..19e68e19d22a53a175bbe5fbff49f8cf6b46246e 100644
--- a/webkit/common/blob/scoped_file.cc
+++ b/webkit/common/blob/scoped_file.cc
@@ -17,15 +17,15 @@ ScopedFile::ScopedFile()
: scope_out_policy_(DONT_DELETE_ON_SCOPE_OUT) {
}
-ScopedFile::ScopedFile(
- const base::FilePath& path, ScopeOutPolicy policy,
- base::TaskRunner* file_task_runner)
+ScopedFile::ScopedFile(const base::FilePath& path,
+ ScopeOutPolicy policy,
+ const scoped_refptr<base::TaskRunner>& file_task_runner)
: path_(path),
scope_out_policy_(policy),
file_task_runner_(file_task_runner) {
- DCHECK(path.empty() || policy != DELETE_ON_SCOPE_OUT || file_task_runner)
- << "path:" << path.value()
- << " policy:" << policy
+ DCHECK(path.empty() || policy != DELETE_ON_SCOPE_OUT ||
+ file_task_runner.get())
+ << "path:" << path.value() << " policy:" << policy
<< " runner:" << file_task_runner;
}

Powered by Google App Engine
This is Rietveld 408576698