am e010f65e: audioflinger: Fix for a deadlock in track creation

* commit 'e010f65e6337267cb15f8894c950a3f64370dd36':
  audioflinger: Fix for a deadlock in track creation

audioflinger: Fix for a deadlock in track creation

AudioFlinger enters a deadlock (with itself) on trying to free a
RecordTrack or Track object that failed initialization. Clear this
bad object from the caller instead.

Bug: 12423233
Change-Id: I926f2beb922a70f6924e593e2bbf1a5b5df85b16

am a4f24ff9: AudioTrack: fix position callback after restore

* commit 'a4f24ff9b239300b4bb2098c1413c0a60edc2d7e':
  AudioTrack: fix position callback after restore

am 1bf0eecd: Merge "AudioMixer: Remove tracks from enabledTracks after reseting outTemp"

* commit '1bf0eecd2ec6a3b3e29ad2c91bcefda76abef2fe':
  AudioMixer: Remove tracks from enabledTracks after reseting outTemp

Merge "AudioMixer: Remove tracks from enabledTracks after reseting outTemp"

AudioMixer: Remove tracks from enabledTracks after reseting outTemp

If any track goes through AudioMixer::process__genericNoResampling, and
its getnextbuffer returns NULL, Then that track is removed by AudioMixer
from enabledTracks.

Thus if all tracks getnextbuffer return NULL, Then this function doesn't
reset outTemp and last buffer in AudioFlinger's mMixBuffer will be
repeated and noise is observed.

Remove tracks from enabledTracks after reseting outTemp to zero, so that
process__genericNoResampling will reset outTemp and noise won't appear.

Bug: 12450065
Change-Id: I1ccac7ee4a3bf7fd930254356c072099e11e7c19
Signed-off-by: Gaurav Kumar <gaurav.kumar@broadcom.com>
Signed-off-by: Pierre Couillaud <pierre@broadcom.com>

am 06da69d5: Merge "frameworks/av: convert LOCAL_MODULE_PATH to LOCAL_MODULE_RELATIVE_PATH"

* commit '06da69d57d90e180e1165905e6c703227a06f041':
  frameworks/av: convert LOCAL_MODULE_PATH to LOCAL_MODULE_RELATIVE_PATH

Merge "frameworks/av: convert LOCAL_MODULE_PATH to LOCAL_MODULE_RELATIVE_PATH"

frameworks/av: convert LOCAL_MODULE_PATH to LOCAL_MODULE_RELATIVE_PATH

LOCAL_MODULE_PATH doesn't work for multiarch builds, replace it
with LOCAL_MODULE_RELATIVE_PATH.

Change-Id: I4e4ceec61d026bbe74ba604554c06104bde42e5e

am 4ef1dc2d: Merge "Only increase the counter when we do take a wakelock"

* commit '4ef1dc2d5b5f5df93b173b0b4c4719d4484e031d':
  Only increase the counter when we do take a wakelock

am d0e7e801: Merge "Do partial reads in MemoryLeakTrackUtil"

* commit 'd0e7e8012e19ac750e7858756247be1036da32c4':
  Do partial reads in MemoryLeakTrackUtil

Merge "Only increase the counter when we do take a wakelock"

Merge "Do partial reads in MemoryLeakTrackUtil"

Do partial reads in MemoryLeakTrackUtil

Do partial read in MemoryLeakTrackUtil dumpMemoryAddresses
to avoid using more memory than what is allocated.

Change-Id: I94feb4e00647407f938571167b981c7371f39e3d

Only increase the counter when we do take a wakelock

We keep track on how many wakelocks we have taken. We always
just take one real wakelock but increase / decrease the
counter every time we try to acquire / release a wakelock.
The counter is always increased even if the power manager is
not ready, leading to an incorrect counter that could cause a
crash when we try to release it.
Make sure we only increase the counter when a wakelock, real
or counted, is taken.

Change-Id: Iad940e052694932f1dad8a1a71fa63601d289d6a

am 96a7b5a9: Merge "Long mp3 metadata displays as corrupted file after transfer"

* commit '96a7b5a9c87ec3f8cadad5bffc06a82f9f4d86d8':
  Long mp3 metadata displays as corrupted file after transfer

Merge "Long mp3 metadata displays as corrupted file after transfer"

am 07a017d3: Merge "Added support for ID3v2 meta data in 3gp files"

