summaryrefslogtreecommitdiffstats
path: root/media
Commit message (Collapse)AuthorAgeFilesLines
* Pulled out Callback code into base/callback.h. This is the first step ↵akalin@chromium.org2010-02-1928-31/+55
| | | | | | | | | | | | | 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
* Solaris: Second set adding in Solaris as an OS optionevan@chromium.org2010-02-181-2/+2
| | | | | | | | | | 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
* gles2: fix stride supportpiman@chromium.org2010-02-171-3/+26
| | | | | | Review URL: http://codereview.chromium.org/626009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@39283 0039d316-1c4b-4281-b951-d872f2087c98
* Add comment to unit tests previously disabledhclam@chromium.org2010-02-171-0/+8
| | | | | | | | | | | | | 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
* Disable FFmpegGlueTest.Write and friendshclam@chromium.org2010-02-171-4/+4
| | | | | | | | | | | | 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
* API to allow strategy class to work on the output buffer of OpenMAXhclam@chromium.org2010-02-1712-147/+1343
| | | | | | | | | | | | | | 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
* media: remove redundant gyp excludesevan@chromium.org2010-02-171-24/+0
| | | | | | | | | | 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
* Software performance qualification.fbarchard@chromium.org2010-02-132-0/+474
| | | | | | | | | 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
* Fix player_x11 so it works properly with GLEShclam@chromium.org2010-02-132-68/+85
| | | | | | | | | | | | | | | 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
* Implement GLES video renderer in player_x11hclam@chromium.org2010-02-128-316/+910
| | | | | | | | | | 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
* Fix using of OmxInputBufferhclam@chromium.org2010-02-101-15/+12
| | | | | | | | | | 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
* Unit test for OmxInputBufferhclam@chromium.org2010-02-092-0/+68
| | | | | | | | | 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
* Fix usage of OmxInputBuffer in OpenMAX code since it's refcounted.hclam@chromium.org2010-02-093-13/+15
| | | | | | | | | | 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
* linux: build with -Wextraevan@chromium.org2010-02-051-7/+7
| | | | | | | | | | | | 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
* Refactor media::InputBuffer to consolidate it with media::Bufferhclam@chromium.org2010-02-058-46/+49
| | | | | | | | | | | | 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
* Remove size_t from audio IPC code.jam@chromium.org2010-02-0526-242/+244
| | | | | | | 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
* Refactoring of media::OmxCodec to split configuration of IO portshclam@chromium.org2010-02-0412-529/+704
| | | | | | | | | | | | | 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
* Use FFmpeg demuxer and bitstream converters in omx_testhclam@chromium.org2010-01-306-243/+579
| | | | | | | | | | | | | | | | 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
* Fix libssl.so link failures on Linux shared lib build.mmoss@google.com2010-01-271-0/+8
| | | | | | | | | | | | 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
* Refactor some code that depends on FFmpeghclam@chromium.org2010-01-2624-60/+60
| | | | | | | | | | | | | 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
* Use OpenMAX for video decoding in the media pipelinehclam@chromium.org2010-01-235-48/+111
| | | | | | | | | | | | | 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
* More unit tests for OpenMAXhclam@chromium.org2010-01-221-22/+95
| | | | | | | | | | | | | | | 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
* linux: add display option to media_bench, using GLpiman@chromium.org2010-01-224-54/+310
| | | | | | Review URL: http://codereview.chromium.org/523120 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@36886 0039d316-1c4b-4281-b951-d872f2087c98
* Switches to enable OpenMAX for video decoding in Chromehclam@chromium.org2010-01-224-5/+9
| | | | | | | | | 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
* Unit tests for testing OpenMAX supporthclam@chromium.org2010-01-221-24/+331
| | | | | | | | | | | | | 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
* Video and OpenMAX: add role name to replace vendor specific component name.hclam@chromium.org2010-01-214-36/+106
| | | | | | | | | | | | | | | | | | | 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
* Separate input data into NALUs for testing OpenMAXhclam@chromium.org2010-01-201-30/+103
| | | | | | | | | | 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
* Fix build caused by omx_testhclam@chromium.org2010-01-151-0/+2
| | | | | | | | | | 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
* enable simple encoding in omx_codechclam@chromium.org2010-01-155-267/+585
| | | | | | | | | | | | | | | | | | | 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
* A simple sample video for decoder testing.fbarchard@chromium.org2010-01-131-0/+0
| | | | | | | | | 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
* Simple unit tests for OpenMAX video decoderhclam@chromium.org2010-01-131-3/+36
| | | | | | | | | | 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
* More plumbing of the low latency modecpu@chromium.org2010-01-136-27/+69
| | | | | | | | | | | | | | - 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
* omx_test refactor to add encoder support. (only start)hclam@chromium.org2010-01-124-85/+316
| | | | | | | | | | | | | 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
* Retry 35837 linux: grab device name before closing itajwong@chromium.org2010-01-092-3/+14
| | | | | | | | | | | | 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
* Revert 35837 - Failed media tests - linux: grab device name before closing itrsesek@chromium.org2010-01-081-3/+2
| | | | | | | | | | | | | | | 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
* linux: grab device name before closing itevan@chromium.org2010-01-081-2/+3
| | | | | | | | | | | | 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
* Revert my last two commits that brokoe the buildhclam@chromium.org2010-01-064-311/+80
| | | | | | | | 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
* Fix linux buildhclam@chromium.org2010-01-061-3/+4
| | | | | | | | | | | 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
* omx_test refactor to add encoder support. (only start)hclam@chromium.org2010-01-064-80/+310
| | | | | | | | | | | | | | | 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
* Use the standard DeleteTask rather than rolling my own.ajwong@chromium.org2009-12-301-13/+1
| | | | | | | | | TBR=scherkus BUG=none TEST=none git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35364 0039d316-1c4b-4281-b951-d872f2087c98
* Clarify licenses on a bunch of files.evan@chromium.org2009-12-291-0/+4
| | | | | | | | | | | 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
* Retry r35228: Fix windows build due to missing mock ffmpeg entry points.ajwong@chromium.org2009-12-232-0/+6
| | | | | | | | 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
* TBR: levinajwong@chromium.org2009-12-232-6/+0
| | | | git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35229 0039d316-1c4b-4281-b951-d872f2087c98
* Fix windows build due to missing mock ffmpeg entry points.ajwong@chromium.org2009-12-232-0/+6
| | | | | | Review URL: http://codereview.chromium.org/522004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@35228 0039d316-1c4b-4281-b951-d872f2087c98
* Implementation of OmxVideoDecodeEngine.ajwong@chromium.org2009-12-2227-700/+1602
| | | | | | | | | | | | | | | | | | 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
* Add a low-latency audio test that uses syncsocketscpu@chromium.org2009-12-171-0/+118
| | | | | | | | | 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
* Give classes with virtual methods virtual protected destructors instead of ↵jamesr@chromium.org2009-12-161-0/+3
| | | | | | | | | | | | | | | | | | | | | | 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
* Link the following tests against tcmalloc:glider@chromium.org2009-12-101-0/+7
| | | | | | | | | | | | | | | | | | | | | 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
* Refactor FFmpegVideoDecoder to try and generalize code common to all video ↵ajwong@chromium.org2009-12-1015-360/+913
| | | | | | | | | | 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
* Checking in the beginnings of omx_unittests.scherkus@chromium.org2009-12-093-3/+38
| | | | | | | | | | | | 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