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

Unified Diff: remoting/protocol/pepper_stream_channel.cc

Issue 9240033: Use scoped_ptr<>.Pass() to pass ownership in the remoting protocol code. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: - Created 8 years, 11 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
Index: remoting/protocol/pepper_stream_channel.cc
diff --git a/remoting/protocol/pepper_stream_channel.cc b/remoting/protocol/pepper_stream_channel.cc
index 477f9fa35cd9f02f046fce770f083dc1c2f4536e..1f1ec79d0a14c99760c609d7c67971d08589ab27 100644
--- a/remoting/protocol/pepper_stream_channel.cc
+++ b/remoting/protocol/pepper_stream_channel.cc
@@ -54,12 +54,13 @@ PepperStreamChannel::~PepperStreamChannel() {
DCHECK(!connected_ || channel_ == NULL);
}
-void PepperStreamChannel::Connect(pp::Instance* pp_instance,
- const TransportConfig& transport_config,
- ChannelAuthenticator* authenticator) {
+void PepperStreamChannel::Connect(
+ pp::Instance* pp_instance,
+ const TransportConfig& transport_config,
+ scoped_ptr<ChannelAuthenticator> authenticator) {
DCHECK(CalledOnValidThread());
- authenticator_.reset(authenticator);
+ authenticator_ = authenticator.Pass();
pp::Transport_Dev* transport =
new pp::Transport_Dev(pp_instance, name_.c_str(),

Powered by Google App Engine
This is Rietveld 408576698