am bf208ec9: Merge "Support .avi tracks that have a sample size of 1, i.e. samples != chunks"

* commit 'bf208ec91ec80fd2a3c2474c236b09b37c1a02c4':
  Support .avi tracks that have a sample size of 1, i.e. samples != chunks

Merge "Support .avi tracks that have a sample size of 1, i.e. samples != chunks"

Support .avi tracks that have a sample size of 1, i.e. samples != chunks

Assume for now that all chunks are the same size, i.e. have the same number of
samples.

Change-Id: Ib8a7dfeb16cc3e5be199ff9d98b68dd0b9c23eb8
related-to-bug: 5279872

am fa6dfdcf: Don\'t post position updates too frequently

* commit 'fa6dfdcfd741980ba1c1d1971b69fc6f8a34dfeb':
  Don't post position updates too frequently

Don't post position updates too frequently

i.e. don't hog my message queue.

related-to-bug: 5284760
Change-Id: I8e2d36a11dbee1567b18ed13dc5257a65c6eeda7

Merge "Hold the AAHDecoderPump ThreadWrapper in a ref-counting pointer" into ics-aah

Hold the AAHDecoderPump ThreadWrapper in a ref-counting pointer

Change-Id: I8929905a7bf07a751b5deb4c74875172a33d874d

am 7fc8f515: Merge "Turn an another assertion into a runtime error in ACodec\'s implementation"

* commit '7fc8f515a0add6837d525b4f2b4baa7ebe37fc7f':
  Turn an another assertion into a runtime error in ACodec's implementation

Merge "Turn an another assertion into a runtime error in ACodec's implementation"

Turn an another assertion into a runtime error in ACodec's implementation

Change-Id: I6779b29f200b90d088273ab3204724ef3d8d59bd
related-to-bug: 5284760

am 465d42ef: Merge "Increase the avi extractor\'s confidence slighly over mp3\'s"

* commit '465d42ef781db15a603427115bd66341eb907c31':
  Increase the avi extractor's confidence slighly over mp3's

Increase the avi extractor's confidence slighly over mp3's

So that we don't accidentally identify a .avi file containing an mp3 track as
an mp3 file.

Change-Id: I73495b80cbb5bc1007bf810f2db453c9f175171b
related-to-bug: 5288189

am 973f553b: Make MediaScanner skip certain directories

* commit '973f553be4aabf9656d6c1596b47767896677b0a':
  Make MediaScanner skip certain directories

Make MediaScanner skip certain directories

The list of directories to skip are configurable via setprop.
The main motivation is that some test data folder takes long time
to scan, and media scanner may compete for CPU time against perf
tests therefore skewing the results.

Bug: 5263115
Change-Id: I568213e2a4babf6033021c1d336ef0347c0e3315

am 6f84da50: Merge "Don\'t check mFilledBuffers whether it is empty or not when the port reconfiguration is not meant for buffer reallocation"

* commit '6f84da50ad9a9fa044543fa5dde197f609b01a8f':
  Don't check mFilledBuffers whether it is empty or not when the port reconfiguration is not meant for buffer reallocation

Merge "Don't check mFilledBuffers whether it is empty or not when the port reconfiguration is not meant for buffer reallocation"

Don't check mFilledBuffers whether it is empty or not when the port reconfiguration is not meant for buffer reallocation

Change-Id: Iee9b18449bce9d7565ab5cc7f9e999484051196c
related-to-bug: 5233843

am 11e2e5de: Merge "AudioTrack: extend callback thread sleep time"

* commit '11e2e5dece7a92f50b73240dc0300343a7b495bf':
  AudioTrack: extend callback thread sleep time

am 5303254c: Merge "Always set geometry in queuebuffer"

* commit '5303254cc944449d08c8489caf288516445f7af0':
  Always set geometry in queuebuffer

Merge "AudioTrack: extend callback thread sleep time"

Merge "Always set geometry in queuebuffer"

