am f64bd473: (-s ours) Reconcile with jb-mr2-release jb-mr2-zeroday-release - do not merge

* commit 'f64bd473e4c500dc54923baac7261e6d2d72daa6':
  Update HTTP proxy configuration for all media playback inside stagefright.

Reconcile with jb-mr2-release jb-mr2-zeroday-release - do not merge

Change-Id: Idf0f0cdf02e6e94603ff003b6baf869b4414e1cf

merge in jb-mr2-zeroday-release history after reset to jb-mr2-dev

am 0eefffd8: am 47a1fe04: am a9aa6baa: Merge "Set u.ext_data to null after it has been freed"

* commit '0eefffd8fc8abfbe694814fcec397aeac3f545b8':
  Set u.ext_data to null after it has been freed

am 47a1fe04: am a9aa6baa: Merge "Set u.ext_data to null after it has been freed"

* commit '47a1fe04637793f48d808261b1fac47acc3de1e1':
  Set u.ext_data to null after it has been freed

am a9aa6baa: Merge "Set u.ext_data to null after it has been freed"

* commit 'a9aa6baa5d86fc1a5bd37fc636d766d9e3ad2507':
  Set u.ext_data to null after it has been freed

Merge "Set u.ext_data to null after it has been freed"

Update HTTP proxy configuration for all media playback inside stagefright.

Change-Id: Ie0dd00045aba668d8b49da73224e7a7c9c04f69b
related-to-bug: 8873723
(cherry picked from commit 2704965b8a1ff3b7450ff58ccecf86d8ec688c40)

am 903871da: am 83790fcc: am bdf33c98: Merge "Fix compilation warning: incorrect format specifier for type \'ssize_t\'."

* commit '903871da7a7c71b8718133e60e2f0835837b5f3f':
  Fix compilation warning: incorrect format specifier for type 'ssize_t'.

Merge "Public API changes for audio offload support."

resolved conflicts for merge of 364a2677 to master

Change-Id: I21f23c1c5e4408c24c359bef6270dffce47a988d

am 83790fcc: am bdf33c98: Merge "Fix compilation warning: incorrect format specifier for type \'ssize_t\'."

* commit '83790fcce4f5cda1e7c89f1cbed452426502f227':
  Fix compilation warning: incorrect format specifier for type 'ssize_t'.

am bdf33c98: Merge "Fix compilation warning: incorrect format specifier for type \'ssize_t\'."

* commit 'bdf33c98912090176183d3e9b9a9217dbbdca6da':
  Fix compilation warning: incorrect format specifier for type 'ssize_t'.

Merge "Fix compilation warning: incorrect format specifier for type 'ssize_t'."

am a691ff3c: Camera2/3: Don\'t allow recording and callbacks to coexist.

* commit 'a691ff3c03e38e148bbefed35ebb15e552a12613':
  Camera2/3: Don't allow recording and callbacks to coexist.

Set u.ext_data to null after it has been freed

SEGV_MAPPER crash could happen in MetaData::typed_data::freeStorage.
The faulty address could be both random pointers and deadbaad. It
seems like free could be called more than once on u.ext_data. Set
u.ext_data to null after it has been freed as there already is a
null check before trying to free it.

Change-Id: I710c49f9c231bd8852159914c0b2add58a685f0e

Public API changes for audio offload support.

NOTE: this does _not_ include all private member variables added
to classes as part of offload support. Only public/protected functions
and stubs functions/variables needed to make the changes buildable.

- isOffloadSupported() added to audio policy service
    A stub implementation is required to build, this always returns false
- setParameters() added to IAudioTrack
    A stub implementation is required to build, this always returns
    INVALID_OPERATION

- CBlk flag for stream end

- Change AudioSystem::getRenderPosition() to take an audio_output_t
    so caller can specify which output to query

- Add AudioSystem::isOffloadSupported()
    This is fully implemented down to the AudioFlinger function
    AudioPolicyServer::isOffloadSupported() which is just a stub
    that always returns false.

- Add EVENT_STREAM_END to AudioTrack interface.
    STREAM_END is used to signal when the hardware has actually finished
    playing all the data it was sent.

- Add event type enumeration to media player interface AudioSink callbacks
    so that the same callback can be used to handle multiple types of
    event. For offloaded tracks we also have to handle STREAM_END and
    TEAR_DOWN events

