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

Unified Diff: components/nacl/renderer/file_downloader.cc

Issue 1548203002: Convert Pass()→std::move() in //components/[n-z]* (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix bad headers Created 5 years 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: components/nacl/renderer/file_downloader.cc
diff --git a/components/nacl/renderer/file_downloader.cc b/components/nacl/renderer/file_downloader.cc
index b2771f25db4e276a1d4f0b2c14a7fedee80e76db..fa94d846ed2a9637d4d0eea26c40df4abd155141 100644
--- a/components/nacl/renderer/file_downloader.cc
+++ b/components/nacl/renderer/file_downloader.cc
@@ -4,6 +4,8 @@
#include "components/nacl/renderer/file_downloader.h"
+#include <utility>
+
#include "base/callback.h"
#include "components/nacl/renderer/nexe_load_manager.h"
#include "net/base/net_errors.h"
@@ -17,8 +19,8 @@ FileDownloader::FileDownloader(scoped_ptr<blink::WebURLLoader> url_loader,
base::File file,
StatusCallback status_cb,
ProgressCallback progress_cb)
- : url_loader_(url_loader.Pass()),
- file_(file.Pass()),
+ : url_loader_(std::move(url_loader)),
+ file_(std::move(file)),
status_cb_(status_cb),
progress_cb_(progress_cb),
http_status_code_(-1),
@@ -74,7 +76,7 @@ void FileDownloader::didFinishLoading(
if (file_.Seek(base::File::FROM_BEGIN, 0) != 0)
status_ = FAILED;
}
- status_cb_.Run(status_, file_.Pass(), http_status_code_);
+ status_cb_.Run(status_, std::move(file_), http_status_code_);
delete this;
}
@@ -98,7 +100,7 @@ void FileDownloader::didFail(
// some implementations of blink::WebURLLoader will do after calling didFail.
url_loader_.reset();
- status_cb_.Run(status_, file_.Pass(), http_status_code_);
+ status_cb_.Run(status_, std::move(file_), http_status_code_);
delete this;
}
« no previous file with comments | « components/nacl/loader/sandbox_linux/nacl_sandbox_linux.cc ('k') | components/nacl/renderer/manifest_downloader.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698