| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
|
| |
towards redoing the Callback interfaces.
Added and removed includes as needed.
BUG=35223
TEST=trybots
Review URL: http://codereview.chromium.org/646061
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@39419 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
TEST=compiles
BUG=30101
Patch by James Choi <jchoi42@pha.jhu.edu>.
Review URL: http://codereview.chromium.org/606075
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@39362 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: http://codereview.chromium.org/626009
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@39283 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
FFmpegGlueTest and friends are crashing on the build bots. Some
of them are disabled and need comment in the code to indicate
the bug number.
BUG=36037
TBR=scherkus, finnur
Review URL: http://codereview.chromium.org/619012
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@39253 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
FFmpegGlueTest.Write starts to crash. Can't fix it shortly so
disable it and its friends.
BUG=36037
TBR=finnur, scherkus
Review URL: http://codereview.chromium.org/624001
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@39252 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Added interface of OmxOutputSink to interact with OmxCodec to
perform buffer negotiation and buffer read signaling.
TEST=media_unittests --gtest_filter=Omx*
BUG=32753
BUG=32754
BUG=32870
Review URL: http://codereview.chromium.org/593047
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@39247 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
I moved all of these general patterns to common.gypi.
Also rearrange the common.gypi patterns to cover more common
Linux patterns.
Review URL: http://codereview.chromium.org/608032
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@39236 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=32758
TEST=build this test in release mode, get test data (see source), and run the test.
Review URL: http://codereview.chromium.org/552231
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38989 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Fixed the following two things:
1. Some drivers doesn't like EGL context switching between threads.
So initialize GLES lazily in the paint call.
2. glUniformMatrix3fv() failed because some drivers doesn't want to
transpose the matrix so we do it ourself.
TEST=player_x11 works with GLES
Review URL: http://codereview.chromium.org/596104
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38986 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
This patch does the following:
1. Split the implementation of X11 and GL into two separate files
2. Add implementation of GLES render
Review URL: http://codereview.chromium.org/596055
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38867 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
OmxInputBuffer is now ref counted, but some code still try to
delete it explicitly. This delete call is hidden in a template
function so it was hard to find!
Review URL: http://codereview.chromium.org/601015
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38571 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
TEST=media_unittest --gtest_filter=OmxInputBuffer.*
BUG=32753
Review URL: http://codereview.chromium.org/580012
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38502 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
In last refactoring of OpenMAX code, OmxInputBuffer was changed
to refcounted but it wasn't probably used. This is now fixed.
Unittest is coming in another patch.
Review URL: http://codereview.chromium.org/596018
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38498 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
95% of this is removing "const" from return types, but turning this
on found one bug! (A "for" loop that expected its iterator to go negative but
which was using an unsigned type.)
BUG=34160
Review URL: http://codereview.chromium.org/570012
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38266 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
Rename media::InputBuffer to media::OmxInputBuffer and inherit from
media::Buffer. Also provide the timestamp to media::OmxCodec.
TEST=omx_test
BUG=32753
Review URL: http://codereview.chromium.org/577012
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38247 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
| |
The change got to this size because I had to modify the surrounding code (I didn't want to just cast at the last minute).
Review URL: http://codereview.chromium.org/577006
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38192 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Split encoder/decoder configuration of media::OmxCodec into
strategy classes. This will help OmxCodec free from any codec
specific bits.
Also included the refactoring of omx_test to split the file
output part to a separate class.
Review URL: http://codereview.chromium.org/566036
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@38121 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
omx_test now uses FFmpeg demuxer and bistream converters. It
can now parse mp4 files and convert them to H264 Annex B stream
for input to OpenMAX decoder.
Also refactored the file to move different reader methods to
separate classes.
BUG=32753
TEST=this is a test by itself
Review URL: http://codereview.chromium.org/556032
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@37592 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
http://build.chromium.org/buildbot/waterfall.fyi/builders/Chromium%20Linux%20Builder%20(dbg-shlib)/builds/6369/steps/compile/logs/stdio
This has been broken for a long time, I think basically since Linux switched to
use_system_ssl=0.
Also fix a couple unrelated shared lib failures in media.
Review URL: http://codereview.chromium.org/553097
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@37256 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Some FFmpeg header and source files are misplaced:
media/filters/ffmpeg_common.{cc, h} -> media/ffmpeg
media/tools/media_bench/file_protocol.{cc, h} -> media/ffmpeg
And the necessary refactoring for the new location is done.
Review URL: http://codereview.chromium.org/555087
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@37080 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This patch is to fix some issues when using OpenMAX video decoder
in the media pipeline. It will enable player_x11 to decode using
OpenMAX.
This still requires some more work to enable Chrome to use
OpenMAX for decoding.
Review URL: http://codereview.chromium.org/549124
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@36936 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Added the following test cases:
- Transition to executing state and back
- Free component handle during executing state to expose leakage.
- Request component handle using callbacks struct on stack and
clear. This exposes bugs in library that doesn't copy the struct.
TEST=omx_unittests
BUG=NONE
Review URL: http://codereview.chromium.org/548095
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@36912 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: http://codereview.chromium.org/523120
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@36886 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
Added a --enable-openmax switch to enable OpenMAX for video
decoding.
Review URL: http://codereview.chromium.org/555046
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@36883 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Added the following test cases:
1. GetComponentsByRole() now works with valid roles
2. GetHandle() / FreeHandle() pair on video decoder component
3. General configuration of video decoder component
TEST=omx_unittests
Review URL: http://codereview.chromium.org/550051
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@36834 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Add role name to replace vendor specific component name.
Remove vendor specific Open MAX component name. use role
name to find the appropriate component name in current OMX core
library. We only pick the first component name returned and ignore all
the other components with the same role.
BUG=none
TEST=none
Reviewed: http://codereview.chromium.org/551084/show
Submitted for: jiesun@chromium.org
Review URL: http://codereview.chromium.org/546098
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@36716 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
Feed OpenMAX H264 decoder into chunks of NALUs. That will solve
some compability issues with decoders that require NALUs as
boundary.
Review URL: http://codereview.chromium.org/546018
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@36686 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
TBR=scherkus
Initialize struct variables by memset()ing them.
Review URL: http://codereview.chromium.org/552017
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@36401 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
1. add Format callback to notify media format change.
2. add timestamp to encoder input sample.
3. add loop count to input streams.
4. add enable-csc in encoder path.
5. measure initial delay
6. move color space conversion inside TestApp.(according to Alpha).
7. copy component name in OmxCodec ( Andrew ).
BUG=31804
Submitted for: jiesun@chromium.org
Reviewed: http://codereview.chromium.org/519084/show
Review URL: http://codereview.chromium.org/552016
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@36400 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=30789
TEST=play this video with media_bench and ensure content decodes quickly and correctly.
Review URL: http://codereview.chromium.org/545034
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@36174 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
Added two simple test cases for OpenMAX video decoder. One
does init and deinit of OpenMAX. The other one enumerate
available components of video_decoder role.
Review URL: http://codereview.chromium.org/524044
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@36170 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
- added flag to factory to request low latency mode
- implemented switch from triple buffered to double buffer on windows (mac is already double buffered all the time)
- added tests
BUG=28292
TEST=ut included
Review URL: http://codereview.chromium.org/523073
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@36090 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
1. add --dumpraw options
2. add --dumpyuv options to convert NV21 to IYUV
3. change StartUp() function to take input and output format.
Reviewed: http://codereview.chromium.org/523044/show
Submitted for: jiesun@chromium.org
Review URL: http://codereview.chromium.org/542017
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35976 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
PcmClose frees the handle regardless of whether there's an error while shutting down, so grab the PcmName result first in case we want to use it in an error message.
Original Review URL: http://codereview.chromium.org/538005
BUG=20006
Review URL: http://codereview.chromium.org/542002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35854 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
PcmClose frees the handle regardless of whether there's an error
while shutting down, so grab the PcmName result first in case we
want to use it in an error message.
BUG=20006
Review URL: http://codereview.chromium.org/538005
TBR=evan@chromium.org
Review URL: http://codereview.chromium.org/540003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35848 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
PcmClose frees the handle regardless of whether there's an error
while shutting down, so grab the PcmName result first in case we
want to use it in an error message.
BUG=20006
Review URL: http://codereview.chromium.org/538005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35837 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
TBR=scherkus
Review URL: http://codereview.chromium.org/524050
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35642 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
Linux 64 bit build complains about printf. Comment out those
problematic lines.
TBR=ajwong
Review URL: http://codereview.chromium.org/524049
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35641 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
1. add --dumpraw options
2. add --dumpyuv options to convert NV21 to IYUV
3. change StartUp() function to take input and output format.
Submitted=jiesun@google.com
Reviewed=hclam@google.com
Review=http://codereview.chromium.org/523044
BUG=NONE
Review URL: http://codereview.chromium.org/525053
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35639 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
TBR=scherkus
BUG=none
TEST=none
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35364 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
In this process, I also filed bugs on various projects we depend
on that also have unclear licenses.
BUG=28291
Review URL: http://codereview.chromium.org/521009
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35323 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
Original Review URL: http://codereview.chromium.org/522004
TBR: levin
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35238 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
| |
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35229 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
Review URL: http://codereview.chromium.org/522004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35228 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Also moves FFmpegVideoDecodeEngine FFmpegVideoDecoder, OmxVideoDecoder, and VideoDecoderImpl into their own files.
Refactors FFmpegDemuxerTest to be less of a characterization test, and to hopefully be less fragile.
Creates a set of utilities for handling Callbacks versus Tasks, and resource management related to Callbacks.
Re-enables the annexb filters for the chrome build of FFmpeg.
Added a BitstreamConverter class that abstracts the bitstream filter code.
Cleans up a few gyp mistakes with flag exporting.
Review URL: http://codereview.chromium.org/492023
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35171 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=none
TEST=unit test included
Review URL: http://codereview.chromium.org/500002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@34787 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
implicit non-virtual public destructors.
Was originally:
Replace public nonvirtual destructors in classes with virtual members
with protected nonvirtual destructors where possible, and with
public virtual destructors where destruction of a derived class occurs.
(Excluding chrome/browser/...)
(Part 4 of http://www.gotw.ca/publications/mill18.htm
has a rationale for why public nonvirtual destructors in classes with
virtual members is dangerous.)
Patch by: Jacob Mandelson (jlmjln@gmail.com)
BUG=none
TEST=base_unittests & app_unittests
Review URL: http://codereview.chromium.org/200106
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@34633 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
app_unittests
browser_tests
startup_tests
media_unittests
printing_unittests
net_unittests
googleurl_unittests
ipc_tests
unit_tests
ui_tests
This patch was originally reviewed as http://codereview.chromium.org/466089
TBR=willchan
Review URL: http://codereview.chromium.org/488004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@34247 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
decoders.
This changes the DecoderBase API to be fully asynchronous when invoking its subclass's actions.
Review URL: http://codereview.chromium.org/465044
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@34208 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
Also renamed omx_wrapper to the simpler (and more accurate!) omx.
BUG=n/a
TEST=n/a
Review URL: http://codereview.chromium.org/467055
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@34124 0039d316-1c4b-4281-b951-d872f2087c98
|