- Pass audio_offload_info_t to various functions used for opening outputs,
    tracks and audio players. This passes additional information about the
    compressed stream down to the HAL when using offload.
    For publicly-available APIs this is an optional parameter (for some of
    the internal and low-level APIs around the HAL interface it is mandatory)

- Add getParameters() and setParameters() API to AudioTrack
    Currently dummy implementations.

- Change AudioPlayer contructor so that it takes a set of bitflags defining what
    options are required. This replaces the original bool which only specified
    whether to use deep buffering.

- Changes to StageFright class definition related to handling tearing-down of
    an offloaded track when we need to switch back to software decode

- Define new StageFright utility functions used for offloaded tracks
    Currently dummy implementations.

- AudioFlinger changes to use extended audio_config_t.
    Fills in audio_offload_info_t member if this info is passed in when
    opening an output.

- libvideoeditor changes required to add the new event type parameter
    to AudioSink callback functions

- libmediaplayerservice changes required to add the new event type parameter
    to AudioSink callback functions

Change-Id: I3ab41138aa1083d81fe83b886a9b1021ec7320f1
Signed-off-by: Richard Fitzgerald <rf@opensource.wolfsonmicro.com>
Signed-off-by: Eric Laurent <elaurent@google.com>

Camera2/3: Don't allow recording and callbacks to coexist.

- Tear down conflicting streams when necessary.

- Shut down callbacks if recording starts

- Do not allow callbacks to start if recording is active

Per the current camera API, recording and preview callbacks cannot be
active simultaneously. However, the framework did not explicitly
disallow this, and in fact left the streams configured once they were
created, even if switching between the two operational modes.

In addition, no guards existed for trying to enable both recording and
callbacks at the same time.

Bug: 9423825

Change-Id: I7d6e6114c2e14fcfb5299b4c72ad557895cbf4b8

Fix compilation warning: incorrect format specifier for type 'ssize_t'.

warning: format '%ld' expects argument of type 'long int',
but argument 7 has type 'ssize_t {aka int}' [-Wformat]

Change-Id: I823835a26320a3abb82f8dff9fbe82e5132e7556
Signed-off-by: Scott Fan <fancp2007@gmail.com>

am bdebd481: (-s ours) Reconcile with jb-mr2-zeroday-release - do not merge

* commit 'bdebd4817cfe49fafedfe8f9c65930e03f9b508b':

am 1aee0be6: am 3a4ff0b8: am d74a9ee8: Merge "libstagefright: fix the duration "00:00" for some httplive link"

* commit '1aee0be6cc9ecb4ff3d86228bb68c295c12bde9c':
  libstagefright: fix the duration "00:00" for some httplive link

Reconcile with jb-mr2-zeroday-release - do not merge

Change-Id: I4de46f3bfcfb80867bbeb52a76d383f5870e74f4

Merge "Adds VP9 decoding support for stagefright."

am 3a4ff0b8: am d74a9ee8: Merge "libstagefright: fix the duration "00:00" for some httplive link"

* commit '3a4ff0b8bf464c9c286d63326bea42609527051e':
  libstagefright: fix the duration "00:00" for some httplive link

am d74a9ee8: Merge "libstagefright: fix the duration "00:00" for some httplive link"

* commit 'd74a9ee83fd6e033fd2260ca3777390d7e5f817a':
  libstagefright: fix the duration "00:00" for some httplive link

Merge "libstagefright: fix the duration "00:00" for some httplive link"

Merge "Workaround AudioRecord bug for large buffer sizes"

Merge "Use mFutex as an event flag rather than semaphore"

Merge changes I5b2eeab2,I71568560

* changes:
  Make android.hardware.photography.Camera work on HAL3+ devices
  Initial implementation of android.hardware.photography.CameraDevice (service)

merge in jb-mr2-zeroday-release history after reset to jb-mr2-dev

Merge remote-tracking branch 'origin/jb-mr2-zeroday-release' into HEAD

Workaround AudioRecord bug for large buffer sizes

Bug: 9556436
Change-Id: I92d1238b623d2cfd648e0a684d0e710fb0bd8b43

Use mFutex as an event flag rather than semaphore

An event flag can be more fault-tolerant in case of loss of synchronization,
as it cannot overflow.  It also allows more bits to be used in the future.
See http://en.wikipedia.org/wiki/Event_flag

Change-Id: I01ca25d951eb263124da54bb4738f0d94ec4a48b

Adds VP9 decoding support for stagefright.