* commit '07a017d3ff9cb36e47649a9de28e469415eb8103':
  Added support for ID3v2 meta data in 3gp files

am 4cc47940: Merge "Fix compile error in NuPlayerRenderer debug print"

* commit '4cc4794029998b21a1689794cd5bcbd5e4480ef6':
  Fix compile error in NuPlayerRenderer debug print

Merge "Added support for ID3v2 meta data in 3gp files"

Merge "Fix compile error in NuPlayerRenderer debug print"

Added support for ID3v2 meta data in 3gp files

Added support for parsing ID3v2 meta data from
the ID32 chunk in 3gp files. The priority will be
3gpp -> ID3v2 -> iTunes per field.

Change-Id: I0282ecab58e3e5fa6bd738078d562c8bb8ce00ed

am 1e8a57c2: Merge "Avoid jumps to faulty position after seeks"

* commit '1e8a57c212d26c41325e950ed629244071acc326':
  Avoid jumps to faulty position after seeks

Merge "Avoid jumps to faulty position after seeks"

am c7614eb7: Merge "libcameraservice: Fix build in ISO C++11 mode"

* commit 'c7614eb7cf2e3e121baa3de26e3622974a163786':
  libcameraservice: Fix build in ISO C++11 mode

Merge "libcameraservice: Fix build in ISO C++11 mode"

am e311b15c: Merge "Fix c++11 narrowing"

* commit 'e311b15cf821f65f68af2cdfe01e864cbc9a86ce':
  Fix c++11 narrowing

Merge "Fix c++11 narrowing"

am 61540b5f: Merge "Frameworks: AudioFlinger: Fix effects memory leak"

* commit '61540b5fe82cad1c6baa018f02bd2554e62e4364':
  Frameworks: AudioFlinger: Fix effects memory leak

Merge "Frameworks: AudioFlinger: Fix effects memory leak"

am 5ca94d2f: Merge "HLS: Fixed rounding error with decimal segment duration"

* commit '5ca94d2f3c4662aed7b66a97b77eb2d1948464ad':
  HLS: Fixed rounding error with decimal segment duration

Merge "HLS: Fixed rounding error with decimal segment duration"

am 99044adc: Merge "Change M4OSA_ERR_CREATE to return unsigned integer."

* commit '99044adc6e209d31e9c5308d960e2bd5f6999f85':
  Change M4OSA_ERR_CREATE to return unsigned integer.

Merge "Change M4OSA_ERR_CREATE to return unsigned integer."

Fix compile error in NuPlayerRenderer debug print

When activating verbose log prints in NuPlayerRenderer the build
fails because a variable have changed but the log print hasn't been
updated.

Change-Id: I3089b087d296c37dfe6379d7e75d5892912fef96

Change M4OSA_ERR_CREATE to return unsigned integer.

While M4OSA_ERR is defined as M4OSA_UInt32, MOSA_ERR_CREATE
is defined to return M4OSA_Int32. This leads to signed/unsigned
comparison warnings. M4OSA_ERR_CREATE has been changed to return
M4OSA_UInt32 to fix this issue.

Change-Id: I71a5c50a95c7f296469604b486a1d3969d302a3f
Signed-off-by: Ashok Bhat <ashok.bhat@arm.com>

am 8115f4b2: Merge "stagefright: do not offload LD-AAC decoding"

* commit '8115f4b206dbd04c620f062feb3c7785e2732ab3':
  stagefright: do not offload LD-AAC decoding

am 1777ed30: Merge "stagefright: fix offloading HE-AAC sampling rate."

* commit '1777ed30bfb3b9c1edc037a1b5482d5ab8f28b9b':
  stagefright: fix offloading HE-AAC sampling rate.

Merge "stagefright: do not offload LD-AAC decoding"

stagefright: do not offload LD-AAC decoding

For now, do not offload LD and ELD AAC decoding because there
is no way to know if it is supported by the audio DSP implementation.
The longer term fix will be to have mapMimeToAudioFormat() use the
audio object type in track metadata to refine the AAC format and the
audio HAL list supported AAC profiles.

Change-Id: Iaa88ecf3f4ae42ad48c1b42a9b007dd80eb88147

Merge "stagefright: fix offloading HE-AAC sampling rate."

Long mp3 metadata displays as corrupted file after transfer

