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

Unified Diff: remoting/host/it2me/it2me_native_messaging_host_unittest.cc

Issue 1549493004: Use std::move() instead of .Pass() in remoting/host (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@move_not_pass
Patch Set: 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: remoting/host/it2me/it2me_native_messaging_host_unittest.cc
diff --git a/remoting/host/it2me/it2me_native_messaging_host_unittest.cc b/remoting/host/it2me/it2me_native_messaging_host_unittest.cc
index c36bd6131473e87514db2b246761bbd0bbd890f8..10d02e1ca21f6c91a786f7bccc2ebd3e3f7dc509 100644
--- a/remoting/host/it2me/it2me_native_messaging_host_unittest.cc
+++ b/remoting/host/it2me/it2me_native_messaging_host_unittest.cc
@@ -74,8 +74,8 @@ class MockIt2MeHost : public It2MeHost {
base::WeakPtr<It2MeHost::Observer> observer,
const XmppSignalStrategy::XmppServerConfig& xmpp_server_config,
const std::string& directory_bot_jid)
- : It2MeHost(context.Pass(),
- policy_watcher.Pass(),
+ : It2MeHost(std::move(context),
+ std::move(policy_watcher),
nullptr,
observer,
xmpp_server_config,
@@ -156,7 +156,7 @@ class MockIt2MeHostFactory : public It2MeHostFactory {
base::WeakPtr<It2MeHost::Observer> observer,
const XmppSignalStrategy::XmppServerConfig& xmpp_server_config,
const std::string& directory_bot_jid) override {
- return new MockIt2MeHost(context.Pass(), nullptr, observer,
+ return new MockIt2MeHost(std::move(context), nullptr, observer,
xmpp_server_config, directory_bot_jid);
}
@@ -309,12 +309,12 @@ void It2MeNativeMessagingHostTest::WriteMessageToInputPipe(
void It2MeNativeMessagingHostTest::VerifyHelloResponse(int request_id) {
scoped_ptr<base::DictionaryValue> response = ReadMessageFromOutputPipe();
- VerifyCommonProperties(response.Pass(), "helloResponse", request_id);
+ VerifyCommonProperties(std::move(response), "helloResponse", request_id);
}
void It2MeNativeMessagingHostTest::VerifyErrorResponse() {
scoped_ptr<base::DictionaryValue> response = ReadMessageFromOutputPipe();
- VerifyStringProperty(response.Pass(), "type", "error");
+ VerifyStringProperty(std::move(response), "type", "error");
}
void It2MeNativeMessagingHostTest::VerifyConnectResponses(int request_id) {
@@ -335,7 +335,7 @@ void It2MeNativeMessagingHostTest::VerifyConnectResponses(int request_id) {
if (type == "connectResponse") {
EXPECT_FALSE(connect_response_received);
connect_response_received = true;
- VerifyId(response.Pass(), request_id);
+ VerifyId(std::move(response), request_id);
} else if (type == "hostStateChanged") {
std::string state;
ASSERT_TRUE(response->GetString("state", &state));
@@ -391,7 +391,7 @@ void It2MeNativeMessagingHostTest::VerifyDisconnectResponses(int request_id) {
if (type == "disconnectResponse") {
EXPECT_FALSE(disconnect_response_received);
disconnect_response_received = true;
- VerifyId(response.Pass(), request_id);
+ VerifyId(std::move(response), request_id);
} else if (type == "hostStateChanged") {
std::string state;
ASSERT_TRUE(response->GetString("state", &state));
@@ -438,8 +438,8 @@ void It2MeNativeMessagingHostTest::StartHost() {
pipe_.reset(new NativeMessagingPipe());
scoped_ptr<extensions::NativeMessagingChannel> channel(
- new PipeMessagingChannel(input_read_file.Pass(),
- output_write_file.Pass()));
+ new PipeMessagingChannel(std::move(input_read_file),
+ std::move(output_write_file)));
// Creating a native messaging host with a mock It2MeHostFactory.
scoped_ptr<extensions::NativeMessageHost> it2me_host(
@@ -448,7 +448,7 @@ void It2MeNativeMessagingHostTest::StartHost() {
make_scoped_ptr(new MockIt2MeHostFactory())));
it2me_host->Start(pipe_.get());
- pipe_->Start(it2me_host.Pass(), channel.Pass());
+ pipe_->Start(std::move(it2me_host), std::move(channel));
// Notify the test that the host has finished starting up.
test_message_loop_->task_runner()->PostTask(

Powered by Google App Engine
This is Rietveld 408576698