Merge "Camera2: Use CameraMetadata wrapper class." into jb-mr1-dev

Merge "Fixed a potential/rare race condtion in MediaPlayerService::Client::notify() method" into jb-mr1-dev

Merge "Fixed a native crash due to unexpected state at destroy time when encoder component's source fails to start" into jb-mr1-dev

Camera2: Use CameraMetadata wrapper class.

Wrap the use of the camera_metadata library inside a CameraMetadata
class to avoid memory leaks, improve code clarity, and increase type
safety.

Bug: 6243944
Change-Id: I90ca592b5875e27e4c58ea1ca680582dc3c3b56a

Merge "SurfaceMediaSource: a few fixes / cleanups" into jb-mr1-dev

SurfaceMediaSource: a few fixes / cleanups

- Remove the setting of mStopped to true in onBuffersReleased
- Cleaned up a few function call logs
- Removed the 'reset' method.

Change-Id: I56bab1d5cb2f3e6d59d54698c2e045d796d1849f

Fixed a potential/rare race condtion in MediaPlayerService::Client::notify() method

o The mClient could be disconnected before the notify message is sent out. Use
  a local reference could help resolve this issue.

o related-to-bug: 6782035

Change-Id: Ia2047bd7f2666100fed83c6e5417a3bf2e20cd2c

Merge "Fix deadlock" into jb-mr1-dev

Fix deadlock

I wanted to save some space, and ended up with a deadlock.
This change basically restores setDataSource(fd, offset, size) to the
way it was before I added the new setDataSource(DataSource) method,
instead of calling through to it.

Change-Id: Iacf4627c8745369fa84b467530189a9f64f8726b

Merge "Reduce THD when equalizer is ON." into jb-mr1-dev

Reduce THD when equalizer is ON.

Increase headroom in LVM bundle to reduce THD when equalizer is ON
with high band gains.

Also add a workaround for a bug in headroom parameter control.

Change-Id: I1d67c9ae475f6cf852c2e0e49bacfc82a5e484fd

Fixed a native crash due to unexpected state at destroy time when encoder component's source fails to start

o Encoder component initializes to be in the state of EXECUTING before its source gets started, because we wanted to be able
  to configure the source to use the advertised number of input buffers. However, if the source fails to start, then the encoder
  ends up in the state of EXECUTING when OMXCodec object gets destroyed. As a result, the assertion on the expected state in
  OMXCodec's constructor fails. The fix is to stop the video encoder component right way when its source fails to start so to
  bring the state of the encoder component back to the expected state.

o related-to-bug: 7045494

Change-Id: I6d4a221eb809d7137f53e58098a04816998f7a25

Merge "Allow directly setting a DataSource for NuMediaExtractor" into jb-mr1-dev

Allow directly setting a DataSource for NuMediaExtractor

This adds NuMediaExtractor::setDataSource(const sp<DataSource> &source),
which some new code under frameworks/av will use to let applications
provide a custom data source for extractors.

Change-Id: I8fb743441d04b4eadbda3c2d76858c3d60e2a3b2

Merge "Add a missing NULL check." into jb-mr1-dev

am 07315f82: am 4b60ac04: Merge "Fixed redefinition of \'i\' with a different type error"

* commit '07315f8262f843f3624c70974642e4fb733a661c':
  Fixed redefinition of 'i' with a different type error

am 78015984: am 6e55a0f7: Merge "Fixed member access into incomplete type build error"

* commit '780159840d754eae8e4e3f693ea10932634053f8':
  Fixed member access into incomplete type build error

Add a missing NULL check.

Don't crash if its time to start the next media player in a media
player chain, but the current player's client has a NULL mAudioOutput,
as is the case when the low level player implements
MediaPlayerHWInterface.

Change-Id: If555da3a46960cbee91ff26e5b1fd05e52c8ef28

Merge "Send current bandwidth info event when buffering occurs" into jb-mr1-dev

am 4b60ac04: Merge "Fixed redefinition of \'i\' with a different type error"

