Merge change 5482 into donut

* changes:
  Fix crash in PMEM heap destructor due to sequencing issue. Bug 1919195.

Fix crash in PMEM heap destructor due to sequencing issue.
Bug 1919195.

Merge change 5387 into donut

* changes:
  When OpenCore 2.0 was integrated, it broke the audio stream MIO. This patch signals the observer so that playback starts correctly. Bug 1837526

Merge change 4836 into donut

* changes:
  OMX M4V encoding drops the first I frame

OMX M4V encoding drops the first I frame

When OpenCore 2.0 was integrated, it broke the audio stream MIO.
This patch signals the observer so that playback starts correctly.
Bug 1837526

NJ-1425: Add AMR set bitrate support.

Modify camera MIO to use new camera callback interface.
This removes the discrete callbacks and replaces them with two common
callback methods.
Bug 1884362

Don't access queue element without checking its size (bug 1922537)

AStyle change for OpenCORE.

1850579 Fix the valgrind errors reported when running Simulator

Follow-up of NJ-1422 : Fixing PlayerDriver and AuthorDriver to catch any leaves
which happens in player or author, during playback or recording. This is to avoid
a leave to get transferred to upper layers, resulting in calling abort and crashing.

Merge change 4526 into donut

* changes:
  Log null author command in authordriver.cpp (bug 1922537) Note this does not resolve the root cause for the mediaserver crash reported in bug 1922537: 1. why was a null author command received from the author engine (PV is looking into this issue)? 2. why was the time stamp out of the order (QCOM and PV are looking into this issue)? 3. why did we receive multiple event callbacks for a single state transition request (QCOM is looking into this issue)?

Log null author command in authordriver.cpp (bug 1922537)
Note this does not resolve the root cause for the mediaserver crash reported in bug 1922537:
1. why was a null author command received from the author engine (PV is looking into this issue)?
2. why was the time stamp out of the order (QCOM and PV are looking into this issue)?
3. why did we receive multiple event callbacks for a single state transition request (QCOM is looking into this issue)?

[1841509]  Fails to play the audio with the attached video
For the content, size of the audio config buffer exceeds the size of the audio config data
because of which the extra bits were taken as a part of next raw aac stream, which resulted in
bit stream corruption.
Fix, force to consume all bits of the audio config buffer.

Don't ignore compiler warnings (bug fix: 1315153)

http://b/issue?id=1267355
NJ-1090 : disable HEAD request for progressive streaming or download.

Perform some checks on the Xing header, and reject it if it looks funny,
which causes the code fall back to another code path to determine the
length of the file. This makes it so that NPR podcasts don't show up
as being 13 seconds long.

Revert "Perform some checks on the Xing header, and reject it if it looks funny,"

This reverts commit 2c6f649b82e91bda20648083b3b4cef25b7c792c.

Perform some checks on the Xing header, and reject it if it looks funny,
which causes the code fall back to another code path to determine the
length of the file. This makes it so that NPR podcasts don't show up
as being 13 seconds long.

Merge change 3821 into donut

* changes:
  Send Reset() command to PV player engine no matter CancelAllCommands() succeeded or not.

Send Reset() command to PV player engine no matter CancelAllCommands() succeeded or not.

Merge change 3680 into donut

* changes:
  Don't export DSO symbols from camera mio and audio mio for authoring. The code size reduction is about 12 k for libopencore_author.so.

Merge change 3734 into donut

* changes:
  Cleanup value add in makefile.

Fix the stop handling when the writer thread is used.

The flush operation is not the only way to stop the composer.
The other way is by reaching a limit (size or time), in which case
an event is reported to the author engine AND the file is flushed.

This is an issue because the flush call on the writer thread now needs
to be reentrant (not called from the main thread anymore).

To move the call back on the main thread we use an extra field to hold
the event that should be fired. On the next fragment, the event is sent
and the DoStop method will be run from the main thread as a result.

Added extra code to make sure the worker thread flushes its buffer queue.
Added a bool to remember that exit was called. exitPending form Thread
gets cleared before the dtor is invoked.
In the flush method, don't wait if the flush happened.

Don't export DSO symbols from camera mio and audio mio for authoring.
The code size reduction is about 12 k for libopencore_author.so.

Removed trailing whitespaces. Untabify.

Cleanup value add in makefile.

Fix more opencore simulator breakage. Hopefully that's the last of it.

Fix opencore simulator breakage

Export more symbols to make GCC 4.4 happy.
By Jing Yu.

Merge change 3508 into donut

* changes:
  Don't bother adding unsupported metadata to our list, and reject album art over 3 megabytes. This should go a long way towards the media scanner not crashing on corrupted mp3s.

