summaryrefslogtreecommitdiffstats
path: root/media/video/ffmpeg_video_decode_engine.cc
Commit message (Collapse)AuthorAgeFilesLines
* Don't forget the ffmpeg input buffer padding when allocating a codec'scevans@chromium.org2011-06-081-2/+4
| | | | | | | | | extradata buffer. BUG=82438 Review URL: http://codereview.chromium.org/7137002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@88354 0039d316-1c4b-4281-b951-d872f2087c98
* Removing FFmpegVideoAllocator as it provides little benefit while increasing ↵scherkus@chromium.org2011-06-031-58/+31
| | | | | | | | | | | | code complexity. BUG=none TEST=media_unittests, media layout tests, ui_tests Review URL: http://codereview.chromium.org/6993018 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@87830 0039d316-1c4b-4281-b951-d872f2087c98
* Respect pixel aspect ratio of video if it contains one.scherkus@chromium.org2011-06-031-2/+2
| | | | | | | | | | Patch by scottfr@chromium.org: http://codereview.chromium.org/7086002 BUG=18941 TEST=Check correct AR on http://people.xiph.org/~giles/2009/celt-aspect.html, media/video-display-aspect-ratio git-svn-id: svn://svn.chromium.org/chrome/trunk/src@87751 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 87578 - Respect pixel aspect ratio of video if it contains one.michaeln@google.com2011-06-021-2/+2
| | | | | | | | | | | | | | BUG=18941 TEST=Check correct AR on http://people.xiph.org/~giles/2009/celt-aspect.html Alternately, fix 59412 and run media/video-display-aspect-ratio layout test Review URL: http://codereview.chromium.org/7086002 TBR=scottfr@chromium.org Review URL: http://codereview.chromium.org/7109002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@87579 0039d316-1c4b-4281-b951-d872f2087c98
* Respect pixel aspect ratio of video if it contains one.scottfr@chromium.org2011-06-021-2/+2
| | | | | | | | | | | BUG=18941 TEST=Check correct AR on http://people.xiph.org/~giles/2009/celt-aspect.html Alternately, fix 59412 and run media/video-display-aspect-ratio layout test Review URL: http://codereview.chromium.org/7086002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@87578 0039d316-1c4b-4281-b951-d872f2087c98
* Forward decoder errors from FFmpegVideoDecodeEngine to FFmpegVideoDecoder.scherkus@chromium.org2011-04-291-8/+6
| | | | | | | | | | | Previously they were getting dropped on the floor, which could leave the media pipeline in lame duck mode. BUG=80187 TEST=media_unittests Review URL: http://codereview.chromium.org/6899058 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@83577 0039d316-1c4b-4281-b951-d872f2087c98
* Rename VideoCodecConfig to VideoDecoderConfig and move to ↵scherkus@chromium.org2011-04-271-2/+2
| | | | | | | | | | | media/base/video_decoder_config.{cc,h}. BUG=none TEST=compiles Review URL: http://codereview.chromium.org/6905065 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@83226 0039d316-1c4b-4281-b951-d872f2087c98
* Create helper methods to process FFmpeg index data.acolwell@chromium.org2011-04-011-1/+1
| | | | | | | | | BUG=none TEST=FFmpegCommonTest Review URL: http://codereview.chromium.org/6708130 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@80153 0039d316-1c4b-4281-b951-d872f2087c98
* Fold media/ffmpeg/ffmpeg_util.{cc,h} into media/ffmpeg/ffmpeg_common.{cc,h}scherkus@chromium.org2011-03-121-1/+0
| | | | | | | | | BUG=none TEST=compiles Review URL: http://codereview.chromium.org/6686011 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@77896 0039d316-1c4b-4281-b951-d872f2087c98
* Remove FFmpegVideoDecodeEngine's dependency on AVStream (again).scherkus@chromium.org2011-03-111-24/+38
| | | | | | | | | | | First step of many towards removing DemuxerStream::QueryInterface, AVStreamProvider, and MediaFormat. This time without memory leaks! BUG=28206 TEST=media_unittests git-svn-id: svn://svn.chromium.org/chrome/trunk/src@77875 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 77476 - Remove FFmpegVideoDecodeEngine's dependency on AVStream.tyoshino@chromium.org2011-03-111-37/+24
| | | | | | | | | | | | | | | | | This change seems to have caused Linux Heapcheck bot redness. http://build.chromium.org/p/chromium.memory/builders/Linux%20Heapcheck/builds/4846 First step of many towards removing DemuxerStream::QueryInterface, AVStreamProvider, and MediaFormat. BUG=28206 TEST=media_unittests Review URL: http://codereview.chromium.org/6624062 TBR=scherkus@chromium.org Review URL: http://codereview.chromium.org/6665020 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@77770 0039d316-1c4b-4281-b951-d872f2087c98
* Remove FFmpegVideoDecodeEngine's dependency on AVStream.scherkus@chromium.org2011-03-091-24/+37
| | | | | | | | | | | First step of many towards removing DemuxerStream::QueryInterface, AVStreamProvider, and MediaFormat. BUG=28206 TEST=media_unittests Review URL: http://codereview.chromium.org/6624062 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@77476 0039d316-1c4b-4281-b951-d872f2087c98
* Implement webkit media metrics in chromium.scherkus@chromium.org2011-02-161-4/+8
| | | | | | | | | | | | | | | This implements the chromium side of the webkit media statistics feature. A followup cl (in webkit) will wire the two sides together. Patch by sjl@chromium.org: http://codereview.chromium.org/6246091/ BUG=71255 TEST=media_unittests git-svn-id: svn://svn.chromium.org/chrome/trunk/src@75050 0039d316-1c4b-4281-b951-d872f2087c98
* Chromium build failed with heap leak checker, or configured withtony@chromium.org2011-01-131-3/+2
| | | | | | | | | | | | | | | | | 'gyp_chromium -Dlinux_use_heapchecker=1 -Dlinux_keep_shadow_stacks=1' parameters. This configuration set gcc option '-finstrument-functions' and cause 'control reaches end of non-void function' error in some sources. Some errors occur in WebKit, and this changelist doesn't include patches for WebKit sources. TEST=pass sync_unit_tests, browser_tests, net_unittests Original patch by Takashi Toyoshima <toyoshim@google.com> at http://codereview.chromium.org/5991005/ Review URL: http://codereview.chromium.org/6262002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@71274 0039d316-1c4b-4281-b951-d872f2087c98
* Handle changes in video frame size.scherkus@chromium.org2011-01-071-7/+13
| | | | | | | | | BUG=67303 TEST=media_unittests Review URL: http://codereview.chromium.org/6034007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@70703 0039d316-1c4b-4281-b951-d872f2087c98
* Deinline even more destructors.erg@google.com2010-12-141-0/+8
| | | | | | | | | BUG=none TEST=compiles Review URL: http://codereview.chromium.org/5794003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@69084 0039d316-1c4b-4281-b951-d872f2087c98
* Rename media::StreamSample::kInvalidTimestamp to media::kNoTimestamp.scherkus@chromium.org2010-11-121-2/+2
| | | | | | | | | | | Old name was overly verbose. BUG=54110 TEST=compiles Review URL: http://codereview.chromium.org/4877002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@65989 0039d316-1c4b-4281-b951-d872f2087c98
* Convert LOG(INFO) to VLOG(1) - media/.pkasting@chromium.org2010-10-191-7/+5
| | | | | | | | | | Also, remove some extra {}s, remove "else" after "return", eliminate a "using". BUG=none TEST=none Review URL: http://codereview.chromium.org/3912001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@63140 0039d316-1c4b-4281-b951-d872f2087c98
* xRevert 62319 - re-enable ogg threadsfbarchard@chromium.org2010-10-121-2/+4
| | | | | | | | | | | | BUG=53967 TEST=ogv videos still play but more efficiently Review URL: http://codereview.chromium.org/3723004 TBR=fbarchard@chromium.org Review URL: http://codereview.chromium.org/3737002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62326 0039d316-1c4b-4281-b951-d872f2087c98
* re-enable ogg threadsfbarchard@chromium.org2010-10-121-4/+2
| | | | | | | | | BUG=53967 TEST=ogv videos still play but more efficiently Review URL: http://codereview.chromium.org/3723004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@62319 0039d316-1c4b-4281-b951-d872f2087c98
* Resubmit GpuVideoDecoder and related patches.hclam@chromium.org2010-09-171-0/+1
| | | | | | | | | BUG=53714 TEST=Tree is green Review URL: http://codereview.chromium.org/3442006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@59860 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 59785 - Implement FakeGlVideoDecodeEngine using FakeGlVideoDecodeContextmlloyd@chromium.org2010-09-171-1/+0
| | | | | | | | | | | | | | | | | Defines UploadToVideoFrame in VideoDecodeContext. FakeGlVideoDecodeEngine now uses FakeGlVideoDecodeContext to video frame allocation and uploading. BUG=53714 TEST=Tree is green Review URL: http://codereview.chromium.org/3312022 TBR=hclam@chromium.org Review URL: http://codereview.chromium.org/3436014 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@59789 0039d316-1c4b-4281-b951-d872f2087c98
* Implement FakeGlVideoDecodeEngine using FakeGlVideoDecodeContexthclam@chromium.org2010-09-171-0/+1
| | | | | | | | | | | | | | Defines UploadToVideoFrame in VideoDecodeContext. FakeGlVideoDecodeEngine now uses FakeGlVideoDecodeContext to video frame allocation and uploading. BUG=53714 TEST=Tree is green Review URL: http://codereview.chromium.org/3312022 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@59785 0039d316-1c4b-4281-b951-d872f2087c98
* Added FakeGlVideoDecodeEngine to exercise the IPC protocol for hardware ↵hclam@chromium.org2010-09-131-10/+10
| | | | | | | | | | | | | | | | video decoding There are several things done in this patch: 1. Added FakeGlVideoDecodeEngine 2. Fixed style problem in VideoDecodeEngine and gpu_video_common.h 3. Added route to pass texture from gpu process to WebKit BUG=53714 TEST=Tree is green Review URL: http://codereview.chromium.org/3335014 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@59223 0039d316-1c4b-4281-b951-d872f2087c98
* Rename all FillThisBuffer and EmptyThisBuffer to something more meaningfulhclam@chromium.org2010-09-051-7/+8
| | | | | | | | | | | | The FillThisBuffer and EmptyThisBuffer names are very confusing. Anyone new working on the code probably won't understand the code at all. I'm changing the names to something more meaningful, which state the actual action taken by the methods. It is also easier to tell whether the method is for input pin or for output pin by their names. Review URL: http://codereview.chromium.org/3201013 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@58604 0039d316-1c4b-4281-b951-d872f2087c98
* Remove refcounting of VideoDecodeEnginehclam@chromium.org2010-09-031-0/+4
| | | | | | | | | | | | | VideoDecodeEngine doesn't need refcounting, all decode engines live on one thread so there isn't a problem posting task to a random thread. Also the stop is asynchronous, we can always until the stop is done before we destroy the object and doesn't need refcounting to destroy it. TEST=media_unittests Review URL: http://codereview.chromium.org/3125032 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@58539 0039d316-1c4b-4281-b951-d872f2087c98
* Merge remote branch 'origin/trunk' into resubmithclam@chromium.org2010-09-031-0/+359
| | | | | | | | | | | | | | Refactor video decode engines to move them to a new folder Moving video decode engines and friends to media/video. TEST=Tree is green. Video plays. Review URL: http://codereview.chromium.org/3127027 Review URL: http://codereview.chromium.org/3291009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@58446 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 58423 - Refactor video decode engines to move them to a new folderakalin@chromium.org2010-09-021-359/+0
| | | | | | | | | | | | | Moving video decode engines and friends to media/video. TEST=Tree is green. Video plays. Review URL: http://codereview.chromium.org/3127027 TBR=hclam@chromium.org Review URL: http://codereview.chromium.org/3361004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@58428 0039d316-1c4b-4281-b951-d872f2087c98
* Refactor video decode engines to move them to a new folderhclam@chromium.org2010-09-021-0/+359
Moving video decode engines and friends to media/video. TEST=Tree is green. Video plays. Review URL: http://codereview.chromium.org/3127027 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@58423 0039d316-1c4b-4281-b951-d872f2087c98