| Commit message (Collapse) | Author | Age | Files | Lines | 
| | 
| 
| 
| 
|  | 
Change-Id: I3ab0215419214b652573945403eb45eb09d41aff
related-to-bug: 5114811
 | 
| |\  
| | 
| | 
| |  | 
the next"
 | 
| | | 
| | 
| | 
| | 
| | 
| | 
| |  | 
this change intends to support its very limited case and signals an error in
all other cases of unexpected PID changes that we cannot recover from.
Change-Id: Icbfdf9fe7461969e2a8781ed416f54d891dd789a
 | 
| |\ \  
| |/  
| |   
| |    | 
identified, i.e."
 | 
| | | 
| | 
| | 
| | 
| | 
| | 
| | 
| | 
| | 
| | 
| | 
| | 
| |  | 
those that have a queue. This ensures that the player doesn't observe discontinuities
that don't match up across streams.
Also, make sure output buffers arriving from the decoder to be rendered are sent
back to the decoder if we started flushing.
Finally, don't parse TS packets for streams we don't support. And don't allocate
memory for them.
Change-Id: I708e0de4cba8110a62e4c8ceb1e5702430d5d2bb
 | 
| |\ \   | 
 | 
| | | | 
| | | 
| | | 
| | | 
| | | 
| | | 
| | | 
| | |  | 
SurfaceMediaSource_test now does not use pbuffer configs
to create a surface. Always uses a window config.
Also, cleaned up the test a bit.
Change-Id: Idda40dcb8474249eccaa8d2dad20ee840d2dd484
 | 
| |\ \ \  
| | | | 
| | | | 
| | | |  | 
still pending."
 | 
| | | | | 
| | | | 
| | | | 
| | | | 
| | | | 
| | | | 
| | | |  | 
pending.
Change-Id: Ide111264501441b51bb10629ae2021accf9e1fb5
related-to-bug: 5234126
 | 
| |\ \ \ \   | 
 | 
| | | | | | 
| | | | | 
| | | | | 
| | | | | 
| | | | | 
| | | | | 
| | | | | 
| | | | |  | 
Was not basing timestamps on startTimeUs.
Now synchronizes properly with audio.
Bug: 4510826
Change-Id: I613db0aa91e51fc75d120e65540e742d8ab2ae43
 | 
| |\ \ \ \ \  
| |_|_|_|/  
|/| | | |   
| | | | |    | 
not NULL and AudioPlayer was not started. Fix for 5236419."
 | 
| | | | | | 
| | | | | 
| | | | | 
| | | | | 
| | | | | 
| | | | | 
| | | | |  | 
and AudioPlayer was not started.
Fix for 5236419.
Change-Id: I5eadd069118c3533a19b5d6f6dca2ce16d8505fb
 | 
| |\ \ \ \ \  
| |/ / / /  
|/| | | |   
| | | | |    | 
absolute time"
 | 
| | | |/ /  
| |/| |   
| | | |   
| | | |   
| | | |   
| | | |    | 
i.e. PTS 0 <=> media time 0
Change-Id: If10a0f98c129b95e8430dc419664b1def92e547e
 | 
| |\ \ \ \   | 
 | 
| | | | | | 
| | | | | 
| | | | | 
| | | | | 
| | | | | 
| | | | | 
| | | | | 
| | | | | 
| | | | | 
| | | | | 
| | | | | 
| | | | | 
| | | | | 
| | | | | 
| | | | | 
| | | | |  | 
When the A2DP headset is connected, there is a possible
race condition when the audio tracks are moved from
the mixer thread attached to the speaker output to the thread
attached to A2DP output.
As the request to clear the stream type to output mapping cache in
the client process is asynchronous, it is possible that the flag
indicating to the client audio track to re-create the IAudioTrack
on the new thread is processed before the cache is invalidated.
In this case, the track will be attached to the old thread and
music will continue playing over the device speaker instead of being
redirected to A2DP headset.
Change-Id: Ib2ce1eb5320eaff83287b93779061bf4e7a330df
 | 
