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

Change-Id: I13e45e13c3abf52cef3fb58de3f4707fb8fedaac

am c582fde9: resolved conflicts for merge of c158971f to stage-aosp-master

* commit 'c582fde93ded7219107157333a9e46d780adcf9c':
  Store rtsp accessunit until PLAY response parsed

resolved conflicts for merge of c158971f to stage-aosp-master

Change-Id: I3d77b86f7e616af62a826fc37126706ad8ff6158

Merge "Store rtsp accessunit until PLAY response parsed"

am 5de9ef20: am 365ed394: Merge "Typo in CameraParameters::set"

* commit '5de9ef20d596bc20f53cba664f1ac7e358910ea2':
  Typo in CameraParameters::set

am 365ed394: Merge "Typo in CameraParameters::set"

* commit '365ed394c4135d9d42082c85638c076aed4bd156':
  Typo in CameraParameters::set

Merge "Typo in CameraParameters::set"

am 5d2ab2a6: am 8eef093c: Merge "libstagefright: Use LOG_BASIC when use chromium http stack"

* commit '5d2ab2a6d5e90c045d0989ef1be6c66a2295ee2d':
  libstagefright: Use LOG_BASIC when use chromium http stack

am 06197925: am 267363ee: Merge "Prevent Media Server crash while the AwesomePlayer getBitrate return true with mBitrate equals to zero."

* commit '061979253985e59c4fcb82ede232717736e633b1':
  Prevent Media Server crash while the AwesomePlayer getBitrate return true with mBitrate equals to zero.

am 8eef093c: Merge "libstagefright: Use LOG_BASIC when use chromium http stack"

* commit '8eef093c213094c7efb655ef4d9e31439646bd7f':
  libstagefright: Use LOG_BASIC when use chromium http stack

am 7266bc80: am ca7f1609: Merge "Prevent double disconnects in RTSPSource"

* commit '7266bc80850dc327e9f18c27b81613f959399630':
  Prevent double disconnects in RTSPSource

Merge "libstagefright: Use LOG_BASIC when use chromium http stack"

am 267363ee: Merge "Prevent Media Server crash while the AwesomePlayer getBitrate return true with mBitrate equals to zero."

* commit '267363ee2836ff9bc18d7fdd93aaedf19de546ad':
  Prevent Media Server crash while the AwesomePlayer getBitrate return true with mBitrate equals to zero.

am ca7f1609: Merge "Prevent double disconnects in RTSPSource"

* commit 'ca7f160921a4a1a5da32ebe8fab1067bd8998a1f':
  Prevent double disconnects in RTSPSource

Merge "Prevent Media Server crash while the AwesomePlayer getBitrate return true with mBitrate equals to zero."

Merge "Prevent double disconnects in RTSPSource"

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

am eced745a: Merge "Part of fix for TEE resource leaks when app crashes" into jb-mr2-dev

* commit 'eced745af7d2d27a8c45e04eb071d4fb30cd3e48':
  Part of fix for TEE resource leaks when app crashes

Merge "Part of fix for TEE resource leaks when app crashes" into jb-mr2-dev

Part of fix for TEE resource leaks when app crashes

bug: 9672606
Change-Id: I98cf65260d5e43d34e3897452f6f26ed2c89294e

am f59a4b39: fix scheduling policy service death detection

* commit 'f59a4b393f4844c5bbc8d6212364bdddea33d232':
  fix scheduling policy service death detection

Prevent Media Server crash while the AwesomePlayer getBitrate return true with mBitrate equals to zero.

[Cause]
   - getBitrate(int64_t *bitrate) will return true if mBitrate = 0.
   - Then, in getCachedDuration_l(int64_t *durationUs, bool *eos),
     we might execute the following function when
     getBitrate(&bitrate) = true and bitrate = 0.
        *durationUs = cachedDataRemaining * 8000000ll / bitrate;
   - Mediaserver will be crashed when divided by zero.
   - The mediaserver crash often occur when DLNA IOP tests on Golden DMS
     with AAC_ADTS_320 audio.
[Solution]
   - Prevent the divide function executing when bitrate is zero.

Change-Id: I4439d92cee5faec95df2109e9186c33b3fff6c66

fix scheduling policy service death detection

Check status of transactions to scheduling policy service
and re-acquire a binder interface in case of DEAD_OBJECT.

Bug: 8875559.
Change-Id: I1e00bd44e2d4723b3ec95d5c31d9652ba08e238a

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

Change-Id: Idf0f0cdf02e6e94603ff003b6baf869b4414e1cf

Prevent double disconnects in RTSPSource

Prevent a null pointer from occuring by checking that we are not
already disconnected before performing a disconnect, in case
multiple disconnects have been issued.

Change-Id: Ib88800cd246086fd57f77d10748d41b1aefe2084

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

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 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

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>

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