Cut the metadata to 1 Byte and return in function
getObjectPropertyList in order to show it properly on a PC.

Change-Id: Iefacf9fa86c20ece2572e6d95d35877a94066fe7

stagefright: fix offloading HE-AAC sampling rate.

Fix HE AAC SBR sampling rate reading for explicit
signaling.

Bug: 11697128.
Change-Id: Ifec0ab9d48d9293f6774ec1efd9da9445994cb7c

HLS: Fixed rounding error with decimal segment duration

If segment duration is specified with decimal value only the
integer value will be used, causing the stream duration to
be wrong.

Reported to Android public issue tracker:
https://code.google.com/p/android/issues/detail?id=56223

Change-Id: I34fb7a81af6ad3d9a214228cfe3724636ebf5ab5

Avoid jumps to faulty position after seeks

When seeking multiple times it is possible that some seeks
are discarded in AwesomePlayer, which causes unwanted jumps
to faulty positions.
The reason is that a seek flag is reset twice in AwesomePlayer.
At first when the video seek is completed and then again when the
audio seek is completed. If a new seek is made after the previous
video seek completed but before the previous audio seek completed,
the new seek position is discarded by the previous audio seek
completion.

This fix makes sure that the seek flag is reset only when video
has completed the seek.

Change-Id: I8f8741d4cb8682345f1d1855bbad57c05f4e3c8d

am 1f95555c: Merge "libeffects: do not use GNU old-style field designators"

* commit '1f95555c69219180f792ac742cc0e386496c29e6':
  libeffects: do not use GNU old-style field designators

Merge "libeffects: do not use GNU old-style field designators"

libeffects: do not use GNU old-style field designators

Avoiding the use of GCC extensions improves code portability

Change-Id: I9edbedc5c8ad4aa46ca54bc2e28280441431a530

am 274cc85d: Merge "libstagefright: Delay release of wakelock in TimedEventQueue"

* commit '274cc85dcb255185838705a91dba00efa52bf436':
  libstagefright: Delay release of wakelock in TimedEventQueue

am f81f7a52: Merge "audioflinger: check for condition before waiting"

* commit 'f81f7a52d4720f441197f75918d2b2c05d41ab45':
  audioflinger: check for condition before waiting

am 645e4397: Merge "AudioTrack: fix position callback after restore"

* commit '645e43977d3aabc5addab022d772accc32fd5bdc':
  AudioTrack: fix position callback after restore

am 9060d498: Merge "update offloaded audio track sampling rate"

* commit '9060d498be0c54d3caf84e2dbf1ba62516b3e76e':
  update offloaded audio track sampling rate

Merge "libstagefright: Delay release of wakelock in TimedEventQueue"

libstagefright: Delay release of wakelock in TimedEventQueue

Delay release of wakelock in the TimedEventQueue to
after an event has been processed.
This ensures AP shutdown does not happen while an event
is ready but hasn't been processed yet.

Bug: 11976087.

Change-Id: I71a5f3ac4a57e1d05dd5d9ab5c6f91ed7bb64c87

Merge "audioflinger: check for condition before waiting"

audioflinger: check for condition before waiting

AsyncCallbackThread must check for any condition that
has already been satisfied before waiting.

Bug: 11824817
Change-Id: Ic8c2090d521ecd6a30b76ee75635258d35eb1eff

Merge "AudioTrack: fix position callback after restore"

AudioTrack: fix position callback after restore

When restoring an AudioTrack, the next position callback point
should not be modified and set ahead of current buffer head.
Otherwise, as frames are dropped, the new position is never reached
and an application relying on position callbacks to reload the buffer
would be stalled.

Bug: 11868603.
Change-Id: I93b2a311642a0c89944b78bcc0482d4ceed98ae4

Merge "update offloaded audio track sampling rate"

update offloaded audio track sampling rate

AudioPlayer must read the sampling rate from offloaded audio sinks
whenever a new time position is computed as the decoder can update
the sampling rate on the fly.

Change-Id: I997e5248cfd4017aeceb4e11689324ded2a5bc88

am a67c7653: Merge "Increase kFastTrackMultiplier from 1 to 2"

* commit 'a67c76538fff102cd44b2f38dd073df5f2a33605':
  Increase kFastTrackMultiplier from 1 to 2

Merge "Increase kFastTrackMultiplier from 1 to 2"

am a0e1c391: Merge changes Ia684fde5,I58fcb526