AudioTrack: extend callback thread sleep time

Do not force wake up the AudioTrack thread every 10ms if no timed
events (loop, markers..) have to be processed.
This will help reduce power consumption.

Change-Id: Icb425b13800690008dd07c27ffac84739e3dbba3

Always set geometry in queuebuffer

The queuebuffer could return early due to timestamp issues. Need to set
the geometry even in that case.

Change-Id: I04d7cd1df3996d640c269285398c0042923ba920

am 13561602: Merge "Audio player must not be used as a timesource before it\'s started."

* commit '135616025f06dfdfff3d6bed51885ef8de5f49b5':
  Audio player must not be used as a timesource before it's started.

Audio player must not be used as a timesource before it's started.

Otherwise it'll trigger a division-by-zero exception since the audio sample rate
is as yet unknown.

Change-Id: I0793aa7c1c348ffa2611272bb646eff6ecf6ff53
related-to-bug: 5242451

am 65e7815d: Merge "Bug 5270905 fix MediaPlayer with IStreamSource"

* commit '65e7815dd49a836e410bfa2817d9d48576547a9a':
  Bug 5270905 fix MediaPlayer with IStreamSource

Merge "Bug 5270905 fix MediaPlayer with IStreamSource"

Bug 5270905 fix MediaPlayer with IStreamSource

Change-Id: Ia8a6381a6c88b4a0a1378aab03f5275f0fa1125a

am 166c1b2e: Merge "Fixing bug in SurfaceMediaSource_test"

* commit '166c1b2e7a1a48a39ecd2400b94f9f21c9868810':
  Fixing bug in SurfaceMediaSource_test

Merge "Fixing bug in SurfaceMediaSource_test"

am 6a39c020: Merge "Apparently keyframe status in audio tracks of .webm/.mkv files is unreliable"

* commit '6a39c0201ca72d6ccbc63e5fec75949a08fcb043':
  Apparently keyframe status in audio tracks of .webm/.mkv files is unreliable

Apparently keyframe status in audio tracks of .webm/.mkv files is unreliable

fortunately in all our supported audio encodings we can treat every frame as
a keyframe.

Change-Id: I32f21d0077bbae7ef9efe725dd351baf531179e2
related-to-bug: 5263837

am 01dba603: Merge "Stop metadata retriever from dereferencing a NULL pointer if"

* commit '01dba603d7c74756126297b843aca0befed8810c':
  Stop metadata retriever from dereferencing a NULL pointer if

Merge "Stop metadata retriever from dereferencing a NULL pointer if"

am a57c639f: Merge "Revert "To make SniffMP3() more concrete so that we can remove false-positve responses from MPEG-PS streams.""

* commit 'a57c639fa7639d80ad29b5b876934f46851dcc69':
  Revert "To make SniffMP3() more concrete so that we can remove false-positve responses from MPEG-PS streams."

Revert "To make SniffMP3() more concrete so that we can remove false-positve responses from MPEG-PS streams."

This reverts commit 983ab88e117fcbda9370b8f4ba94271d37149672.

related-to-bug: 5231309

Stop metadata retriever from dereferencing a NULL pointer if

the media extractor failed to initialize (malformed or unsupported content)

Change-Id: Icfad4e9eeb8d6713ad12eee7979ab30b696c06e0
related-to-bug: 5263840

am b6738fc6: Merge "Issue 5247986: Battery drain due to audio wakelock"

* commit 'b6738fc6a5ab08c23ecc7c0d84316f5fa5e05064':
  Issue 5247986: Battery drain due to audio wakelock

Merge "Issue 5247986: Battery drain due to audio wakelock"

Issue 5247986: Battery drain due to audio wakelock