Change-Id: I4de46f3bfcfb80867bbeb52a76d383f5870e74f4

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 in jb-mr2-zeroday-release history after reset to jb-mr2-dev

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

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 209bbbcf: Apply EQ and bass boost first in effect chain

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

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 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 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 241b5279: Camera3: Power: Properly shut down unused thread.

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

Camera3: Power: Properly shut down unused thread.

Bug: 9466221
Change-Id: I9ed0740561b2e04954eebc1b8aec89cfab40a3a0

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

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

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 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

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

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

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

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 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

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

Change-Id: I87a7986866714bfd922259cf961d6d5f5b752362

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

stagefright: synchronously change nativewindow crop

Only change the nativewindow crop before queuing frames
with the new crop.

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

am ea26c777: Camera3: Skip no-op stream configurations.

* commit 'ea26c7772f4721657db409068d4bed194ae49c94':
  Camera3: Skip no-op stream configurations.

Camera3: Skip no-op stream configurations.

If configuring the same set of streams more than once, don't actually
call into the HAL for the second and subsequent configure calls, since
they're no-ops.

This can speed up camera operation substantially if the
HAL implementation does not detect no-ops on its own and does a full
shutdown/restart on each configure call.

Bug: 9392513
Change-Id: I23baf4acbae2304735899adcf8e17565fa94d31d

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

Camera2/3: Create JPEG stream unconditionally.

Instead of creating the JPEG output stream only at first use,
create it at time of preview start. This is important for reducing
the first-capture latency on HAL3 devices, and for ZSL on them.

Bug: 9339858
Change-Id: I4296f706cfd151c47ef315149e87d01fe554aaa4

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

Change-Id: I9864de7c2785357cea0c4b669607fee76e9f09e8

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

camera2/3: Fix deadlock take picture while preview callback

Acquiring mInputMutex before SharedParameters mutex in preview callback thread
causes circular locking dependency between callback thread and capture callback
thread.
Enforce the right lock ordering to break the dead lock.

Bug: 9323319
Change-Id: Iea2e025e4d9e29debcf74297c19930075017e179

am ba696bbf: Merge "camera2/3: Fix deadlock take picture while preview callback" into jb-mr2-dev

* commit 'ba696bbff88d5627beaa0be95be78ba30138983d':
  camera2/3: Fix deadlock take picture while preview callback

Merge "camera2/3: Fix deadlock take picture while preview callback" into jb-mr2-dev

am 18b46679: Merge "AudioFlinger: do not cache audio device when 0." into jb-mr2-dev

* commit '18b466790aa0a55ff4282ed8b5e550e58e0444b2':
  AudioFlinger: do not cache audio device when 0.

Merge "AudioFlinger: do not cache audio device when 0." into jb-mr2-dev

am 911a184b: am 132bc5bd: Merge "Noise will be heard if audio sample rate not matched with audio track"

* commit '911a184bb5f5177a00bb4a8c80453f22e67b449c':
  Noise will be heard if audio sample rate not matched with audio track

am 132bc5bd: Merge "Noise will be heard if audio sample rate not matched with audio track"

* commit '132bc5bd1c68aa807a20e1655f009f74433be588':
  Noise will be heard if audio sample rate not matched with audio track

Merge "Noise will be heard if audio sample rate not matched with audio track"

Noise will be heard if audio sample rate not matched with audio track

When audio sample rate which set to audio track is not the same with
the actual pcm data, noise will be heard. Fix the bug when write 8 bit
pcm samples.

Change-Id: Idcb0d7b0e9aaa250dd22b758c8337e23d1706049
Signed-off-by: Ming Zhou <b42586@freescale.com>
Signed-off-by: guoyin.chen <guoyin.chen@freescale.com>

AudioFlinger: do not cache audio device when 0.

AudioFlinger should not cache the audio device
passed to the audio HAL when it is AUDIO_DEVICE_NONE
but keep previous valid selection instead.

Bug: 9323399.
Change-Id: I6f9480e55a21be4115453e6a5eebc5cf2536c476

camera2/3: Fix deadlock take picture while preview callback

Acquiring mInputMutex before SharedParameters mutex in preview callback thread
causes circular locking dependency between callback thread and capture callback
thread.
Enforce the right lock ordering to break the dead lock.

Bug: 9323319
Change-Id: Iea2e025e4d9e29debcf74297c19930075017e179

camera3: Fix crash in set error state.

When camera open fails in camera3device initialize call, setErrorStateLockedV()
tries to access RequestThread object that is not created yet.

Bug: 9319518
Change-Id: Id5c6c0150ecaecb198045d5c6eb1b2ceebf5a727

camera2/3: Add protection to callback processing call

Callback stream in callbackprocessor could be delleted while process or
discard callback are ongoing, which leads CpuConsumer related call crash.

Bug 9303911

Change-Id: Ib6137e512208e5603468eba166640729b744e8f9

am b05eeaed: camera3: Fix crash in set error state.

* commit 'b05eeaedacaff92b6e5ac89f99b0fccdf7643f09':
  camera3: Fix crash in set error state.