* commit 'a0e1c3918dd1356445df98e89dab6be2c8a78da3':
  Fix the help text
  screenrecord fixes

am 1c9ca7d1: Merge "Add "--bugreport" option to screenrecord"

* commit '1c9ca7d13e948af4633d5e5f8117d8c53ac684f6':
  Add "--bugreport" option to screenrecord

Merge changes Ia684fde5,I58fcb526

* changes:
  Fix the help text
  screenrecord fixes

Merge "Add "--bugreport" option to screenrecord"

Fix the help text

Pesky bloggers.

(cherry-pick from Ia8677b054423db292a34e28337431b57804df259)

Change-Id: Ia684fde52697ea78fca79de958ef8b31a50e68ba

screenrecord fixes

Fixes to issues identified during code review.

(cherry-pick from I2203694acb5c0544878f64f4347d29ad1a0725c4)

Change-Id: I58fcb5264fc17b26fac4b03f95d35262e9e199e2

Add "--bugreport" option to screenrecord

The --bugreport option adds two visible features: (1) a timestamp
overlay that (mostly) matches logcat, making it easier to match what
appears in the video with what's in the log, and (2) an "info page"
at the start of the video that shows the system configuration.

Enabling this option adds an additional composition step,
increasing the overhead of screenrecord.  Depending on the device
and circumstances, this may be unnoticeable or very pronounced.
If --bugreport is not enabled, the overhead of screenrecord is
unchanged.

We also now track device orientation changes.  This is currently
detected by polling surfaceflinger, which is suboptimal.  As a
result, we detect the rotation too late, and get a weird mixed
frame before the start of the animation for 90-degree changes.

Also, allow the bit rate to be specified as e.g. "4M" for 4Mbps.

Also, --rotate is now deprecated.

Bug 11220305
Bug 11136964

(cherry pick from Ibb94b81d2f73547b95d7a47e027da75fab187a4f)

Change-Id: I829a91aaca5ab82a07c14172d9e188ec38f14e57

am e2d617f5: Merge commit \'2381f06f374ee0cb8bca0edf5388394432b00e6d\' into HEAD

* commit 'e2d617f5ba7fb90f27b03e2593666b2c927e4dc9':

Merge commit '2381f06f374ee0cb8bca0edf5388394432b00e6d' into HEAD

Increase kFastTrackMultiplier from 1 to 2

Change-Id: I158f147295eebcea96e4047d7618069bc48bdd7d

am 92630351: Merge "NuPlayer: Use a software renderer when using software codecs"

* commit '92630351d265a61faae2dfe006a8bb330283aa7b':
  NuPlayer: Use a software renderer when using software codecs

Merge "NuPlayer: Use a software renderer when using software codecs"

AudioTrack: fix position callback after restore

When restoring an AudioTrack, the next position callback point
should not be modified and set ahead of current buffer head.
Otherwise, as frames are dropped, the new position is never reached
and an application relying on position callbacks to reload the buffer
would be stalled.

Bug: 11868603.
Change-Id: I93b2a311642a0c89944b78bcc0482d4ceed98ae4

am 90892d1b: Merge "Fix SIGABRT when playing mp4 file"

* commit '90892d1be84776f59a619ea4ef9ce51be5828ff8':
  Fix SIGABRT when playing mp4 file

Merge "Fix SIGABRT when playing mp4 file"

am 11e71558: Merge "Fixed data offset at parsing IPMP Descriptors"

* commit '11e715589e0f3f96389d0eedab677369da680fd4':
  Fixed data offset at parsing IPMP Descriptors

Fix SIGABRT when playing mp4 file

If the track duration was shorter than the segment duration, the calculated
encoder padding would be negative, resulting in a crash.
b/11823061
https://code.google.com/p/android/issues/detail?id=62610

Change-Id: I3989ad88caea38d212b61355c15aec13382c6116

Merge "Fixed data offset at parsing IPMP Descriptors"

Fixed data offset at parsing IPMP Descriptors

MPEG4Extractor::parseDrmSINF() miscalculated data offset in parsing
IPMP Descriptors. This commit makes it to take in 2 bytes which is
the size of IPMPS_Type field.

Change-Id: I42cbb6793af9d9b2f14dbfdd7a616096002793f9

am 5bf2560c: Merge commit \'b2059ff384eee8ffb70a7ec8fc5570405201c734\' into HEAD

