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

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

Merge "avcenc: Properly indent assembly blocks"

avcenc: Properly indent assembly blocks

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

Change-Id: Icb269909b78746e26b28ab7dcb6979c4655a0b0c

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

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 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 "stagefright amrnb: Remove unused leftover source files"

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

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

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

am 5efa7bf9: Merge "amrnb: Fix audio distortion in an AMR-NB clip"

* commit '5efa7bf945f53c419501b0a4f9718962f24f0df5':
  amrnb: Fix audio distortion in an AMR-NB clip

Merge "amrnb: Fix audio distortion in an AMR-NB clip"

am 24893fef: Merge "MPEG2TSWriter: Write a proper CRC in PAT and PMT"

* commit '24893fef8e0f816a8a21749fa3eb9eed9173a0fd':
  MPEG2TSWriter: Write a proper CRC in PAT and PMT

Merge "MPEG2TSWriter: Write a proper CRC in PAT and PMT"

am 2001d2ac: Merge "MPEG2TSWriter: Write proper continuity counters for PAT and PMT"

* commit '2001d2ac01e36c13c1abda5f73b6404cba2648e1':
  MPEG2TSWriter: Write proper continuity counters for PAT and PMT

am c64a1787: Merge "Fix initialization of audio mixer track resources for MIPS."

* commit 'c64a178711bf7bb9b71d65a47156b34083311392':
  Fix initialization of audio mixer track resources for MIPS.

Merge "MPEG2TSWriter: Write proper continuity counters for PAT and PMT"

Merge "Fix initialization of audio mixer track resources for MIPS."

am e28df5b9: Merge "Store correct time stamps in recorded mp4 files"

* commit 'e28df5b996af3480b457c7a9908dbb9d52ad8d7c':
  Store correct time stamps in recorded mp4 files

Merge "Store correct time stamps in recorded mp4 files"

am efee0095: Merge "Correct ID3::StringSize calculation for UCS-2 data."

* commit 'efee009528f446542d51f1ce50e2189e8a2f71dd':
  Correct ID3::StringSize calculation for UCS-2 data.

am a858aa96: Merge "avcenc: Remove redundant zero-initialization of allocated memory"

* commit 'a858aa963d2a7e767d0d8694e29cb6fe87c9ef9f':
  avcenc: Remove redundant zero-initialization of allocated memory

am b71298ee: Merge "avcenc: Initialize all memory allocated by the CBAVC_Malloc callback function"

* commit 'b71298eeeb16d942dd5144e550731e6e29b6436e':
  avcenc: Initialize all memory allocated by the CBAVC_Malloc callback function

am 0ab18f85: Merge "avcenc: Remove useless casts"

* commit '0ab18f853571149de700930fbb783ecd6b2c5582':
  avcenc: Remove useless casts

am 67f694d6: Merge "Initialize pool\'s buffer pointers to null."

* commit '67f694d6b4234ea0d7ce2f8be8e6a9cc3b104eb9':
  Initialize pool's buffer pointers to null.

am 0822156c: Merge "avcenc: Fix indentation"

* commit '0822156cfd25c111cd33932ee9a89b3100d0350c':
  avcenc: Fix indentation

am 874a074a: Merge "avcenc: Switch malloc/free callbacks to use pointers instead of ints"

* commit '874a074ad88117d9fabf178b5b4a1c885dc512f5':
  avcenc: Switch malloc/free callbacks to use pointers instead of ints

am b7ee9c44: Merge "mediacodec: Return an error on getOutputFormat if there is no output format yet"

* commit 'b7ee9c4436c50cc5ff6e72059bf27c2c16dc1b92':
  mediacodec: Return an error on getOutputFormat if there is no output format yet

Merge "Correct ID3::StringSize calculation for UCS-2 data."

Merge "avcenc: Remove redundant zero-initialization of allocated memory"

Merge "avcenc: Initialize all memory allocated by the CBAVC_Malloc callback function"

Merge "avcenc: Remove useless casts"

Merge "Initialize pool's buffer pointers to null."

Merge "avcenc: Fix indentation"

Merge "avcenc: Switch malloc/free callbacks to use pointers instead of ints"

Merge "mediacodec: Return an error on getOutputFormat if there is no output format yet"

Fix initialization of audio mixer track resources for MIPS.

The value 1 << 32 (maxNumTracks defaults to 32) is surprisingly not defined
in C, and differs on MIPS than the other arch.

