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

Unified Diff: remoting/protocol/webrtc_transport.cc

Issue 1534193004: Use std::move() instead of scoped_ptr<>::Pass(). (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
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/protocol/webrtc_transport.cc
diff --git a/remoting/protocol/webrtc_transport.cc b/remoting/protocol/webrtc_transport.cc
index e4fd5c8f0600a1107d06f03f278f5fe022ff531b..6c4fc1c3bd7ed80a952b7d50bf648a9d7875a3cc 100644
--- a/remoting/protocol/webrtc_transport.cc
+++ b/remoting/protocol/webrtc_transport.cc
@@ -233,7 +233,7 @@ bool WebrtcTransport::ProcessTransportInfo(XmlElement* transport_info) {
return false;
}
} else {
- pending_incoming_candidates_.push_back(candidate.Pass());
+ pending_incoming_candidates_.push_back(std::move(candidate));
}
}
@@ -284,7 +284,7 @@ void WebrtcTransport::OnLocalSessionDescriptionCreated(
offer_tag->SetAttr(QName(std::string(), "type"), description->type());
offer_tag->SetBodyText(description_sdp);
- event_handler_->OnOutgoingTransportInfo(transport_info.Pass());
+ event_handler_->OnOutgoingTransportInfo(std::move(transport_info));
peer_connection_->SetLocalDescription(
SetSessionDescriptionObserver::Create(base::Bind(
@@ -471,7 +471,7 @@ void WebrtcTransport::SendTransportInfo() {
DCHECK(pending_transport_info_message_);
event_handler_->OnOutgoingTransportInfo(
- pending_transport_info_message_.Pass());
+ std::move(pending_transport_info_message_));
pending_transport_info_message_.reset();
}

Powered by Google App Engine
This is Rietveld 408576698