The problem occurs when activating or deactivating A2DP connection
while SoudPool has a channel active. This can happen quite frequently now
that the UI sound effects are enabled by default.
If PCM data is remaining in the AudioTrack buffer when it is restroyed and
re-created on the new AudioFlinger output thread, this data is flushed.
As a consequence, no underrun or request for new data callback is sent to
SoundPool and the sound channel remains active for ever as the end of the
sample is never detected.

Change-Id: I13e0c11e4ce3f83bff7f58d347ca814b6a86712b

am f12abaf4: Merge "Handle setDataSource failures properly. #5261671"

* commit 'f12abaf40588615fa418fb711a4d754098b589f3':
  Handle setDataSource failures properly. #5261671

Merge "Handle setDataSource failures properly. #5261671"

Handle setDataSource failures properly. #5261671

Change-Id: Iea0aa474d1939db23da9aabdfae2081e834f30d9

am 351f44af: Merge "Don\'t overwrite pending port setting change if there are more than one port setting change event."

* commit '351f44af54f001ee67908f9e1f601f8ffa7905d2':
  Don't overwrite pending port setting change if there are more than one port setting change event.

Merge "Don't overwrite pending port setting change if there are more than one port setting change event."

Fixing bug in SurfaceMediaSource_test

The test was making EGL calls once disconnected. Instead of calling
"disconnect" directly on the Surface, and EGL calls should be made to that
effect.

Change-Id: I21468ac8cbc2cb3145a49269e32a884736cd452e

am 929827b8: Merge "Fix issue 5252593: any app can restart the runtime"

* commit '929827b819415101d526ff68fb0b2c8adb885132':
  Fix issue 5252593: any app can restart the runtime

Fix issue 5252593: any app can restart the runtime

Replace null device address string by empty sting.

Change-Id: I285c35f3345334e6d2190493b1a8a5aca1a361a4

am 69a5f6c3: Properly serialize access to FileSource::getSize by acquiring the mutex

* commit '69a5f6c3e04aa0f3f87eed7578a32782c97bd48f':
  Properly serialize access to FileSource::getSize by acquiring the mutex

Properly serialize access to FileSource::getSize by acquiring the mutex

Both ::getSize and ::read call into lseek64, if this happens simultaneously
from multiple threads the results are undefined if not properly serialized.

Change-Id: I737cafebd836f3d8eb702beac557b4731f69c6f6
related-to-bug: 5196490

am 1d477c53: Merge "Require INTERNET permission for network-based content. Bug #1870981"

* commit '1d477c53bf2f389813d04d5163d210fccf00ceb9':
  Require INTERNET permission for network-based content. Bug #1870981

Require INTERNET permission for network-based content.
Bug #1870981

Change-Id: Ia3ad166390c4d60cea19c3783895b078a2c4c15f

Don't overwrite pending port setting change if there are more than one port setting change event.

Change-Id: I3413f5235b191031252aa3785f5aaef00caef1ce
related-to-bug: 5122512

am ae510085: Merge "Guard the audio notifications with a separate lock to avoid deadlocks during reset."

* commit 'ae51008564a7a32199a144544a4ab57b976ffa9d':
  Guard the audio notifications with a separate lock to avoid deadlocks during reset.

am 7f5ea61b: Merge "Signal an error when encountering a malformed .ts stream."

* commit '7f5ea61b0f0ac479f836c0c47b8b212d73486a0d':
  Signal an error when encountering a malformed .ts stream.

Merge "Guard the audio notifications with a separate lock to avoid deadlocks during reset."

Guard the audio notifications with a separate lock to avoid deadlocks during reset.

Change-Id: I9a97372e1f777d1cbd6774cac2ed34b4b8cd9392
related-to-bug: 5179827

Signal an error when encountering a malformed .ts stream.

Change-Id: I3ab0215419214b652573945403eb45eb09d41aff
related-to-bug: 5114811

am 98bb251c: Add video snapshot camera parameter.

* commit '98bb251c8b527600e7a4cca6cc33499710bbfbf1':
  Add video snapshot camera parameter.

Add video snapshot camera parameter.