Therefore the track resources were not initialized properly, resulting
in failure to play any audio.

The fix allows the mConfiguredNames bitmask to be correctly set to all
1's in the 32-track case, for all arch.

Change-Id: Ied3e1305952e9567602e2cd76c5ef3acb0809ee0
Signed-off-by: Paul Lind <plind@mips.com>

am ffd99cb7: Merge "Shutter sound is heard at the end of the recorded video"

* commit 'ffd99cb795c92caee7f4bd0b26af9ae37b100fc3':
  Shutter sound is heard at the end of the recorded video

Merge "Shutter sound is heard at the end of the recorded video"

Store correct time stamps in recorded mp4 files

According to the specification (ISO_IEC_14496-12_2008 page 17, 19 and
21) the time is "in seconds since midnight, Jan. 1, 1904, in UTC time"
which is not the same as Unix epoch time. This is already correctly
handled in MPEG4Extractor.cpp when reading files.

The specification also supports 64 bit timestamps which would make
it a bit more future proof, but unfortunately it seems like support
for this in players are poor (tested both Windows media player and
Quicktime player).

Change-Id: I102d728f098b8b0a21013956f017b0c4f2d69112

Reconcile with jb-release

Change-Id: Iab9b19384ab6706f1dfca89f21ffc142f47225a3

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

Reconcile with jb-mr0-release

Change-Id: I783dd9fbd723bb284932123dfa9e06fb7c7ab879

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

merge in jb-mr0-release history after reset to jb-dev

am 65e8d62a: Merge "Fixed sample video corrupted in MediaCodec mode." into jb-dev

* commit '65e8d62a5b243518795cdafa498df4a49baf6940':
  Fixed sample video corrupted in MediaCodec mode.

Merge "Fixed sample video corrupted in MediaCodec mode." into jb-dev

Fixed sample video corrupted in MediaCodec mode.

Change-Id: I50c5660072d40f6e297e58fd026ea04c89d6a70a
related-to-bug: 6732061

mediacodec: Return an error on getOutputFormat if there is no output format yet

This avoids crashing if calling the public java method
MediaCodec.getOutputFormat immediately after MediaCodec.start
(as the documentation example shows that one should do!).

Change-Id: Icd3dd7ab9ae9e8f2a67cef171ff176735de95f58

avcenc: Remove redundant zero-initialization of allocated memory

The allocation callback is now supposed to always return initialized
memory.

Change-Id: I7a4d036e498c9f87c730dda3aab5057eb764a1e7

avcenc: Initialize all memory allocated by the CBAVC_Malloc callback function

Valgrind reported use of uninitialized memory in AVCEncodeSlice,
which this fixes.

Change-Id: Ia09ad3e50d05b2b0487a4d588d9b00fc0828c816

Initialize pool's buffer pointers to null.

If these pointers are not properly initialized to null and
allocation of one of the buffers subsequently fail, the freePool
mechanism might erroneously free uninitialized pointers.

Change-Id: I2101bdb2cc576e5f50b5a1440e6fc0c54dc4e81b

Correct ID3::StringSize calculation for UCS-2 data.

Album art was not displayed when albums were transferred via MediaGo
because the length of the description in the APIC tag was not
correctly calculated.

The method StringSize used to calculate the length of the tag
does not add the size of the null termination in UCS-2 case.

Change-Id: I5409d42f5e87d315e6e03c4d7e6cbd8a378a160d

avcenc: Fix indentation

Change-Id: I4fcdf619e3b75387a7909272f3164b8d608b6cc2

Shutter sound is heard at the end of the recorded video

On fast hardware, which is more common, the shutter sound is
played and recorded before video recording is stopped.

Before we call play shutter sound before calling stop recording.
With this fix we call play shutter sound after calling stop recording.

Change-Id: I06a3e65fd2700f1d2457fea5ff0f64500eac436f

avcenc: Clarify the ifdefs surrounding inline assembly

Previously, the code was structured like this:

#if defined(__GNUC__) && defined(__arm__)
    /* Generic C implementation */
#elif defined(__CC_ARM)
    /* RVCT version */
#elif defined(__GNUC__) && defined(__arm__)
    /* GCC arm assembly version */
#endif

This had two implications - the gcc arm version never was used
in practice, and the code required -D__arm__ to build at all for
other architectures.

