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

Unified Diff: components/nacl/browser/pnacl_host_unittest.cc

Issue 2514323004: Convert NaCl renderer-browser messages to mojo. (Closed)
Patch Set: Created 3 years, 10 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
« no previous file with comments | « components/nacl/browser/pnacl_host.cc ('k') | components/nacl/common/BUILD.gn » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/nacl/browser/pnacl_host_unittest.cc
diff --git a/components/nacl/browser/pnacl_host_unittest.cc b/components/nacl/browser/pnacl_host_unittest.cc
index 992f607c27305beeac66d978358eac12a308b8e3..5d08380e0078c69c5cf65d608c979caf2405e7eb 100644
--- a/components/nacl/browser/pnacl_host_unittest.cc
+++ b/components/nacl/browser/pnacl_host_unittest.cc
@@ -69,7 +69,7 @@ class PnaclHostTest : public testing::Test {
// CallbackExpectMiss checks that the fd is valid and a miss is reported,
// and also writes some data into the file, which is read back by
// CallbackExpectHit
- void CallbackExpectMiss(const base::File& file, bool is_hit) {
+ void CallbackExpectMiss(base::File file, bool is_hit) {
EXPECT_FALSE(is_hit);
ASSERT_TRUE(file.IsValid());
base::File::Info info;
@@ -84,7 +84,7 @@ class PnaclHostTest : public testing::Test {
static_cast<size_t>(mutable_file->Write(0, str, kBufferSize)));
temp_callback_count_++;
}
- void CallbackExpectHit(const base::File& file, bool is_hit) {
+ void CallbackExpectHit(base::File file, bool is_hit) {
EXPECT_TRUE(is_hit);
ASSERT_TRUE(file.IsValid());
base::File::Info info;
« no previous file with comments | « components/nacl/browser/pnacl_host.cc ('k') | components/nacl/common/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698