The API is still hidden.

bug:5187868

Change-Id: I59e1b5774ca32be049723d1a0887faa90d8884bb

am abf82f9e: Merge "Incr. MIN_UNDEQUEUED_BUFFERS in SurfaceMediaSource"

* commit 'abf82f9ea4960ead8716b183aac1792820cccdd1':
  Incr. MIN_UNDEQUEUED_BUFFERS in SurfaceMediaSource

Merge "Incr. MIN_UNDEQUEUED_BUFFERS in SurfaceMediaSource"

Incr. MIN_UNDEQUEUED_BUFFERS in SurfaceMediaSource

The buffers in SurfaceMediaSource were 4 earlier, whereas
we need 5 for one of the devices.

Change-Id: I87bf5e28bbb0f7c9661dc01b040baa903d95d2c2

am 9100473a: Merge "I have one HLS stream that switches stream PIDs from one bandwidth to the next"

* commit '9100473a7e908c7292008140f76ca5009a4335d4':
  I have one HLS stream that switches stream PIDs from one bandwidth to the next

am afbb70b4: Merge "Discontinuities are only signalled on streams that have been identified, i.e."

* commit 'afbb70b48e348410279816639eeb61f8a33064d2':
  Discontinuities are only signalled on streams that have been identified, i.e.

Merge "I have one HLS stream that switches stream PIDs from one bandwidth to the next"

Merge "Discontinuities are only signalled on streams that have been identified, i.e."

I have one HLS stream that switches stream PIDs from one bandwidth to the next

this change intends to support its very limited case and signals an error in
all other cases of unexpected PID changes that we cannot recover from.

Change-Id: Icbfdf9fe7461969e2a8781ed416f54d891dd789a

Discontinuities are only signalled on streams that have been identified, i.e.

those that have a queue. This ensures that the player doesn't observe discontinuities
that don't match up across streams.

Also, make sure output buffers arriving from the decoder to be rendered are sent
back to the decoder if we started flushing.

Finally, don't parse TS packets for streams we don't support. And don't allocate
memory for them.

Change-Id: I708e0de4cba8110a62e4c8ceb1e5702430d5d2bb

am d318b73e: Merge "Update SurfaceMediaSource_test to not use pbuffer configs"

* commit 'd318b73e8e7832f257797ce08b3eea02780b7909':
  Update SurfaceMediaSource_test to not use pbuffer configs

Merge "Update SurfaceMediaSource_test to not use pbuffer configs"

am e3fb1cb1: Merge "Fix a crash where the EOS was signalled but some output buffer were still pending."

* commit 'e3fb1cb1b1f333aadc780c85de0735143ad864b4':
  Fix a crash where the EOS was signalled but some output buffer were still pending.

Merge "Fix a crash where the EOS was signalled but some output buffer were still pending."

am b2c4f0bf: Merge "Fix SurfaceMediaSource timestamp handling."

* commit 'b2c4f0bf11f38fd31d80f1256c89b9db043a2929':
  Fix SurfaceMediaSource timestamp handling.

Merge "Fix SurfaceMediaSource timestamp handling."

am 3727e3d8: Merge "Stop the AudioSource in reset() of AwesomePlayer if AudioSource is not NULL and AudioPlayer was not started. Fix for 5236419."

* commit '3727e3d80d37fc8004ddaa4ec589bd8d9c304fad':
  Stop the AudioSource in reset() of AwesomePlayer if AudioSource is not NULL and AudioPlayer was not started. Fix for 5236419.

Merge "Stop the AudioSource in reset() of AwesomePlayer if AudioSource is not NULL and AudioPlayer was not started. Fix for 5236419."

Stop the AudioSource in reset() of AwesomePlayer if AudioSource is not NULL
and AudioPlayer was not started.
Fix for 5236419.

Change-Id: I5eadd069118c3533a19b5d6f6dca2ce16d8505fb