Also change the VP8 encoder role name from
video_encoder.vpx to video_encoder.vp8 for
future VP9 encoder support.

Requires the change in frameworks/native and media_codecs.xml
corresponding to the device.

VP9 decoding test will be added to cts repo later.

Merge "stagefright: support for video decoder metadata mode"

stagefright: support for video decoder metadata mode

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

am e6105c26: (-s ours) Reconcile with jb-mr2-zeroday-release - do not merge

* commit 'e6105c26a5b506f96e9fe1d3fec64a8b2ed1d6dd':

Reconcile with jb-mr2-zeroday-release - do not merge

Change-Id: Id1c9edbb5b7b398fb863cbbd79e673ab2c10df9a

merge in jb-mr2-zeroday-release history after reset to jb-mr2-dev

am 455479bc: am 209bbbcf: Apply EQ and bass boost first in effect chain

* commit '455479bc2c6a88746d4801f3f5a0e8d9f8609528':
  Apply EQ and bass boost first in effect chain

am 209bbbcf: Apply EQ and bass boost first in effect chain

* commit '209bbbcf4190231f9dede758cbe77d109919f9f1':
  Apply EQ and bass boost first in effect chain

Make android.hardware.photography.Camera work on HAL3+ devices

Bug: 9213377
Change-Id: I5b2eeab28985f53dfcb7b8e3029930f5adcd74f5

Initial implementation of android.hardware.photography.CameraDevice (service)

* Verified preview streaming requests
* Other things *should* work but unverified / unimplemented in client side

Missing:
* CameraService needs to return static camera info metadata

Bug: 9213377
Change-Id: I71568560fcf18d0e2b408ed1c4d0066647314868

camera2/3: Warn on missing active array size parameters

Change-Id: I752659e1f0522392b902839cd7f4d997dcd85966

Apply EQ and bass boost first in effect chain

EQ and bass boost can improperly interact with other effects
 (e.g. virtualizer) outside of the bundle. Apply them first
 in an effect chain, which is equivalent to using them to affect
 the content of a track, rather than how a track is presented,
 when additional effects are cascaded behind EQ and BB.

Bug 8589316

Change-Id: Ifb34e46bc1969f613cdc90ee2a79af677e2fad92

am d3ac2fc0: am f3e21c30: am 60f3f102: Merge "Prevent AudioCommands being freed before read"

* commit 'd3ac2fc0c5429003f69d161e42bba7e94434ec09':
  Prevent AudioCommands being freed before read

am f3e21c30: am 60f3f102: Merge "Prevent AudioCommands being freed before read"

* commit 'f3e21c301b8fef49eecf9c716373fcbfbaa47670':
  Prevent AudioCommands being freed before read

am 60f3f102: Merge "Prevent AudioCommands being freed before read"

* commit '60f3f102ba2973cc31e5c7ea00d95bbf19968e8b':
  Prevent AudioCommands being freed before read

Merge "Prevent AudioCommands being freed before read"

Prevent AudioCommands being freed before read

When AudioCommandThread::threadLoop process AudioCommands it was
possible for it to delete a command where the posting thread still
hadn't read that status from it.

If a second command signaled the thread loop to continue after it
had inserted a new command while the thread loop was actually waiting
for the first command to report that it had read the status the
thread loop would continue and delete the first command.

Changed the wait condition when waiting for the calling thread to
read status to use command->mCond instead of mWaitWorkCV. This way
it's guaranteed that the signal to continue comes from the correct
thread.

Change-Id: Ia69b48cb4fdfaf8b4c83b56a197fb9f2058a92d1

am 9c8714da: am e6478de7: Camera2/3: Avoid shutdown race in callback processor.

* commit '9c8714daaf6a402abd85557e9b652c86f0df86e4':
  Camera2/3: Avoid shutdown race in callback processor.

am e6478de7: Camera2/3: Avoid shutdown race in callback processor.

* commit 'e6478de72a8ae1d3abb3e8c20898977c45354a4f':
  Camera2/3: Avoid shutdown race in callback processor.

Camera2/3: Avoid shutdown race in callback processor.

It's possible, during shutdown, for callback processor's heap to be
destroyed when it's about to send the callback back to the user.
Properly copy the heap reference to a local variable before unlocking
the mutex.

Bug: 9485959
Change-Id: I301347b77145f19c7ac721b9127dc74f122acce2

am bda5d1f2: am 241b5279: Camera3: Power: Properly shut down unused thread.