* commit '5bf2560ce9b70bee077e0c264ac06648f0f63acc':

Merge commit 'b2059ff384eee8ffb70a7ec8fc5570405201c734' into HEAD

am 5ea5b020: Merge "WA: Queue extra buffers on output port during reconfig if input EOS-ed" into klp-dev

* commit '5ea5b020b311a68f513d4a40aca5551abc6a07d8':
  WA: Queue extra buffers on output port during reconfig if input EOS-ed

Merge "WA: Queue extra buffers on output port during reconfig if input EOS-ed" into klp-dev

am 24605338: Merge "Fix metadata access" into klp-dev

* commit '246053380a9f628405a29a055a3f1f4fba13ed5b':
  Fix metadata access

Merge "Fix metadata access" into klp-dev

Fix metadata access

Metadata string pointers become invalid after setting more metadata,
so don't cache them.
b/11692062

Change-Id: Iaf1afb24cf53f7fa36f49ce759355693494076e5

am d7e59228: audioflinger: do not use raw pointer for tracks

* commit 'd7e59228caad3867794d847f6bf163c6495e9506':
  audioflinger: do not use raw pointer for tracks

audioflinger: do not use raw pointer for tracks

Commit 9da3d95 surfaced a problem caused by the use of a raw
pointer to a track in offload thread implementation.

Pointers to tracks should always be weak or strong pointers.

Bug: 11708529.
Change-Id: Ic48632532d186c9be8261f73cefdf824b9fbbd2b

am 7dae71d6: Merge "AwesomePlayer: correct stream type for offload" into klp-dev

* commit '7dae71d606ded1dbc2aa9733c3d98ffac57988f2':
  AwesomePlayer: correct stream type for offload

Merge "AwesomePlayer: correct stream type for offload" into klp-dev

AwesomePlayer: correct stream type for offload

canOffloadStream() function in stagefright utils forces the
stream type to AUDIO_STREAM_MUSIC when querying the audio policy
manager if a particular track is offloadable or not.
This causes MP3 ringtones to be offloaded which is not a validated use case.

The fix consists in using the actual stream type read from the AudioSink.

Bug: 11410937.
Change-Id: I44b8e033a8e785a79cdc291b142f80b5580bdc4d

am d8a62e25: Camera2: Rework the FPS range vs. FPS single setting detection

* commit 'd8a62e25ba6520c2531c7a3d32cc8066e1dab776':
  Camera2: Rework the FPS range vs. FPS single setting detection

WA: Queue extra buffers on output port during reconfig if input EOS-ed

Some codecs may return input buffers before having them processed.
This causes a halt if we already signaled an EOS on the input
port.  For now keep submitting output meta buffers one at a time if
the input EOS-ed, but the output not yet.

Normally, we submit an output buffer for each input buffer that is
with the component (waiting to be processed).

Change-Id: I8a1251bfb504f40f1e4085a1e220bf9a4d0b05d9
Signed-off-by: Lajos Molnar <lajos@google.com>
Bug: 11433909

Camera2: Rework the FPS range vs. FPS single setting detection

Give up on current approach of writing out consistent FPS values
into parameters that will be read back by the app.

- Preserve app's latest set parameters exactly, and compare against
  them when detecting if a new FPS range or single FPS value has been
  selected.

- Since get() returns exactly what was set(), it doesn't matter if the
  app calls getParameters() before its next setParameters(), in terms
  of retriggering FPS selection logic. Before, the behavior varied
  depending on whether the app re-read the parameters.

- As before, if app changes both range and single FPS in a single set
  call, the range set wins. Otherwise, the value that has changed more
  recently is used.

Bug: 11570973
Change-Id: I72b5e60c3f60e88d55127dd1bda87e26eaf929c6

am 4215e616: Merge "audioflinger: fix offload track transition" into klp-dev

* commit '4215e6166fca9f87a6e9e848b3dfd4ab0d25c954':
  audioflinger: fix offload track transition

am 9f357f31: Merge "audioflinger: fix offload resume after drain" into klp-dev

* commit '9f357f319205d52c04a2c8b5cc9d518ddcfdea94':
  audioflinger: fix offload resume after drain

Merge "audioflinger: fix offload track transition" into klp-dev

Merge "audioflinger: fix offload resume after drain" into klp-dev