The inline assembly constraints are buggy (requires changes so as
to not crash), and if fixed, the optimizations still aren't faster
than the generic C code on modern gcc versions. Therefore, just
ignore them and use the generic C code.

Due to the peculiar code block ordering and preprocessor conditions,
the generic C code is what has been used before anyway, but it
did require -D__arm__ to build, which can cause problems if building
for other architectures, if the system headers have ifdefs for this
define. This change makes it build without that define.

Change-Id: Ib3fcf5651ee720310b7dff10fd04279b5f90759c

avcenc: Remove useless casts

Change-Id: I73a6f74268bcf6ca22d489d13936be8fd1b3fc92

avcenc: Switch malloc/free callbacks to use pointers instead of ints

There is no reason for casting the pointers to ints. This fixes
building the code on platforms where pointers are larger than ints,
e.g. 64 bit platforms.

Change-Id: I910cd207d0908287931c9a96eb270139967e029b

stagefright amrnb: Remove unused leftover source files

This allows getting rid of adding the plain common directory
(not common/include) to the include paths in projects that use
this code, too.

Change-Id: Ifb1fe4fc1f5c6f1264cb2b64f7b78e42b1e7833a

stagefright amrnb: Move the bitstream_format enum into frame_type_3gpp.h

Change-Id: I6d6f9dc5799750394a853325466ffa2ebec44625

amrnb: Fix audio distortion in an AMR-NB clip

This fix had the PacketVideo internal tracking number
RIO-9190.

This was committed on the master branch of the opencore
repository in commit 7d6ccc907d3a15ca2ff74b9c76f122ab87d0ffbe.

Change-Id: Icbd7d6fe94e02cae360946cfb6f3b8dc90c8b518

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

This is a safeguard to avoid issues in a rare codepath.

This was committed on the master branch of the opencore
repository in commit f532d145194e474cb30d8644b8eee83873ad24cc,
based on AOSP contribution 10905.

Change-Id: I9688b009bcbe63d88ee5040e883a021aefa30664

MPEG2TSWriter: Write a proper CRC in PAT and PMT

Change-Id: I6ec8039b176878da6f02691194d96b0a1e894055

MPEG2TSWriter: Write proper continuity counters for PAT and PMT

Change-Id: If94e31e51c563867db6d2376ad771f9751416084

Reconcile with jb-release

Change-Id: I7c52b8e82465f28b32e8ebf6df56c258458f2cd3

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

am 109347d4: audioflinger: fix regression in attachAuxEffect().

* commit '109347d421413303eb1678dd9e2aa9d40acf89d2':
  audioflinger: fix regression in attachAuxEffect().

audioflinger: fix regression in attachAuxEffect().

Commit 717e1286 introduced a regression in PlaybackThread::Track::attachAuxEffect()
when called with an effect ID of 0 to detach the auxiliary effect.

It is normal in this case that AudioFlinger::getEffectThread_l() returns 0.

Bug 6768757.

Change-Id: I7430bd1aad2f68da38f7c3e4794e7ad657bfc6be

am 717e1286: audioflinger: fix auxiliary effect attachment

* commit '717e128691f083a9469a1d0e363ac6ecd5c65d58':
  audioflinger: fix auxiliary effect attachment

audioflinger: fix auxiliary effect attachment

Auxiliary effects (Reverb) are global effects and as such follow
the default rule which is to attach them to the output thread that
handles music streams by default. This causes a problem when several
threads are eligible to handle music streams as tracks can be attached
to either thread based on criteria unknown when teh effect is created.

The fix consists in moving the auxiliary effect if necessary when an
AudioTrack is attached to it and this track is not on the same
output thread.

Bug 6608561.

Change-Id: Ib32c3cabc731b2046aba728be1771982999c6069

Reconcile with jb-release

Change-Id: I1ce46fa6332f684c76c276117c7b406bdd5e9336

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

am 33e28dd3: Merge "SurfaceMediaSource: keep refs to current buffers" into jb-dev

* commit '33e28dd3c033641f4db9d1b99c7c88d6d08c61f7':
  SurfaceMediaSource: keep refs to current buffers

Merge "SurfaceMediaSource: keep refs to current buffers" into jb-dev

SurfaceMediaSource: keep refs to current buffers

This change fixes a bug in SurfaceMediaSource where it would not keep a
reference to all of the Gralloc buffers that the video encoder is currently
using.

Bug: 6655597
Change-Id: Ifd99976cc7ae57ed724bbf2bbcff357a39497391