* commit 'bda5d1f2a34f9b7ded0bc0441be4856e78ec066c':
  Camera3: Power: Properly shut down unused thread.

am 241b5279: Camera3: Power: Properly shut down unused thread.

* commit '241b52798809d8db3d369af05ace1f73f723f29b':
  Camera3: Power: Properly shut down unused thread.

Merge "Speed up id3v2 unsynchronization"

Speed up id3v2 unsynchronization

Instead of doing many overlapping memmoves, do a single copy pass
that skips over the inserted unsynchronization bytes. For some
files this reduces parsing time from minutes to milliseconds.

b/9463262

Change-Id: I735b7051e77a093d86fb7a3e46209875946225ed

Camera3: Power: Properly shut down unused thread.

Bug: 9466221
Change-Id: I9ed0740561b2e04954eebc1b8aec89cfab40a3a0

Merge "The software avc decoder would silently drop output frames if not"

am 05e87d53: (-s ours) Reconcile with jb-mr2-zeroday-release - do not merge

* commit '05e87d536ed2b7cf1d68135121b630031e5d56c2':

Reconcile with jb-mr2-zeroday-release - do not merge

Change-Id: I2ab98a5fccea49e0f4e6d22fddca485ad50b63b8

merge in jb-mr2-zeroday-release history after reset to jb-mr2-dev

The software avc decoder would silently drop output frames if not

enough room was available in the output queue at the time they were
available. No more.

Change-Id: I5957290d40ba31bda7944271ec7f2aa0f1f7043c

am 0cc75491: am 26bc908b: Camera2/3: Make sure to wait for idle after still capture.

* commit '0cc754918a3c1353164ccc5798f63b28f65db406':
  Camera2/3: Make sure to wait for idle after still capture.

am e4a453d4: am 214a17fd: Camera3: Always close HAL device even in face of errors.

* commit 'e4a453d4b7563d1dab674df6ce52c291e63c6ca7':
  Camera3: Always close HAL device even in face of errors.

am 767227fc: am 82fc2ca6: am d5018116: Merge "Fix typo."

* commit '767227fcac7be843b338473954420d390e354bf7':
  Fix typo.

am 26bc908b: Camera2/3: Make sure to wait for idle after still capture.

* commit '26bc908b3dafb5d80f092a22c5c71116526eae06':
  Camera2/3: Make sure to wait for idle after still capture.

am 214a17fd: Camera3: Always close HAL device even in face of errors.

* commit '214a17fd37ef85fc841d3157b1e9096e1aa1b42f':
  Camera3: Always close HAL device even in face of errors.

am 82fc2ca6: am d5018116: Merge "Fix typo."

* commit '82fc2ca6905978b4f49f2c6bf948a6872575f2cc':
  Fix typo.

am d5018116: Merge "Fix typo."

* commit 'd5018116252a3ecb491e2eaddad12180b85d1e78':
  Fix typo.

Merge "Fix typo."

Camera2/3: Make sure to wait for idle after still capture.

While this wait should be immediate, it's important to guarantee
that preview is stopped when takePicture completes, and that the
device knows itself to be idle.

Bug: 9413025
Change-Id: I1c151ba920e226107ee048918187364990f444c0

Merge "AudioRecord must be used as sp<> only"

Merge "camera2: accept 4-value active array size"

AudioRecord must be used as sp<> only

Bug: 9423855
Change-Id: I78ba8228c60dff11fb466156bb632c5dda45cdaf

camera2: accept 4-value active array size

Change-Id: Ia320a022b201a938a025efc30a54c05fcfe5b02f

Camera3: Always close HAL device even in face of errors.

To improve robustness, make sure the HAL device instance is always
closed, even if it cannot be put into the IDLE state.

Bug: 9414167
Change-Id: I8e7035ff0afe7214951f9824e561b340cb37592e

am ccc366a9: am 1191454a: Merge "Camera3: Skipping no-ops good, not updating state machine bad." into jb-mr2-dev

* commit 'ccc366a90fdd8d9df5de27820f04bc534d122c07':
  Camera3: Skipping no-ops good, not updating state machine bad.

am 4541531a: am c4a8d2dd: Merge "Camera2/3: Create JPEG stream unconditionally." into jb-mr2-dev

* commit '4541531a6e64329764ad7359775689a8f5aca165':
  Camera2/3: Create JPEG stream unconditionally.

