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

Unified Diff: remoting/host/client_session_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/client_session_unittest.cc
diff --git a/remoting/host/client_session_unittest.cc b/remoting/host/client_session_unittest.cc
index 38d66ef5da40e7b6e98d85818162f1996f9b476b..f6380069eba85e088aacb6321845b1b4f226008e 100644
--- a/remoting/host/client_session_unittest.cc
+++ b/remoting/host/client_session_unittest.cc
@@ -176,7 +176,7 @@ void ClientSessionTest::CreateClientSession() {
// Mock protocol::ConnectionToClient APIs called directly by ClientSession.
// HostStub is not touched by ClientSession, so we can safely pass nullptr.
scoped_ptr<protocol::FakeConnectionToClient> connection(
- new protocol::FakeConnectionToClient(session.Pass()));
+ new protocol::FakeConnectionToClient(std::move(session)));
connection->set_client_stub(&client_stub_);
connection_ = connection.get();
@@ -188,7 +188,7 @@ void ClientSessionTest::CreateClientSession() {
task_runner_, // Encode thread.
task_runner_, // Network thread.
task_runner_, // UI thread.
- connection.Pass(),
+ std::move(connection),
desktop_environment_factory_.get(),
base::TimeDelta(),
nullptr,

Powered by Google App Engine
This is Rietveld 408576698