Pass SessionConfig by reference instead of pointer.

remoting/protocol/session.h

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef REMOTING_PROTOCOL_SESSION_H_
 #define REMOTING_PROTOCOL_SESSION_H_
 
 #include <string>
 
 #include "base/callback.h"
@@ skipping 69 matching lines @@
   // JID of the other side.
   virtual const std::string& jid() = 0;
 
   // Configuration of the protocol that was sent or received in the
   // session-initiate jingle message. Returned pointer is valid until
   // connection is closed.
   virtual const CandidateSessionConfig* candidate_config() = 0;
 
   // Protocol configuration. Can be called only after session has been accepted.
   // Returned pointer is valid until connection is closed.
-  virtual const SessionConfig* config() = 0;
+  virtual const SessionConfig& config() = 0;
 
   // Set protocol configuration for an incoming session. Must be called
   // on the host before the connection is accepted, from
   // ChromotocolServer::IncomingConnectionCallback. Ownership of |config| is

[Wez, 2011/09/10 01:04:29]

nit: This comment is out-of-date; consider updating.

[Sergey Ulanov, 2011/09/10 06:45:58]

Done.

   // given to the connection.
-  virtual void set_config(const SessionConfig* config) = 0;
+  virtual void set_config(const SessionConfig& config) = 0;
 
   // The raw auth tokens from the session-initiate, or session-accept stanzas.
   virtual const std::string& initiator_token() = 0;
   virtual void set_initiator_token(const std::string& initiator_token) = 0;
   virtual const std::string& receiver_token() = 0;
   virtual void set_receiver_token(const std::string& receiver_token) = 0;
 
   // A shared secret to use to mutually-authenticate the SSL channels.
   virtual void set_shared_secret(const std::string& secret) = 0;
   virtual const std::string& shared_secret() = 0;
 
   // Closes connection. Callbacks are guaranteed not to be called
   // after this method returns. Must be called before the object is
   // destroyed, unless the state is set to FAILED or CLOSED.
   virtual void Close() = 0;
 
  private:
   DISALLOW_COPY_AND_ASSIGN(Session);
 };
 
 }  // namespace protocol
 }  // namespace remoting
 
 #endif  // REMOTING_PROTOCOL_SESSION_H_