Created define to switch on/off audio.

remoting/protocol/session_config.cc

 // 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.
 
 #include "remoting/protocol/session_config.h"
 
 #include <algorithm>
 
 namespace remoting {
 namespace protocol {
@@ skipping 157 matching lines @@
                     kDefaultStreamVersion,
                     ChannelConfig::CODEC_UNDEFINED));
   result->mutable_event_configs()->push_back(
       ChannelConfig(ChannelConfig::TRANSPORT_STREAM,
                     kDefaultStreamVersion,
                     ChannelConfig::CODEC_UNDEFINED));
   result->mutable_video_configs()->push_back(
       ChannelConfig(ChannelConfig::TRANSPORT_STREAM,
                     kDefaultStreamVersion,
                     ChannelConfig::CODEC_VP8));
+#if ENABLE_AUDIO

[Sergey Ulanov, 2012/07/23 22:17:48]

#if defined(ENABLE_REMOTING_AUDIO)

[kxing, 2012/07/24 15:02:16]

Done.

+  result->mutable_audio_configs()->push_back(
+      ChannelConfig(ChannelConfig::TRANSPORT_STREAM,
+                    kDefaultStreamVersion,
+                    ChannelConfig::CODEC_VERBATIM));
   result->mutable_audio_configs()->push_back(
       ChannelConfig(ChannelConfig::TRANSPORT_NONE,
                     kDefaultStreamVersion,
+                    ChannelConfig::CODEC_VERBATIM));
+#else

[Sergey Ulanov, 2012/07/23 22:17:48]

#else  // !defined(ENABLE_REMOTING_AUDIO)

[kxing, 2012/07/24 15:02:16]

Done.

+  result->mutable_audio_configs()->push_back(
+      ChannelConfig(ChannelConfig::TRANSPORT_NONE,
+                    kDefaultStreamVersion,
                     ChannelConfig::CODEC_VERBATIM));
   result->mutable_audio_configs()->push_back(

[Sergey Ulanov, 2012/07/23 22:17:48]

I think it's better to remove this config for now until our protocol is stable.
Otherwise in the future we may get into situation when an old client connects to
a newer host, the host chooses this config, but then they try to communicate
using incompatible protocol.

[kxing, 2012/07/24 15:02:16]

Done.

       ChannelConfig(ChannelConfig::TRANSPORT_STREAM,
                     kDefaultStreamVersion,
                     ChannelConfig::CODEC_VERBATIM));
+#endif

[Sergey Ulanov, 2012/07/23 22:17:48]

#endig  // !defined(ENABLE_REMOTING_AUDIO)

[kxing, 2012/07/24 15:02:16]

Done.

   result->mutable_audio_configs()->push_back(
       ChannelConfig(ChannelConfig::TRANSPORT_STREAM,
                     kDefaultStreamVersion,
                     ChannelConfig::CODEC_VORBIS));
   return result.Pass();
 }
 
 }  // namespace protocol
 }  // namespace remoting