* commit '4b60ac04a878eaf8344b56189509ccd6925855c2':
  Fixed redefinition of 'i' with a different type error

am 6e55a0f7: Merge "Fixed member access into incomplete type build error"

* commit '6e55a0f71eef07c6041077e6e9e7fad8c973a752':
  Fixed member access into incomplete type build error

Merge "Fixed redefinition of 'i' with a different type error"

Merge "Fixed member access into incomplete type build error"

Send current bandwidth info event when buffering occurs

There is a dependent change in vendor/widevine repo.

Change-Id: If063f4b1c8857e5bfc7be54ab0f97a4ed169b303
related-to-bug: 5883234

Merge "Let getOriginalMimeType() take a fd passed from drm java applications" into jb-mr1-dev

Fixed redefinition of 'i' with a different type error

Renamed the redefinition of 'i' to 'k'.

Change-Id: I8ae8ed1dee42c896fa15e55be8d58b33b29254f4
Author: Tareq A. Siraj <tareq.a.siraj@intel.com>
Reviewed-by: Kevin P Schoedel <kevin.p.schoedel@intel.com>

Fixed member access into incomplete type build error

Included the ARTPAssembler.h file to fix the 'member access into
incomplete type "android::ARTPAssembler"' error reported by clang.

Change-Id: I10cb1e38bf360858bb7ebdeae82ba1e64431f87d
Author: Tareq A. Siraj <tareq.a.siraj@intel.com>
Reviewed-by: Edwin Vane<edwin.vane@intel.com>

Make tee sink compile OK again

Change-Id: Ie865ceeabaeb698a62c11eac9f96e303c44c28ae

Camera2: Replace MediaConsumer with BufferItemConsumer.

Get rid of MediaConsumer, which was largely duplicated code, and
replace it with the simpler BufferItemConsumer.

Bug: 6243944
Change-Id: I242d80c5fe39f2ee581ec8bb46f362997d994b0a

Camera2: Ignore NOT_FOUND errors for face detect mode on frame receipt.

Reduce log spamming due to incomplete implementations.

Change-Id: I6eb9d657591bd5f1476dd8c96e4a00b8a03e2a5e

am 8d4d590d: am 1e75e972: Merge "avcenc: Update a comment about the inline assembly"

* commit '8d4d590dcbbd2b178976d0bd024625e9d6216365':
  avcenc: Update a comment about the inline assembly

am 8a825ec0: am aa411adf: Merge "h264 streaming: make profile-level-id optional"

* commit '8a825ec079da6f2b89149c53568cb9721c10d923':
  h264 streaming: make profile-level-id optional

am 1e75e972: Merge "avcenc: Update a comment about the inline assembly"

* commit '1e75e9721254ae5df05c3d88b30ed7f778406261':
  avcenc: Update a comment about the inline assembly

am aa411adf: Merge "h264 streaming: make profile-level-id optional"

* commit 'aa411adfc39a4ee6c4ff57952296ba40c2fbfdb3':
  h264 streaming: make profile-level-id optional

Merge "avcenc: Update a comment about the inline assembly"

Merge "h264 streaming: make profile-level-id optional"

am 41ee747f: am 5ebce52c: Merge "avcenc: Fix the gcc inline assembly constraints"

* commit '41ee747fc019eb5856454c698adf23f55073c69e':
  avcenc: Fix the gcc inline assembly constraints

am 894a7e45: am 9e13e927: Merge "avcenc: Properly indent assembly blocks"

* commit '894a7e450704b0c7858a2187a5432b479ac69c1d':
  avcenc: Properly indent assembly blocks

am 5ebce52c: Merge "avcenc: Fix the gcc inline assembly constraints"

* commit '5ebce52c46360b349daa8c944ca81e3c74accec0':
  avcenc: Fix the gcc inline assembly constraints

am 9e13e927: Merge "avcenc: Properly indent assembly blocks"

* commit '9e13e927206b7c7c15359af2b0ccc8f525e78a7d':
  avcenc: Properly indent assembly blocks

