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

Unified Diff: components/nacl/browser/nacl_browser.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
« no previous file with comments | « no previous file | components/nacl/browser/nacl_file_host.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/nacl/browser/nacl_browser.cc
diff --git a/components/nacl/browser/nacl_browser.cc b/components/nacl/browser/nacl_browser.cc
index 541a080a0031a8ee4fe15dce0fbb781e45a1aeb2..fbf7de91fba87b87d47975e278e4c185ce8c877d 100644
--- a/components/nacl/browser/nacl_browser.cc
+++ b/components/nacl/browser/nacl_browser.cc
@@ -130,7 +130,7 @@ base::File OpenNaClReadExecImpl(const base::FilePath& file_path,
flags |= base::File::FLAG_EXECUTE; // Windows only flag.
base::File file(file_path, flags);
if (!file.IsValid())
- return file.Pass();
+ return file;
// Check that the file does not reference a directory. Returning a descriptor
// to an extension directory could allow an outer sandbox escape. openat(...)
@@ -139,7 +139,7 @@ base::File OpenNaClReadExecImpl(const base::FilePath& file_path,
if (!file.GetInfo(&file_info) || file_info.is_directory)
return base::File();
- return file.Pass();
+ return file;
}
NaClBrowser::NaClBrowser()
« no previous file with comments | « no previous file | components/nacl/browser/nacl_file_host.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698