Don't bother adding unsupported metadata to our list, and reject album art over 3 megabytes.
This should go a long way towards the media scanner not crashing on corrupted mp3s.

Merge change 1547 into donut

* changes:
  NJ-1409: (platform/external/opencore) Add support for set sampling rate and number of channels

NJ-1409: (platform/external/opencore) Add support for set sampling rate and number of channels

Merge change 3383 into donut

* changes:
  OpenCORE code size reduction.

OpenCORE code size reduction.

Bug 1897865:
The final solution is not ready for OTA.
This change temporarily marks off the changes for spinning off a separate file writer thread.

Merge change 2998 into donut

* changes:
  RIO-6494: OMX node can go into fatal error without notifying the engine properly. This can result ANR.

RIO-6494: OMX node can go into fatal error without notifying the engine properly. This can result ANR.

Merge change 2817 into donut

* changes:
  NJ-1422 :- Native crash after record the 1 min video After 1min of recording AuthorEngine issues an internal Stop() which Stops the MediaInputNode port and queues Stop on MIO component. MIO component still has some data queued in its queue and keeps sending data to MediaInputNode. MediaInputNode port does a OsclLeave with OsclErrNotReady since it has moved to Stopped State. Audio MIO does not handle this leave and leave is transferred to upper leaves, resulting in Abort. Fixed Audio MIO to handle leave properly by adding TRY CATCH block around writeAsync call. TBD - We need to also make sure that leave is caught in PlayerDriver and AuthorDriver rather thank transferring it to upper layers, so that we can avoid call of abort and avoiding a crash.

NJ-1422 :- Native crash after record the 1 min video
After 1min of recording AuthorEngine issues an internal Stop()
which Stops the MediaInputNode port and queues Stop on MIO component.
MIO component still has some data queued in its queue and keeps sending
data to MediaInputNode. MediaInputNode port does a OsclLeave with OsclErrNotReady
since it has moved to Stopped State. Audio MIO does not handle this leave and leave is
transferred to upper leaves, resulting in Abort. Fixed Audio MIO to handle leave properly
by adding TRY CATCH block around writeAsync call.
TBD - We need to also make sure that leave is caught in PlayerDriver and AuthorDriver rather
thank transferring it to upper layers, so that we can avoid call of abort and avoiding a crash.

Merge change 2902 into donut

* changes:
  Use a worker thread to write the fragment to the sdcard.

Merge change 2873 into donut

* changes:
  Fix broken open() call.

Use a worker thread to write the fragment to the sdcard.

Added a worker thread running at normal priority to write the
fragments on the sdcard.

Fragments are stored in a circular buffer.
There is flush method on the worker to ensure all data has
been written.

Fix broken open() call.

Use setVideoSurface call on MIO when surface changes

Allow setVideoSurface to be called multiple times. Allow a null
to be passed so that video can continue playing without a surface.

Allow subsequent calls to setVideoSurface after prepare()

Player Driver and Author Driver crash fix, if InitializeThread fails

Route audio source from AuthorDriver::setAudioSource() to audio MIO.
This is a continuation of the changes needed to route the audio
source to allow developers to choose an audio source other than
the microphone.
Bug 1846343

Re-apply part of perforce CL 131021, which was lost with a recent integrate from 

Re-apply perforce CL 134361-p9, which was lost with a recent integrate from 

Merge change 1964 into donut

* changes:
  RIO-6460: any pause/resume in the last audio frame will cause AMIO stuck.

RIO-6460: any pause/resume in the last audio frame will cause AMIO stuck.

Enable AGC and noise suppression for MediaRecorder
Bug 1725543

Improve audio quality of recordings by increasing AMR bitrate.

Merge change 1512 into donut

* changes:
  fix a tiny class design flaw in pvmf_omx_videodec_node.h change the access modifier to private for those methods that do not need to be visible outside of the class

fix a tiny class design flaw in pvmf_omx_videodec_node.h
change the access modifier to private for those methods that do not need to be visible outside of the class

Merge change 1423 into donut

* changes:
  Fix OMX enc node memory leak. See also: https://android-git.corp.google.com/g/1396

Fix OMX enc node memory leak.
See also:
https://android-git.corp.google.com/g/1396

Merge change 1415 into donut

* changes:
  NJ-1410 : Fix for large time taken to start playback after seek on youtube contents

NJ-1410 : Fix for large time taken to start playback after seek on youtube contents

Merge change 1421 into donut

* changes:
  Don't ignore exceptions in MediaScanner. The MediaScanner skips a directory when an error occurs while scanning the directory. This is reasonable if a file can't be read or something or a directory is not accessible. However, if a Java exception occurs, calling into Java again with a pending exception will cause the VM to be aborted. This change aborts processing and passes the Java exception back up to Jave for processing instead of continuing.