Merge "avcenc: Fix the gcc inline assembly constraints"

Merge "avcenc: Properly indent assembly blocks"

avcenc: Update a comment about the inline assembly

Since the inline assembly was fixed in commit 3fdb405597f the
comment was no longer accurate.

Change-Id: Id4f8a111d1fa6e0c2e97f7db64262e8d1604c519

Merge "Camera2: Add support for preview callbacks" into jb-mr1-dev

Camera2: Add support for preview callbacks

- Move camera overall state enum to be part of parameters
- No longer use ICameraLock to guard access to overall state enum
- Add callback flag management
- Add callback stream and management

Bug: 6243944

Change-Id: Ideb3dc523a23c05e440e429ebf838b1900a96573

avcenc: Fix the gcc inline assembly constraints

Use +r for registers whose value is used as input to the block.

Also switch input registers to output registers with +r, for
registers that are modified by the assembly block (SUB_SAD in
sad_inline.h).

Without this, the inline assembly fails on modern gcc versions
(tested on gcc 4.6), where the intended input value never is
loaded into the registers, causing segfaults at runtime.

Currently this assembly isn't used, but this change makes it
work properly if enabled.

Change-Id: I37c247cd5c8aff213445e31ab3bd33d8f0f37e05

avcenc: Properly indent assembly blocks

Also line break multiline assembly blocks - previously they were
virtually unreadable.

Change-Id: Icb269909b78746e26b28ab7dcb6979c4655a0b0c

Merge "Revert "Swap the order of creating IAudioTrack and thread"" into jb-mr1-dev

am add7c4d3: am 46b7eeb8: Merge "hardening: eliminate text relocations from h264"

* commit 'add7c4d329586119fb8b5501b049905004400233':
  hardening: eliminate text relocations from h264

am 46b7eeb8: Merge "hardening: eliminate text relocations from h264"

* commit '46b7eeb8be7449515082ecb1a61700affb44115f':
  hardening: eliminate text relocations from h264

Merge "hardening: eliminate text relocations from h264"

am af5f9f05: am cd622346: Merge "MPEG2TSWriter: Fix an off-by-one in the condition for padding"

* commit 'af5f9f05937521acb4855848d77bed1d4ed7db0a':
  MPEG2TSWriter: Fix an off-by-one in the condition for padding

am 3b939a5e: am 02a4ec21: Merge "avcenc: Remove a leftover define"

* commit '3b939a5e893ccfe87f5b7b3fe229439a9b2fc87a':
  avcenc: Remove a leftover define

am cd622346: Merge "MPEG2TSWriter: Fix an off-by-one in the condition for padding"

* commit 'cd622346235829f60246a061560adf5921ba129c':
  MPEG2TSWriter: Fix an off-by-one in the condition for padding

am 02a4ec21: Merge "avcenc: Remove a leftover define"

* commit '02a4ec216cf5ca903e1765ca3e7ff2985436b236':
  avcenc: Remove a leftover define

Merge "MPEG2TSWriter: Fix an off-by-one in the condition for padding"

Merge "avcenc: Remove a leftover define"

MPEG2TSWriter: Fix an off-by-one in the condition for padding

This makes a remuxed file play without any decoding warnings,
while there still were some occasional warnings with the previous
version.

Change-Id: I31d73446ded1eeb3da10783c5ef841270563ca12

h264 streaming: make profile-level-id optional

profile-level-id is made optional according to rfc3984:
"If no profile-level-id is present, the Baseline Profile without
additional constraints at Level 1 MUST be implied."

Change-Id: If868468a48917ceccb963b8ac15767583da29723

Don't log streaming url.

o related-to-bug: 6994761

Change-Id: I79d5549a70b3bde67ddf496cfc6283ac92b1f569

avcenc: Remove a leftover define

This should have been removed as part of f5af6314db, but
the second __arm__ define was introduced after the original commit
was written, and not removing the second one was an oversight in
rebasing the patch on top of the new code.

Change-Id: Iccf08ffdef130446dd6e97dc1e15e6a1d4f9386f