| |\ \ \ \ \  
| |_|/ / /  
|/| | | |    | 
 | 
| | | |/ /  
| |/| |   
| | | |   
| | | |   
| | | |   
| | | |   
| | | |   
| | | |   
| | | |   
| | | |   
| | | |   
| | | |   
| | | |   
| | | |   
| | | |   
| | | |   
| | | |   
| | | |   
| | | |   
| | | |   
| | | |   
| | | |    | 
This previous frame size computation code had two incorrect
 assumptions:
 1/ the ADTS frame length value already contains the header size
 2/ the ADTS header is not of fixed size: it is 2 bytes bigger
  if it contains the CRC value.
For 1/, the code worked because when the header size was added
 to the frame size, the + operator is of stronger precedence
 than the ? operator, so the size added was always 0 (instead
 of ADTS_HEADER_LENGTH.
For 2/, the code worked as long as there was no CRC in the ADTS
 data.
The fix consists in:
 - documenting what the frame length computation code returns
 - fixing the frame length computation
 - when computing the frame length, also returning the header
  size so the correct data can be sent to the decoder.
Change-Id: I92df72a9e531f594f762e63d62f9dee7b0109904
 | 
| |/ / /  
| | |   
| | |   
| | |   
| | |   
| | |   
| | |   
| | |   
| | |    | 
fails
after a surface change.
Change-Id: Ic7758cbeb107032db68a1c3b8e2984710a12dd8b
related-to-bug: 5212725
 | 
| |\ \ \   | 
 | 
| | | | | 
| | | | 
| | | | 
| | | | 
| | | | 
| | | |  | 
Also make sure NuPlayer can properly shutdown in certain edge cases.
Change-Id: Iceb16d600d87ba66c802e60e95bf62f66487a453
 | 
| |\ \ \ \   | 
 | 
| | | | | | 
| | | | | 
| | | | | 
| | | | | 
| | | | | 
| | | | | 
| | | | |  | 
The client does not have to send (w, h) = (0,0) in dequeuebuffer. It can
set the same w, h as the default width and height.
Change-Id: I8202f90261ccaebbd35ea28c153a7472f01912f1
 | 
| |\ \ \ \ \  
| |_|_|/ /  
|/| | | |    | 
 | 
| | | |_|/  
| |/| |   
| | | |   
| | | |   
| | | |   
| | | |   
| | | |    | 
Found that the build was breaking when the logging was enabled in
MPEG4Writer. Fixed.
Change-Id: I62c5e6d16b2ecf5644c8d46b9bbcca68a797b223
 | 
| |\ \ \ \  
| |/ / /  
|/| | |    | 
 | 
| | | | | 
| | | | 
| | | | 
| | | |  | 
Change-Id: Id6c7a08c34fd5cb6c4632f78ece9e7898b29e52c
 | 
| | |_|/  
|/| |   
| | |   
| | |   
| | |   
| | |   
| | |    | 
in order to compute the final delay before posting end-of-stream correctly.
Change-Id: I4ff6d7142fdf1ea2d19d12ec78c1e84322d22e5f
related-to-bug: 5216930
 | 
| |\ \ \  
| |/ /  
|/| |    | 
 | 
| | | | 
| | | 
| | | 
| | | 
| | | 
| | | 
| | | 
| | | 
| | | 
| | | 
| | | 
| | | 
| | |  | 
AudioTrack::stop() is not synchronous, so a stop() followed
 by flush(), which is synchronous, will not always report
 a playhead position of 0 after being called.
This CL adds a flag to mark a track as flushed, and report the
 correct playhead position in this state.
Bug 5217011 has been created to address the real issue in the
 future, where flush could be made synchronous, to properly
 address bug 4364249.
Change-Id: Icf989d41a6bcd5985bb87764c287f3edb7e26d12
 | 
| |\ \ \  
| | | | 
| | | | 
| | | |  | 
up to that"
 | 
| | | | | 
| | | | 
| | | | 
| | | | 
| | | | 
| | | | 
| | | |  | 
point except previously enqueued discontinuities.
Change-Id: Id04a559dc062fa4a5c80c599ad74bf81a543de01
related-to-bug: 5201378
 | 
| |\ \ \ \  
| |/ / /  
|/| | |   
| | | |    | 
5208736."
 | 
| | | | | 
| | | | 
| | | | 
| | | | 
| | | | 
| | | |  | 
Fix for bug 5208736.
Change-Id: Ib1d74608ec289509082d8dcad080769f48098853
 | 
| |\ \ \ \  
| |_|/ /  
|/| | |   
| | | |    | 
controls the stop and release."
 | 
| | | | | 
| | | | 
| | | | 
| | | | 
| | | | 
| | | | 
| | | |  | 
the stop and release.
Change-Id: Ic79a60dad0f617a845211d3fbe146c1648a71da3
related-to-bug: 5205804
 | 
| |\ \ \ \   | 
 | 
| | | | | | 
| | | | | 
| | | | | 
| | | | |  | 
Change-Id: I12a54b80e7ef958b6d451b7add110d38f05ec439
 | 
| |\ \ \ \ \  
| |_|/ / /  
|/| | | |    | 
 | 
| | | |_|/  
| |/| |   
| | | |   
| | | |   
| | | |   
| | | |   
| | | |   
| | | |   
| | | |   
| | | |   
| | | |   
| | | |   
| | | |   
| | | |   
| | | |    | 
This is the test for Gralloc buffers based encoding.
contains a combination of two main changes:
1. GL based encoding tests added to SurfaceMediaSource_test
2. SurfaceMediaSource ::read() colorformat
The SurfaceMediaSource::getformat() sets it to OMX_COLOR_FormatAndroidOpaque.
The omx encoder needs to interpret that colorformat and reads the format
from the Gralloc buffers directly
Change-Id: Iee2fe8901384109a4952e1d6c528c59eb01eb5b1
 | 
| |\ \ \ \  
| |/ / /  
|/| | |   
| | | |    | 
responses from MPEG-PS streams."
 | 
| | | | | 
| | | | 
| | | | 
| | | | 
| | | | 
| | | | 
| | | | 
| | | | 
| | | | 
| | | | 
| | | | 
| | | | 
| | | | 
| | | | 
| | | | 
| | | |  | 
responses from MPEG-PS streams.
Note: current resync logic keeps searching mp3 frames until it meets a
valid one. This resync logic works well with seek() operation, but in
SniffMP3(), this may cause false-positive classification for the
following MPEG-PS file format.
  A possible MPEG-PS video file:
      [PS header]...[mp3 frame][mp3 frame] ...
  MP3 files:
      [ID3 tag][mp3 frame][mp3 frame] ...
BUG: 5104562
Change-Id: Idd5020723f7bdb2fefc2a63fc8651580fd644399
 | 
| | |/ /  
|/| |   
| | |   
| | |   
| | |   
| | |   
| | |    | 
settings change
Change-Id: I4de0d3c377bb12e59524ed370ab3ed0b9a9b70a6
related-to-bug: 5122512
 | 
| | | | 
| | | 
| | | 
| | | 
| | | 
| | | 
| | |  | 
the doctype of the matroska file indicates that it is a webm file.
Change-Id: I467f597690a841043ecd11d2a2cae93351820f1b
related-to-bug: 5042137
 | 
| |\ \ \   | 
 | 
| | |/ /  
| | |   
| | |   
| | |    | 
Change-Id: I3e9765e7271d80fda302e521501720bf8c13e5c7
 | 
| |\ \ \  
| | | | 
| | | | 
| | | |  | 
Java)"
 | 
| | | | | 
| | | | 
| | | | 
| | | | 
| | | | 
| | | |  | 
Also fixes 5118207 add other video codec support for video editor export.
Change-Id: If72427173bd8ff684af07ba00f4425c1deef29c6
 | 
| |\ \ \ \  
| |/ / /  
|/| | |    | 
 |