Make AudioPlayer optional in ChromotingClient

remoting/protocol/connection_to_host.h

 // Copyright (c) 2012 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_CONNECTION_TO_HOST_H_
 #define REMOTING_PROTOCOL_CONNECTION_TO_HOST_H_
 
 #include <set>
 #include <string>
 
@@ skipping 59 matching lines @@
     // Called when ready state of the connection changes. When |ready|
     // is set to false some data sent by the peers may be
     // delayed. This is used to indicate in the UI when connection is
     // temporarily broken.
     virtual void OnConnectionReady(bool ready) = 0;
   };
 
   ConnectionToHost(bool allow_nat_traversal);
   virtual ~ConnectionToHost();
 
-  // |signal_strategy| must outlive connection.
+  // |signal_strategy| must outlive connection. A null
+  // |audio_stub| disables the audio stream.

[Wez, 2013/07/09 20:08:07]

nit: See comment on ChromotingClient; we should document the requirements for
all the non-obvious parameters, really.

[solb, 2013/07/10 00:00:31]

Done.

   virtual void Connect(SignalStrategy* signal_strategy,
                        const std::string& host_jid,
                        const std::string& host_public_key,
                        scoped_ptr<TransportFactory> transport_factory,
                        scoped_ptr<Authenticator> authenticator,
                        HostEventCallback* event_callback,
                        ClientStub* client_stub,
                        ClipboardStub* clipboard_stub,
                        VideoStub* video_stub,
                        AudioStub* audio_stub);
@@ skipping 73 matching lines @@
   std::set<std::string> not_ready_channels_;
 
  private:
   DISALLOW_COPY_AND_ASSIGN(ConnectionToHost);
 };
 
 }  // namespace protocol
 }  // namespace remoting
 
 #endif  // REMOTING_PROTOCOL_CONNECTION_TO_HOST_H_