am 523c09d5: am 64b3fe55: Merge "stagefright amrnb: Remove unused leftover source files"

* commit '523c09d56c78933eea7fb4f028e0bfa595ca82b1':
  stagefright amrnb: Remove unused leftover source files

am f20e4ad9: am c1d2777c: Merge "stagefright amrnb: Move the bitstream_format enum into frame_type_3gpp.h"

* commit 'f20e4ad9f23fe0267f8f3434a6eb961eb69bde3a':
  stagefright amrnb: Move the bitstream_format enum into frame_type_3gpp.h

am 64b3fe55: Merge "stagefright amrnb: Remove unused leftover source files"

* commit '64b3fe55f7edd16be32e07c28c2c90f84f3b9b80':
  stagefright amrnb: Remove unused leftover source files

am c1d2777c: Merge "stagefright amrnb: Move the bitstream_format enum into frame_type_3gpp.h"

* commit 'c1d2777c8004a9491e1381cca33a6768faed50c3':
  stagefright amrnb: Move the bitstream_format enum into frame_type_3gpp.h

Merge "Camera2: Fix SCENE_MODE" into jb-mr1-dev

Camera2: Fix SCENE_MODE

Now can set scene mode back to auto.

Bug: 7001465

Change-Id: Ia25ba54d13de662bef5bf79957987513a7313c18

Merge "stagefright amrnb: Remove unused leftover source files"

Merge "stagefright amrnb: Move the bitstream_format enum into frame_type_3gpp.h"

am 0df213ba: am dde79acf: Merge "hardening: eliminate text relocations from mp3"

* commit '0df213babc4fb41b7daa05cb3c08071f2b60f6a5':
  hardening: eliminate text relocations from mp3

am a109e9d5: am b7ca7dc9: Merge "avcenc: Clarify the ifdefs surrounding inline assembly"

* commit 'a109e9d5d5dbdbbd63a3f32fce131c718f0f8c99':
  avcenc: Clarify the ifdefs surrounding inline assembly

am fba9d50d: am 4f981882: Merge "hardening: eliminate text relocations from amrwb"

* commit 'fba9d50d1b2e487bc08b59ec93d7ed3183c76dbe':
  hardening: eliminate text relocations from amrwb

am 1191287b: am c3bfa026: Merge "aacenc/amrwbenc: Remove the now unused LINUX define"

* commit '1191287bc438c164a89cf86704936aeddbefb136':
  aacenc/amrwbenc: Remove the now unused LINUX define

am eeb9fde0: am 3f5d2e17: Merge "aacenc/amrwbenc: Remove unused and dubious typedefs"

* commit 'eeb9fde0cb8e0ace44eb720bcac272ee70282d4d':
  aacenc/amrwbenc: Remove unused and dubious typedefs

am dde79acf: Merge "hardening: eliminate text relocations from mp3"

* commit 'dde79acfb97083d690e3cceabff78931b10d1539':
  hardening: eliminate text relocations from mp3

am b7ca7dc9: Merge "avcenc: Clarify the ifdefs surrounding inline assembly"

* commit 'b7ca7dc9652456181080bf28ec376bc5a36cf6a2':
  avcenc: Clarify the ifdefs surrounding inline assembly

am 4f981882: Merge "hardening: eliminate text relocations from amrwb"

* commit '4f981882da8080e1d41d000bfbad92df61adcfba':
  hardening: eliminate text relocations from amrwb

am c3bfa026: Merge "aacenc/amrwbenc: Remove the now unused LINUX define"

* commit 'c3bfa0260a530af3e9c091c1c9d90d1283021dd6':
  aacenc/amrwbenc: Remove the now unused LINUX define

am 3f5d2e17: Merge "aacenc/amrwbenc: Remove unused and dubious typedefs"

* commit '3f5d2e1777bcf8856e966b93b2bce0b2c9cd1a49':
  aacenc/amrwbenc: Remove unused and dubious typedefs

Merge "hardening: eliminate text relocations from mp3"