Don't ignore exceptions in MediaScanner.
The MediaScanner skips a directory when an error occurs while scanning
the directory. This is reasonable if a file can't be read or something
or a directory is not accessible. However, if a Java exception occurs,
calling into Java again with a pending exception will cause the VM to be
aborted. This change aborts processing and passes the Java exception
back up to Jave for processing instead of continuing.

Merge change 1295 into donut

* changes:
  RIO-6419, NJ-1416 Mp3 Progress Bar Hang Fix

Merge change 1301 into donut

* changes:
  oscl tls registry race condition bug fix

oscl tls registry race condition bug fix

If the tls registry is removed in a thread while another thread is accessing an item in the registry, this causes crashes

- the registry was not protected at all, and now protected with a lock
- there is no need for an individual lock on the key; and thus it is removed
- called unlock before return for getInstance() and registerInstance()

Merge change 1162 into donut

* changes:
  OC2 oscl registry race condition The idea is to only initialize the singleton table for once and only once and keep it around forever. As a result, the lock for locking the whole singleton table is not needed and thus removed in this change. This approach works only if the client program follows the protocol for using the singleton: it must call initialize() first and then call the rest methods in this class.

RIO-6419, NJ-1416 Mp3 Progress Bar Hang Fix

OC2 oscl registry race condition
The idea is to only initialize the singleton table for once and only once and keep it around forever.
As a result, the lock for locking the whole singleton table is not needed and thus removed in this change.
This approach works only if the client program follows the protocol for using the singleton:
it must call initialize() first and then call the rest methods in this class.

- changed the pointer to a singletontable to be a static object so that it is initialized statically
- removed an extra lock on the whole singletontable, since we don't need it
- moved initialize and cleanup to the header file for optimization. actually, these two methods can be removed. I put the removal of
  these two methods as a FIXME.

Fix the 2way build.

We were logging media scanner failures, but not which file failed, which is not very useful.

Incorporate all the changes in external/opencore in git master branch
- add two patches for fixing thumbnail extraction and authoring compatibility
- remove 2way engine from system image
Not done yet (pending one fix from PV)

Bug fix(1807910): a crash in media recorder due to the use of locked camera
Sorry for the repeatedly review request because I messed up the git repo.

- change the signature of SetCamera() method to return a status code so that
  we can check the status of SetCamera() in authordriver.cpp
- change camera MIO (android_camera_input.cpp) to use create factory method
  for constructing a Camera client from a camera remote so that the status
  of the construction can be verified before we allow recording to proceed.
- change some LOGD statements to remove compiler warnings due to the use
  of %p for ssize_t variable "offset".
- change the call flow in authordriver.cpp to avoid code duplication
- fixed a bug in the previous change of the call flow

merge cupcake into donut

AI 144029: Don't send write complete until frame is no longer displayed
  We seem to be overwriting video frames while they are being displayed.
  This change holds back an acknowledgement for the frame currently
  displayed until the next one is received. The final frame is released
  when EOS, stop, reset, flush, or cancel command is received.
  BUG=1749950

Automated import of CL 144029

AI 144031: am: CL 144029 Don't send write complete until frame is no longer displayed
  We seem to be overwriting video frames while they are being displayed.
  This change holds back an acknowledgement for the frame currently
  displayed until the next one is received. The final frame is released
  when EOS, stop, reset, flush, or cancel command is received.
  Original author: davidsparks
  Merged from: //branches/cupcake/...

Automated import of CL 144031

Automated import from //branches/donutburger/...@142712,142712

Automated import from //branches/cupcake/...@142601,142601

Automated import from //branches/donutburger/...@142513,142513

Automated import from //branches/cupcake/...@142512,142512

Automated import from //branches/donutburger/...@142071,142071

Automated import from //branches/donutburger/...@141819,141819

Automated import from //branches/donutburger/...@140956,140956

Automated import from //branches/cupcake/...@142067,142067

Automated import from //branches/cupcake/...@141816,141816

Automated import from //branches/donutburger/...@140524,140524

auto import from //branches/cupcake_rel/...@141571

auto import from //branches/cupcake_rel/...@140373

auto import from //branches/cupcake_rel/...@138607

auto import from //branches/cupcake/...@137873

auto import from //branches/cupcake/...@137197

auto import from //depot/cupcake/@136654

auto import from //depot/cupcake/@136594

auto import from //depot/cupcake/@135843

auto import from //depot/cupcake/@135843

auto import from //depot/cupcake/@132589

auto import from //depot/cupcake/@137055

auto import from //branches/cupcake/...@132276