Add OpenSL configurations

media/audio/android/opensles_output.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 MEDIA_AUDIO_ANDROID_OPENSLES_OUTPUT_H_
 #define MEDIA_AUDIO_ANDROID_OPENSLES_OUTPUT_H_
 
 #include <SLES/OpenSLES.h>
 #include <SLES/OpenSLES_Android.h>

[Ami GONE FROM CHROMIUM, 2013/03/22 16:24:41]

nit: Are these includes really necessary?
Aren't they part of the contract of opensles_util.h below, and thus can be
assumed to be included anyway?

[leozwang1, 2013/03/22 21:37:09]

We need these two. opensles_util.h #include <SLES/OpenSLES.h>.

[Ami GONE FROM CHROMIUM, 2013/03/22 22:29:55]

Isn't this (<SLES/OpenSLES.h>) one of the two here??

[leozwang1, 2013/03/22 22:44:48]

If I comment out #include <SLES/OpenSLES.h>, I will get error:
/usr/local/google/home/leozwang/src/chromium/src/third_party/android_tools/ndk//platforms/android-14/arch-arm/usr/include/SLES/OpenSLES_Android.h:31:9:
error: 'sl_int64_t' does not name a type
/usr/local/google/home/leozwang/src/chromium/src/third_party/android_tools/ndk//platforms/android-14/arch-arm/usr/include/SLES/OpenSLES_Android.h:33:9:
error: 'sl_uint64_t' does not name a type

[Ami GONE FROM CHROMIUM, 2013/03/22 23:34:20]

OIC the problem is that you're including OpenSLES_Android.h before
opensles_util.h so you can't rely on its includes.  fine.

-#include <SLES/OpenSLES_AndroidConfiguration.h>
 
 #include "base/compiler_specific.h"
 #include "media/audio/android/opensles_util.h"
 #include "media/audio/audio_io.h"
 #include "media/audio/audio_parameters.h"
 
 namespace media {
 
 class AudioManagerAndroid;
 
@@ skipping 62 matching lines @@
 
   // Container for retrieving data from AudioSourceCallback::OnMoreData().
   scoped_ptr<AudioBus> audio_bus_;
 
   DISALLOW_COPY_AND_ASSIGN(OpenSLESOutputStream);
 };
 
 }  // namespace media
 
 #endif  // MEDIA_AUDIO_ANDROID_OPENSLES_INPUT_H_