MediaPlayer changes for Tungsten/Android@Home.

Squashed merge from master-tungsten of the following changes:

commit e8b156c6b224f0fe5b6a81d5f92bade36275027c
Author: Jason Simmons <jsimmons@google.com>
Date:   Fri Aug 12 10:28:48 2011 -0700

    Add a null check to handle failure of android::createPlayer

    Change-Id: Iacbbc90369a825838d4374b1f774c8880b25753b

commit 43be3231034ff8537fdd84422a7954780038671f
Author: John Grossman <johngro@google.com>
Date:   Mon Jun 27 18:59:12 2011 -0700

    Move libaah_rtp over from the vendor directory.

    Also move factor PipeEvent out into utils.

    Change-Id: Id3877c66efe22d771cf3ef4877107e431b828e37

commit 17526eb3148c9c3d4365b6d5b47e8dc13bca71b6
Author: John Grossman <johngro@google.com>
Date:   Mon Jun 27 17:06:49 2011 -0700

    Name changes for the TRTP Players s/tungsten/aah/g

    Change-Id: I55e9ad13003f6aa6a36955b54426a7efbe31ac51

commit cbf2903ab6893b6e662514e2f6d670e268a419df
Author: John Grossman <johngro@google.com>
Date:   Fri Apr 15 09:27:54 2011 -0700

    Migrate Tungsten code from the HC-Tungsten to the Master-Tungsten branch.

    Change-Id: I95372d913a0761d90168edb4016f5ece0ea74502

Change-Id: Ic8fd28b1bbd85e86a325f99013e7aa0c763f5f05
Signed-off-by: Mike J. Chen <mjchen@google.com>
Signed-off-by: John Grossman <johngro@google.com>

Add media/libaah_rtp

RTP library used to broadcast media from one device to a collection
of listeners.  Handles failures/retries/etc.

This is a squashed merge from master-tungsten of the following changes:

commit e1a5101fe627d71739a7c4263bb3a65c7bc44385
Author: Jason Simmons <jsimmons@google.com>
Date:   Fri Aug 12 13:24:21 2011 -0700

    Hold ThreadWrapper in a ref-counting pointer

    Change-Id: Iaf3343182e37bcc0ca99fbaf8f9bbb8c4984072a

commit 89b90d62e164ff3db27c9cba85255fc476d2dd96
Author: Jason Simmons <jsimmons@google.com>
Date:   Wed Aug 10 13:08:25 2011 -0700

    Update the Tungsten TX player to use HTTPBase

    Change-Id: I9f7ecf1b4b496cec1815284dbcdb958a43284169

commit 43be3231034ff8537fdd84422a7954780038671f
Author: John Grossman <johngro@google.com>
Date:   Mon Jun 27 18:59:12 2011 -0700

    Move libaah_rtp over from the vendor directory.

    Also move factor PipeEvent out into utils.

    Change-Id: Id3877c66efe22d771cf3ef4877107e431b828e37

Change-Id: I190ecddf1dc919761c2e4ec740c2f3b2c9541156
Signed-off-by: Mike J. Chen <mjchen@google.com>
Signed-off-by: John Grossman <johngro@google.com>
Signed-off-by: Jason Simmons <jsimmons@google.com>

am 49fb943d: Merge "ATSParser now supports a mode in which PTS is considered to be absolute time"

* commit '49fb943db85c748fc7dd4f5ee4a461d7489fb1b0':
  ATSParser now supports a mode in which PTS is considered to be absolute time

Merge "ATSParser now supports a mode in which PTS is considered to be absolute time"

Update SurfaceMediaSource_test to not use pbuffer configs

SurfaceMediaSource_test now does not use pbuffer configs
to create a surface. Always uses a window config.
Also, cleaned up the test a bit.

Change-Id: Idda40dcb8474249eccaa8d2dad20ee840d2dd484

am c21843a8: Merge "fix Surface positions are not floats"