am 1191454a: Merge "Camera3: Skipping no-ops good, not updating state machine bad." into jb-mr2-dev

* commit '1191454a3a960a1972b0f36e625796c6c1b2c9ca':
  Camera3: Skipping no-ops good, not updating state machine bad.

am c4a8d2dd: Merge "Camera2/3: Create JPEG stream unconditionally." into jb-mr2-dev

* commit 'c4a8d2ddfff6d91e493692251733f20ece3e713d':
  Camera2/3: Create JPEG stream unconditionally.

Merge "Camera3: Skipping no-ops good, not updating state machine bad." into jb-mr2-dev

Merge "Camera2/3: Create JPEG stream unconditionally." into jb-mr2-dev

am a4539807: (-s ours) Reconcile with jb-mr2-zeroday-release - do not merge

* commit 'a45398078e4f798a34310a2d5213e07eefb14238':

Reconcile with jb-mr2-zeroday-release - do not merge

Change-Id: Id9419bf573d550f1ebebcc96fe615e1aaf8f36ef

merge in jb-mr2-zeroday-release history after reset to jb-mr2-dev

am a1d73d9b: (-s ours) Reconcile with jb-mr2-release - do not merge

* commit 'a1d73d9be0beafd10642d6db0136b16353a32096':
  audioflinger: fix effects on direct output threads

Reconcile with jb-mr2-release - do not merge

Change-Id: I024b04bf1e47cc7c10209e83f640cc6f12e03274

Camera3: Skipping no-ops good, not updating state machine bad.

If a HAL-level stream reconfigure is not needed, still need to
transition camera device to ACTIVE state in configureStreams.
Otherwise, shutdown functions will not wait for device to drain
properly.

Bug: 9413025
Change-Id: I0b15792097c7e5771015f2a362735d5f88b8c6ea

New control block for AudioTrack and AudioRecord

Main differences between old and new control block:
 - removes the mutex, which was a potential source of priority inversion
 - circular indices into shared buffer, which is now always a power-of-2 size

Change-Id: I4e9b7fa99858b488ac98a441fa70e31dbba1b865

audioflinger: fix effects on direct output threads

PlaybackThread::addTrack_l() uses the assumption that
effects are attached to a track only if the track accumulation
buffer is different from the mixer thread output buffer.
This is not true for direct output threads where only one track is active
an only one buffer is needed.
This assumption is an optimization to avoid checking for effect chains with
the same session ID each time a track is processed. The optimization
is not key if only one track is attached to the thread which is the case for
direct outputs.

Current code fails to increment the active track count in the effect chain
on direct output threads when a track is started thus making the effect
framework clear the mix buffer and produce silence each time the mixer runs.

The fix consists in removing the optimization described above.

Bug: 9324989.

Change-Id: Id7a6337450ed90d326299c2ce9fc02f4b9e2fa6f

Converter now takes the desired _output_ format instead of the

input format, allowing control over the type of encoding.

Change-Id: Iaaa1a825f447ea130e373bbd8e5dc96f2762db18

am f8af99b7: am 0662f7b6: Merge "stagefright: synchronously change nativewindow crop" into jb-mr2-dev

* commit 'f8af99b7c10f1441d9a52676724b168f20752270':
  stagefright: synchronously change nativewindow crop

am 0662f7b6: Merge "stagefright: synchronously change nativewindow crop" into jb-mr2-dev

* commit '0662f7b6735df0a140f6512285d05f8f9a38bb98':
  stagefright: synchronously change nativewindow crop

Merge "stagefright: synchronously change nativewindow crop" into jb-mr2-dev

am bb35be24: am 7be48189: Merge "audioflinger: fix effects on direct output threads" into jb-mr2-dev

* commit 'bb35be24239f05f59715960a93cd491a4cd3b4d6':
  audioflinger: fix effects on direct output threads

am 7be48189: Merge "audioflinger: fix effects on direct output threads" into jb-mr2-dev

* commit '7be48189c0aefb82825a7689d9e1005175570779':
  audioflinger: fix effects on direct output threads

Merge "audioflinger: fix effects on direct output threads" into jb-mr2-dev

am 765f99f3: (-s ours) Reconcile with jb-mr2-zeroday-release - do not merge

* commit '765f99f30fdc18396ce4edc9facbe6632c9b858e':

Reconcile with jb-mr2-zeroday-release - do not merge

Change-Id: I87a7986866714bfd922259cf961d6d5f5b752362