Speex encoding/decoding.

remoting/remoting.gyp

 # 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.
 
 {
   'variables': {
     # TODO(dmaclach): can we pick this up some other way? Right now it's
     # duplicated from chrome.gyp
     'chromium_code': 1,
 
@@ skipping 1110 matching lines @@
       'type': 'static_library',
       'variables': { 'enable_wexit_time_destructors': 1, },
       'dependencies': [
         '../base/base.gyp:base',
         '../base/third_party/dynamic_annotations/dynamic_annotations.gyp:dynamic_annotations',
         '../ui/ui.gyp:ui',
         '../net/net.gyp:net',
         '../skia/skia.gyp:skia',
         '../third_party/libvpx/libvpx.gyp:libvpx',
         '../third_party/protobuf/protobuf.gyp:protobuf_lite',
+        '../third_party/speex/speex.gyp:libspeex',
         '../third_party/zlib/zlib.gyp:zlib',
         '../media/media.gyp:yuv_convert',
         'remoting_jingle_glue',
         'proto/chromotocol.gyp:chromotocol_proto_lib',
       ],
       'export_dependent_settings': [
         '../base/base.gyp:base',
         '../net/net.gyp:net',
         '../skia/skia.gyp:skia',
         '../third_party/protobuf/protobuf.gyp:protobuf_lite',
@@ skipping 38 matching lines @@
         'base/running_average.cc',
         'base/running_average.h',
         'base/stoppable.cc',
         'base/stoppable.h',
         'base/util.cc',
         'base/util.h',
         # TODO(kxing): Seperate the audio and video codec files into a separate
         # target.
         'codec/audio_decoder.cc',
         'codec/audio_decoder.h',
+        'codec/audio_decoder_speex.cc',
+        'codec/audio_decoder_speex.h',
         'codec/audio_decoder_verbatim.cc',
         'codec/audio_decoder_verbatim.h',
         'codec/audio_encoder.h',
+        'codec/audio_encoder_speex.cc',
+        'codec/audio_encoder_speex.h',
         'codec/audio_encoder_verbatim.cc',
         'codec/audio_encoder_verbatim.h',
       ],
     },  # end of target 'remoting_base'
 
     {
       'target_name': 'remoting_host',
       'type': 'static_library',
       'variables': { 'enable_wexit_time_destructors': 1, },
       'dependencies': [
         'remoting_base',
         'remoting_jingle_glue',
         'remoting_protocol',
         'differ_block',
         '../crypto/crypto.gyp:crypto',
       ],
+      'include_dirs': [
+        # chromoting_host.cc needs to call the AudioEncoderSpeex constructor,
+        # which needs this include.
+        # TODO(kxing): Refactor the code out of chromoting_host.cc and remove
+        # this include directory.
+        '../third_party/speex/include',

[Sergey Ulanov, 2012/08/10 20:41:37]

You can avoid it - see my comments in audio_decoder_speex.h . In any case it
would be better to add dependency directly in this target instead of specifying
include_dir

[kxing, 2012/08/13 21:39:45]

Done.

+      ],
       'sources': [
         'host/audio_capturer.cc',
         'host/audio_capturer.h',
         'host/audio_capturer_linux.cc',
         'host/audio_capturer_mac.cc',
         'host/audio_capturer_win.cc',
         'host/audio_scheduler.cc',
         'host/audio_scheduler.h',
         'host/capture_scheduler.cc',
         'host/capture_scheduler.h',
@@ skipping 637 matching lines @@
                   '../base/allocator/allocator.gyp:allocator',
                 ],
               },
             ],
           ],
         }],
       ],  # end of 'conditions'
     },  # end of target 'remoting_unittests'
   ],  # end of targets
 }