Audio extension API.

chrome/chrome_browser_extensions.gypi

 # Copyright (c) 2013 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.
 
 {
   'targets': [
     {
       'target_name': 'browser_extensions',
       'type': 'static_library',
       'variables': { 'enable_wexit_time_destructors': 1, },
@@ skipping 70 matching lines @@
         'browser/extensions/api/alarms/alarm_manager.cc',
         'browser/extensions/api/alarms/alarm_manager.h',
         'browser/extensions/api/alarms/alarms_api.cc',
         'browser/extensions/api/alarms/alarms_api.h',
         'browser/extensions/api/app_current_window_internal/app_current_window_internal_api.cc',
         'browser/extensions/api/app_current_window_internal/app_current_window_internal_api.h',
         'browser/extensions/api/app_runtime/app_runtime_api.cc',
         'browser/extensions/api/app_runtime/app_runtime_api.h',
         'browser/extensions/api/app_window/app_window_api.cc',
         'browser/extensions/api/app_window/app_window_api.h',
+        'browser/extensions/api/audio/audio_api.cc',
+        'browser/extensions/api/audio/audio_api.h',
+        'browser/extensions/api/audio/audio_service.h',
         'browser/extensions/api/autotest_private/autotest_private_api.cc',
         'browser/extensions/api/autotest_private/autotest_private_api.h',
         'browser/extensions/api/autotest_private/autotest_private_api_factory.cc',
         'browser/extensions/api/autotest_private/autotest_private_api_factory.h',
         'browser/extensions/api/bluetooth/bluetooth_api.cc',
         'browser/extensions/api/bluetooth/bluetooth_api.h',
         'browser/extensions/api/bluetooth/bluetooth_api_factory.cc',
         'browser/extensions/api/bluetooth/bluetooth_api_factory.h',
         'browser/extensions/api/bluetooth/bluetooth_api_utils.cc',
         'browser/extensions/api/bluetooth/bluetooth_api_utils.h',
@@ skipping 642 matching lines @@
         'browser/extensions/webstore_standalone_installer.h',
         'browser/extensions/webstore_startup_installer.cc',
         'browser/extensions/webstore_startup_installer.h',
         'browser/extensions/window_controller.cc',
         'browser/extensions/window_controller.h',
         'browser/extensions/window_controller_list.cc',
         'browser/extensions/window_controller_list.h',
         'browser/extensions/window_controller_list_observer.h',
       ],
       'conditions': [
+        ['chromeos==0', {
+          'sources': [
+             'browser/extensions/api/audio/audio_service.cc',

[Nico, 2013/04/23 00:28:18]

I think it's more common to always include this, and to conditionally exclude it
on cros. I think you can also always include the _chromeos.cc file, it'll be
excluded automatically on non-cros (see build/file_patters.gypi)

[hshi1, 2013/04/23 00:46:55]

I tried to conditionally exclude this file on cros before, but it somehow didn't
seem to work. What I did was to add the path of audio_service.cc under
'sources!' (and also tried to list it as a excludes rule under 'sources/'), and
then run ./build/gyp_chromium again before rebuilding, but in both attempts I
still end up linking the audio_service.o and ended up calling the stub functions
there.

I wonder if I did anything wrong. Is it required to clean up the entire out
directory if I make gypi changes like this?

On 2013/04/23 00:28:18, Nico wrote:

[Nico, 2013/04/23 02:08:46]

No, you don't have to remove the whole out folder. `sources!` should work, or
you could try an `excludes` section (which takes a pattern instead of a literal
string)

+          ],
+        }],
         ['chromeos==1', {
           'dependencies': [
             '../build/linux/system.gyp:dbus',
             '../chromeos/ime/input_method.gyp:gencode',
             '../third_party/libevent/libevent.gyp:libevent',
             'contacts_proto',
           ],
           'sources!': [
             'browser/extensions/default_apps.cc',
             'browser/extensions/default_apps.h',
           ],
           'sources': [
+            'browser/extensions/api/audio/audio_service_chromeos.cc',
             'browser/extensions/api/enterprise_platform_keys_private/enterprise_platform_keys_private_api.cc',
             'browser/extensions/api/enterprise_platform_keys_private/enterprise_platform_keys_private_api.h',
             'browser/extensions/api/input_ime/input_ime_api.cc',
             'browser/extensions/api/input_ime/input_ime_api.h',
             'browser/extensions/api/rtc_private/rtc_private_api.cc',
             'browser/extensions/api/rtc_private/rtc_private_api.h',
             'browser/extensions/api/terminal/terminal_extension_helper.cc',
             'browser/extensions/api/terminal/terminal_extension_helper.h',
             'browser/extensions/api/terminal/terminal_private_api.cc',
             'browser/extensions/api/terminal/terminal_private_api.h',
@@ skipping 177 matching lines @@
           'variables': {
             'proto_in_dir': 'browser/chromeos/contacts',
             'proto_out_dir': 'chrome/browser/chromeos/contacts',
           },
           'includes': [ '../build/protoc.gypi' ]
         },
       ],
     },],  # 'chromeos=1'
   ],  # 'conditions'
 }