Audio extension API.

chrome/chrome_browser_extensions.gypi

Index: chrome/chrome_browser_extensions.gypi
diff --git a/chrome/chrome_browser_extensions.gypi b/chrome/chrome_browser_extensions.gypi
index 896c2852a291717af5b6f5ba22d4153cf960043d..1f28bc6b6af269588565740af43a2ffe9ea73271 100644
--- a/chrome/chrome_browser_extensions.gypi
+++ b/chrome/chrome_browser_extensions.gypi
@@ -88,6 +88,9 @@
         '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',
@@ -750,6 +753,11 @@
         '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',
@@ -762,6 +770,7 @@
             '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',