* commit 'c21843a8b5c395cbaae6db0f786bdee556075f5d':
  fix Surface positions are not floats

Merge "fix Surface positions are not floats"

fix Surface positions are not floats

Added Surface.setPosition(float, float) which allows to set a surface's
position in float.

Bug: 5239859
Change-Id: I903aef4ad5b5999142202fb8ea30fe216d805711

Fix a crash where the EOS was signalled but some output buffer were still pending.

Change-Id: Ide111264501441b51bb10629ae2021accf9e1fb5
related-to-bug: 5234126

ATSParser now supports a mode in which PTS is considered to be absolute time

i.e. PTS 0 <=> media time 0

Change-Id: If10a0f98c129b95e8430dc419664b1def92e547e

am 0e35c78d: Merge "226483: A2DP connected, but music out to speaker"

* commit '0e35c78d25a8df8b3bde566ad6c474f9df242ea4':
  226483: A2DP connected, but music out to speaker

Merge "226483: A2DP connected, but music out to speaker"

Fix SurfaceMediaSource timestamp handling.

Was not basing timestamps on startTimeUs.
Now synchronizes properly with audio.

Bug: 4510826
Change-Id: I613db0aa91e51fc75d120e65540e742d8ab2ae43

226483: A2DP connected, but music out to speaker

When the A2DP headset is connected, there is a possible
race condition when the audio tracks are moved from
the mixer thread attached to the speaker output to the thread
attached to A2DP output.
As the request to clear the stream type to output mapping cache in
the client process is asynchronous, it is possible that the flag
indicating to the client audio track to re-create the IAudioTrack
on the new thread is processed before the cache is invalidated.
In this case, the track will be attached to the old thread and
music will continue playing over the device speaker instead of being
redirected to A2DP headset.

Change-Id: Ib2ce1eb5320eaff83287b93779061bf4e7a330df

am c5d0fac0: Merge "Bug 5156756 Fix AAC ADTS header size computation"

* commit 'c5d0fac081c4c496b29586bff79dfff05755b716':
  Bug 5156756 Fix AAC ADTS header size computation

Merge "Bug 5156756 Fix AAC ADTS header size computation"

Bug 5156756 Fix AAC ADTS header size computation

This previous frame size computation code had two incorrect
 assumptions:
 1/ the ADTS frame length value already contains the header size
 2/ the ADTS header is not of fixed size: it is 2 bytes bigger
  if it contains the CRC value.
For 1/, the code worked because when the header size was added
 to the frame size, the + operator is of stronger precedence
 than the ? operator, so the size added was always 0 (instead
 of ADTS_HEADER_LENGTH.
For 2/, the code worked as long as there was no CRC in the ADTS
 data.

The fix consists in:
 - documenting what the frame length computation code returns
 - fixing the frame length computation
 - when computing the frame length, also returning the header
  size so the correct data can be sent to the decoder.

Change-Id: I92df72a9e531f594f762e63d62f9dee7b0109904

resolved conflicts for merge of 35c0bf31 to ics-aah

Change-Id: If736c93821359a2f97a0670ad3be8ada5f398bd7

am 499f3658: Merge "Return an error to the client instead of asserting if decoder instantiation fails"

* commit '499f36581c63579f24c3d9bbf95c7f8677cd1191':
  Return an error to the client instead of asserting if decoder instantiation fails

Merge "Audioflinger: reverse logic of BT NREC indication"

Audioflinger: reverse logic of BT NREC indication

The interpretation of BT NREC by AudioFlinger to enable
or disable AEC and NS was wrong: NREC to ON (default) means
the phone (Audio Gateway) must enable local AEC and NS.

Change-Id: I88a264e7fc9831c43bbace4f6b585baec73f2006

Return an error to the client instead of asserting if decoder instantiation fails

after a surface change.

Change-Id: Ic7758cbeb107032db68a1c3b8e2984710a12dd8b
related-to-bug: 5212725