Merge "avcenc: Clarify the ifdefs surrounding inline assembly"

Merge "hardening: eliminate text relocations from amrwb"

Merge "aacenc/amrwbenc: Remove the now unused LINUX define"

Merge "aacenc/amrwbenc: Remove unused and dubious typedefs"

aacenc/amrwbenc: Remove the now unused LINUX define

This is no longer used by the codec code (except for the
amrwbenc sample).

Change-Id: I78c7f62f257c490bf44477217975a93abcc7779e

aacenc/amrwbenc: Remove unused and dubious typedefs

These typedefs aren't used anywhere, and their description
doesn't really match their actual definition. The VO_TCHAR
type is even defined as pointer for one configuration and
as non-pointer for another configuration.

This gets rid of the last use of the LINUX define in the
VO codecs (except for the amrwbenc sample application).
Within the codecs, the use of this define is unclear - does it
indicate code specific to android but not normal linux, linux
but not other unixes, unix but not windows, or gcc but not
other compilers, etc.

Originally, prior to cleanup, the define has meant many
different of these - these uses has now been cleaned up to
use the proper feature detection macros (e.g. looking for
defines automatically set by the compiler, instead of
relying on manually setting this define).

Change-Id: I1a675d653d6d305f558aa7f74c7255858dc62758

am add9f820: am 2c3d9b7b: Merge "MPEG2TSWriter: Correctly add padding using the adaptation field"

* commit 'add9f820eaeb180b5e72a4420995a54062cfabd7':
  MPEG2TSWriter: Correctly add padding using the adaptation field

am 2c3d9b7b: Merge "MPEG2TSWriter: Correctly add padding using the adaptation field"

* commit '2c3d9b7b9e34c9122c0a568d6b1fb24404021f55':
  MPEG2TSWriter: Correctly add padding using the adaptation field

Merge "MPEG2TSWriter: Correctly add padding using the adaptation field"

MPEG2TSWriter: Correctly add padding using the adaptation field

Change-Id: I268cce36ada910ba168c4c9dcedab0b20f6e6ebb

Merge "Camera2: Skeleton for output frame processing, plus face detect" into jb-mr1-dev

Camera2: Skeleton for output frame processing, plus face detect

- Plumbing for processing output metadata frames from the HAL
- Support for passing face detection metadata from said frames
  to the application.
- Switch calls on ICameraClient interface to use separate mutex to
  avoid deadlock scenarios with messages being communicated from the
  HAL to the camera user while calls from the user to the service are
  active.

Bug: 6243944

Change-Id: Id4cf821d9c5c3c0069be4c0f669874b6ff0d1ecd

Merge "Fix AAC DRC metadata being ignored" into jb-mr1-dev

Fix AAC DRC metadata being ignored

The decoder will bypass all DRC processing during decode unless
  any of the DRC parameters is set.
The fix consists in setting the DRC reference level to its
  default value when after the decoder is initialized.

Change-Id: Ibb8355ab8c3c65020c5eef0752957040f7316c2a

am 979df012: am 98159acd: Merge "amrnb: Restart the p_xn pointer when retraversing the array on overflow"

* commit '979df012eb0588ac7f87d5f6e8bff77e2049ee46':
  amrnb: Restart the p_xn pointer when retraversing the array on overflow

am 98159acd: Merge "amrnb: Restart the p_xn pointer when retraversing the array on overflow"

* commit '98159acd68307bf0fcec40ae0fffc1a715517d1d':
  amrnb: Restart the p_xn pointer when retraversing the array on overflow

Merge "amrnb: Restart the p_xn pointer when retraversing the array on overflow"

hardening: eliminate text relocations from h264

Change-Id: I8bdf9746f28f2f7bed6c1109c0f4badbab765a7c

hardening: eliminate text relocations from mp3

Change-Id: I671ed4c8c7bb15172a8b64d513e2844319495102

hardening: eliminate text relocations from amrwb

Change-Id: I47235fc4afe6b916da1e999401fb3c8b12c94e62