summaryrefslogtreecommitdiffstats
path: root/media/video
Commit message (Collapse)AuthorAgeFilesLines
* Fix some grammar in comments, error messages and documentation.gavinp@chromium.org2012-04-032-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Thanks to all my reviewers, you are legion. I hope I didn't waste too much of your time. BUG=None Review URL: http://codereview.chromium.org/9854039 Review URL: http://codereview.chromium.org/9854043 Review URL: http://codereview.chromium.org/9863058 Review URL: http://codereview.chromium.org/9863059 Review URL: http://codereview.chromium.org/9887005 Review URL: http://codereview.chromium.org/9890002 Review URL: http://codereview.chromium.org/9891002 Review URL: http://codereview.chromium.org/9895003 Review URL: http://codereview.chromium.org/9896002 Review URL: http://codereview.chromium.org/9896003 Review URL: http://codereview.chromium.org/9897002 Review URL: http://codereview.chromium.org/9897003 Review URL: http://codereview.chromium.org/9903004 Review URL: http://codereview.chromium.org/9904003 Review URL: http://codereview.chromium.org/9904002 Review URL: http://codereview.chromium.org/9904004 Review URL: http://codereview.chromium.org/9906002 Review URL: http://codereview.chromium.org/9906001 Review URL: http://codereview.chromium.org/9906003 Review URL: http://codereview.chromium.org/9909001 Review URL: http://codereview.chromium.org/9909002 Review URL: http://codereview.chromium.org/9909003 Review URL: http://codereview.chromium.org/9909004 Review URL: http://codereview.chromium.org/9910001 Review URL: http://codereview.chromium.org/9910002 Review URL: http://codereview.chromium.org/9910010 Review URL: http://codereview.chromium.org/9911001 Review URL: http://codereview.chromium.org/9912001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@130359 0039d316-1c4b-4281-b951-d872f2087c98
* Replace the unnecessary VDA::Profile typedef with media::VideoCodecProfilefischman@chromium.org2012-03-301-4/+1
| | | | | | | | | Originally landed as: http://src.chromium.org/viewvc/chrome?view=rev&revision=129752 Reverted in: http://src.chromium.org/viewvc/chrome?view=rev&revision=129765 Review URL: http://codereview.chromium.org/9863027 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@129816 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 129752 - Replace the unnecessary VDA::Profile typedef with ↵mihaip@chromium.org2012-03-301-1/+4
| | | | | | | | | | | media::VideoCodecProfile Review URL: http://codereview.chromium.org/9863027 TBR=fischman@chromium.org Review URL: https://chromiumcodereview.appspot.com/9937008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@129765 0039d316-1c4b-4281-b951-d872f2087c98
* Replace the unnecessary VDA::Profile typedef with media::VideoCodecProfilefischman@chromium.org2012-03-301-4/+1
| | | | | | Review URL: http://codereview.chromium.org/9863027 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@129752 0039d316-1c4b-4281-b951-d872f2087c98
* [Coverity] Fixed possible resource leakgroby@chromium.org2012-03-291-1/+1
| | | | | | | | | | | | | Return value of 0 from open is a valid (if uncommon) result. Current code would leak that CID=101985 BUG= TEST= Review URL: https://chromiumcodereview.appspot.com/9837091 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@129656 0039d316-1c4b-4281-b951-d872f2087c98
* [Coverity] Pass-by-val -> pass-by-refgroby@chromium.org2012-03-291-5/+6
| | | | | | | | | | | | CID=101489,101490,101491,101492 CID_COUNT=4 BUG= TEST= Review URL: https://chromiumcodereview.appspot.com/9873008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@129654 0039d316-1c4b-4281-b951-d872f2087c98
* [Coverity] Fixed frame size wjia@chromium.org2012-03-282-2/+5
| | | | | | | | | It's wrong to use sizeof(pointer) to get the size of video frame. Need remember the frame size when it's allocated. CID=100299 Review URL: https://chromiumcodereview.appspot.com/9837095 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@129428 0039d316-1c4b-4281-b951-d872f2087c98
* PinBase: Fix assign-to-self bug.jhawkins@chromium.org2012-01-301-2/+2
| | | | | | | | | | BUG=none TEST=none R=perkj Review URL: https://chromiumcodereview.appspot.com/9225049 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@119685 0039d316-1c4b-4281-b951-d872f2087c98
* Convert use of int ms to TimeDelta in files owned by vrk.tedvessenes@gmail.com2012-01-142-4/+4
| | | | | | | | | | | R=vrk@chromium.org BUG=108171 TEST= Review URL: http://codereview.chromium.org/9190026 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@117809 0039d316-1c4b-4281-b951-d872f2087c98
* Remove VideoDecodeAccelerator::Client::NotifyEndOfStream / ↵fischman@chromium.org2012-01-131-4/+0
| | | | | | | | | | | | | | | PPP_VideoDecoder_Dev::EndOfStream Nobody's ever called it and it doesn't make sense given the API: EOS is signalled when no more bits are available to Decode(), via Flush(). BUG=109819 TEST=trybots Review URL: http://codereview.chromium.org/9186022 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@117545 0039d316-1c4b-4281-b951-d872f2087c98
* Remove task.h and finish base::Bind() migration.ajwong@chromium.org2012-01-061-1/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Over 341 CLs, in ~3 months, touching 3251 unique files! Top 5 most CLs: (121) jhawkins ( 45) dcheng ( 24) achuith ( 23) csilv ( 12) tfarina ( 12) groby ~1000 files touched: (918) jhawkins 100+ files touched: (486) ajwong (385) willchan (372) dcheng (126) csilv (123) fischman (112) sergeyu 49+ files touched: (65) tfarina (57) acolwell (52) adamk (49) tzik BUG=35223 TEST=existing Review URL: http://codereview.chromium.org/9114020 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@116748 0039d316-1c4b-4281-b951-d872f2087c98
* And now NewRunnableMethod(), you die.ajwong@chromium.org2012-01-043-8/+3
| | | | | | | | | | | Kill off NewRunnableMethod() and remove silly declarations of unused ScopedRunnableMethodFactory(s). BUG=none TEST=existing. Review URL: http://codereview.chromium.org/9034032 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@116250 0039d316-1c4b-4281-b951-d872f2087c98
* base::Bind: Remove callback_old.h.jhawkins@chromium.org2012-01-031-2/+1
| | | | | | | | | | BUG=none TEST=none R=groby,awong,csilv Review URL: http://codereview.chromium.org/9028009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@116169 0039d316-1c4b-4281-b951-d872f2087c98
* Coverity: Initialize member variables.jhawkins@chromium.org2011-12-202-1/+3
| | | | | | | | | | | | | CID=100356,100360,100368,100373,100374,100562,101529,101530,101536,101541, 101542,101595,101630,101685,101687,101689,101776,101812,101813,101854, 101861,101862,101871,101883,101886,101887,102034 BUG=none TEST=none R=binji Review URL: http://codereview.chromium.org/8965054 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@115225 0039d316-1c4b-4281-b951-d872f2087c98
* <video> decode in hardware! fischman@chromium.org2011-12-131-20/+4
| | | | | | | | | | | | | | | | | | | | | | | This uses the GpuVideoDecodeAccelerator machinery (already written to enable ppapi to take advantage of OpenMAX HW where available) to decode <video> data. This increases idle CPU from 20% to 45% on one particularly large (internal) test video (red0.mp4), on an ARM crosbook. HW decode is done on a best-effort basis; if the GPU code doesn't know how to deal with a codec/profile we still fall back to ffmpeg for decode. Because the vast majority of chrome installs will be on HW with no video decode support (yet) we only attempt HW video decode on platforms we know have a shot at it. This is a copy of http://codereview.chromium.org/8686010/ PS#13 to work around rietveld losing the "status" for some files in the patch. BUG=104579 TEST=manual testing w/ video test matrix, trybots. Review URL: http://codereview.chromium.org/8922010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@114183 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 113895 - <video> decode in hardware!fischman@chromium.org2011-12-101-3/+19
| | | | | | | | | | | | | | | | | | | | | | This uses the GpuVideoDecodeAccelerator machinery (already written to enable ppapi to take advantage of OpenMAX HW where available) to decode <video> data. This increases idle CPU from 20% to 45% on one particularly large (internal) test video (red0.mp4), on an ARM crosbook. HW decode is done on a best-effort basis; if the GPU code doesn't know how to deal with a codec/profile we still fall back to ffmpeg for decode. Because the vast majority of chrome installs will be on HW with no video decode support (yet) we only attempt HW video decode on platforms we know have a shot at it. BUG=104579 TEST=manual testing w/ video test matrix, trybots. Review URL: http://codereview.chromium.org/8686010 TBR=fischman@chromium.org Review URL: http://codereview.chromium.org/8897022 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@113908 0039d316-1c4b-4281-b951-d872f2087c98
* <video> decode in hardware!fischman@chromium.org2011-12-101-19/+3
| | | | | | | | | | | | | | | | | | | This uses the GpuVideoDecodeAccelerator machinery (already written to enable ppapi to take advantage of OpenMAX HW where available) to decode <video> data. This increases idle CPU from 20% to 45% on one particularly large (internal) test video (red0.mp4), on an ARM crosbook. HW decode is done on a best-effort basis; if the GPU code doesn't know how to deal with a codec/profile we still fall back to ffmpeg for decode. Because the vast majority of chrome installs will be on HW with no video decode support (yet) we only attempt HW video decode on platforms we know have a shot at it. BUG=104579 TEST=manual testing w/ video test matrix, trybots. Review URL: http://codereview.chromium.org/8686010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@113895 0039d316-1c4b-4281-b951-d872f2087c98
* Removing MessageLoop::QuitTask() from media/dcheng@chromium.org2011-12-091-1/+1
| | | | | | | | | | | | | base/task.h is going away and being replaced by base/bind.h and base/callback.h. This CL was automatically generated by sed. BUG=none TEST=none Review URL: http://codereview.chromium.org/8879032 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@113740 0039d316-1c4b-4281-b951-d872f2087c98
* Collapse FFmpegVideoDecodeEngine into FFmpegVideoDecoder and remove ↵scherkus@chromium.org2011-12-073-291/+0
| | | | | | | | VideoDecodeEngine. Review URL: http://codereview.chromium.org/8772069 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@113358 0039d316-1c4b-4281-b951-d872f2087c98
* fix a racing condition when renderer process requests StopCapture before ↵wjia@chromium.org2011-11-231-11/+0
| | | | | | | | | | | | host gets controller fix a typo (thanks ihf@) BUG=104581 TEST=trybots Review URL: http://codereview.chromium.org/8549010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@111373 0039d316-1c4b-4281-b951-d872f2087c98
* Add OVERRIDE to media/.avi@chromium.org2011-11-163-15/+15
| | | | | | | | | BUG=104314 TEST=no change Review URL: http://codereview.chromium.org/8520033 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@110304 0039d316-1c4b-4281-b951-d872f2087c98
* Simplify VideoDecodeEngine interface by making everything synchronous.scherkus@chromium.org2011-11-044-558/+86
| | | | | | | | | | | | | | | Although I plan to remove VideoDecodeEngine entirely it requires detangling some of the code first. Other noteworthy changes: - It's no longer valid to call VideoFrameReady(NULL), instead FFmpegVideoDecoder will raise an error the moment it finds one - Buffer recycling has been vanquished (for now), with video frames always allocated in the decoder - Produce/ConsumeVideoFrame() has been replaced by Read() - Video decode byte statistics are only updated if more than 0 bytes were decoded - FFmpegVideoDecodeEngine no longer attempts to preroll Review URL: http://codereview.chromium.org/8417019 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@108612 0039d316-1c4b-4281-b951-d872f2087c98
* Fix crash on unsupported pixel formats.dalecurtis@chromium.org2011-11-022-14/+30
| | | | | | | | | | | | | | Replaces the NOTREACHED() calls with DLOG(WARNING), avoids creating frames when invalid pixel formats are detected, and finally calls OnInitializeComplete(false) in FFmpegVideoDecodeEngine::Initialize. BUG=101803 TEST=Ran unittests. Ran test video. Review URL: http://codereview.chromium.org/8437021 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@108338 0039d316-1c4b-4281-b951-d872f2087c98
* refactor video capture in renderer processwjia@chromium.org2011-11-011-1/+0
| | | | | | | | | | | | | This patch is a split of http://codereview.chromium.org/8304017/ . 1. Since all clients of VideoCaptureImpl accept pixel frames with different dimension than requested, there is no need to distinguish them. Remove "resolution_fixed" in media::VideoCapture::VideoCaptureCapability. 2. VideoCaptureImpl is now taking the largested dimension from client requests and send it to browser process. 3. remove some unnecessary code in VideoCaptureModuleImpl since webrtc ViE can handle different frame size. BUG=none TEST=trybots Review URL: http://codereview.chromium.org/8400084 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@108126 0039d316-1c4b-4281-b951-d872f2087c98
* Remove media::VideoDecodeContext as it has no implementation and is unused.scherkus@chromium.org2011-10-287-178/+5
| | | | | | Review URL: http://codereview.chromium.org/8417016 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@107796 0039d316-1c4b-4281-b951-d872f2087c98
* Miscellaneous header file cleanup for src/media.scherkus@chromium.org2011-10-281-1/+0
| | | | | | Review URL: http://codereview.chromium.org/8418017 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@107772 0039d316-1c4b-4281-b951-d872f2087c98
* Remove DemuxerStream::GetAVStream() once and for all.scherkus@chromium.org2011-10-272-19/+10
| | | | | | | | | We now use AudioDecoderConfig and VideoDecoderConfig to pass decoder initialization information. Review URL: http://codereview.chromium.org/8341033 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@107494 0039d316-1c4b-4281-b951-d872f2087c98
* fix v4l support on openbsdrobert.nagy@gmail.com2011-10-251-0/+4
| | | | | | | | | | | | | media/video/capture/linux/video_capture_device_linux.cc: OpenBSD has V4L support but the header's filename is different BUG= TEST= Review URL: http://codereview.chromium.org/8380018 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@107227 0039d316-1c4b-4281-b951-d872f2087c98
* Adding VideoCaptureDevice for Mac.mflodman@chromium.org2011-10-185-4/+473
| | | | | | | | | | | Adding dependency on QTKit and CoreVideo. BUG= TEST= Review URL: http://codereview.chromium.org/8177008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@106036 0039d316-1c4b-4281-b951-d872f2087c98
* Fix ASAN error in FFmpegVideoDecodeEngineTest.DecodeFrame_SmallerHeight ↵scherkus@chromium.org2011-10-051-6/+15
| | | | | | | | | | introduced in r103961. VideoFrame dimensions don't change while AVCodecContext dimensions do, so make sure we keep using AVCodecContext for all dimension calculations for now. Review URL: http://codereview.chromium.org/8133011 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@104035 0039d316-1c4b-4281-b951-d872f2087c98
* Fix support for yuv_422 pixel format.scherkus@chromium.org2011-10-042-12/+14
| | | | | | | | | | | | Added pix_fmt field to the VideoDecoderConfig class. The pixel format is passed to the codec_context_ and used to correctly initialize VideoFrames. Patch by shadi@chromium.org: http://codereview.chromium.org/8052002/ BUG=95642 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@103961 0039d316-1c4b-4281-b951-d872f2087c98
* Clean up VideoDecoderConfig and replace VideoCodecInfo with a bool.scherkus@chromium.org2011-10-013-41/+16
| | | | | | | | | | | Similar to AudioDecoderConfig that was introduced in r102183, add Initialize() and IsValidConfig() to VideoDecoderConfig and update documentation. This helps pave the way to remove DemuxerStream::GetAVStream(). Since natural_size isn't used by neither VideoDecoderConfig nor VideoDecodeEngines, remove it from both and replace VideoCodecInfo with a bool. Review URL: http://codereview.chromium.org/8084021 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@103603 0039d316-1c4b-4281-b951-d872f2087c98
* gcc 4.6 warnings cleanuppph34r@gmail.com2011-09-301-6/+1
| | | | | | | | | | BUG=87490 TEST=compile with gcc 4.6 Review URL: http://codereview.chromium.org/8050008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@103513 0039d316-1c4b-4281-b951-d872f2087c98
* Follow-up cleanup promised during r103376's CR.fischman@chromium.org2011-09-301-1/+0
| | | | | | | | | | | | | | | | | | | | | | | - mock_task.{h,cc} is gone. - MockCallback is now MockClosure, and its commentary brought up to date (the commentary checked in was a mix of old and attempt-at-new that never materialized) - NewExpectedCallback is NewExpectedClosure. A bit of background on FooCallback vs. FooCB: when acolwell@ & I did the first conversions to the new world, everything was named FooCallback. I proposed using FooCB for the migrated ones as a way to both easily visually differentiate as well as save characters (!). Now that we have an additional "don't typedef Closures" guideline I like having FooCB for non-closure new-style callbacks, and FooClosure for new-style closures. BUG=none TEST=trybots Review URL: http://codereview.chromium.org/8085017 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@103464 0039d316-1c4b-4281-b951-d872f2087c98
* Here are more exports needed for content_unittests to link in the component ↵dpranke@chromium.org2011-09-301-1/+2
| | | | | | | | | | | | build. R=jam@chromium.org, darin@chromium.org BUG=90442 TEST= Review URL: http://codereview.chromium.org/8054037 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@103394 0039d316-1c4b-4281-b951-d872f2087c98
* Convert all of media/ over to the new base::{Bind,Callback} hotness.fischman@chromium.org2011-09-294-25/+33
| | | | | | | | | | | | | | | | | | | Mostly this was a rote conversion, replacing: - Pass-by-pointer CallbackN<>'s become pass-by-const-ref Callback<>'s. - scoped_ptr<CallbackN<>> members become Callback<> members. - several dedicated FooCallback typedefs became base::Closure. Because it was only used in a small handful of places and only in one place profitably, I deleted AutoCallbackRunner. Because it tickles a Bind bug I disabled mfdecoder in .gyp (about to get deleted in a scherkus CL). BUG=none TEST=media_tests, trybots Review URL: http://codereview.chromium.org/8071007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@103376 0039d316-1c4b-4281-b951-d872f2087c98
* Properly scope the lifetime of the |PPB_VideoCapture_Impl|.viettrungluu@chromium.org2011-09-282-1/+15
| | | | | | | | | | | | | | | | | | Since it/its proxy are |media::VideoCapture::EventHandler|s, they must remain alive from the |StartCapture()| call until |OnRemoved()| is received. (Precisely, the |PPB_VideoCapture_Impl| owns a |PlatformVideoCapture(Impl)|, which owns a |VideoCaptureHandlerProxy|. Keeping the |PPB_VideoCapture_Impl| alive keeps all these things alive.) BUG=none TEST=Closing a page with a PPAPI plugin using the video capture interface doesn't cause the renderer to crash. Other similar things should also work more or less properly (and at least not crash). Review URL: http://codereview.chromium.org/8052023 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@103138 0039d316-1c4b-4281-b951-d872f2087c98
* Add OnRemoved() in VideoCapture::EventHandler APIwjia@chromium.org2011-09-273-0/+11
| | | | | | | | | | This is to allow client to know when the event handler can be deleted. BUG=none TEST=trybots Review URL: http://codereview.chromium.org/8037055 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@103016 0039d316-1c4b-4281-b951-d872f2087c98
* Introduce AudioDecoderConfig to migrate away from GetAVStream().scherkus@chromium.org2011-09-211-2/+4
| | | | | | | | | Instead add DemuxerStream::audio_decoder_config() to break FFmpegAudioDecoder's dependency on the AVCodecContext object maintained by FFmpegDemuxer. Review URL: http://codereview.chromium.org/7867051 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@102183 0039d316-1c4b-4281-b951-d872f2087c98
* Delete Tracked, and move Location to its own file.ajwong@chromium.org2011-09-211-0/+1
| | | | | | | | | | | | | The Birth/Death tracking of tasks has been moved out-of-band into MessageLoop's PendingTask structure. Thus, Task no longer needs to inherit from Tracked. Since Task was the only child of Tracked, delete the Tracked class and move Location to its own file. BUG=none TEST=builds Review URL: http://codereview.chromium.org/7879006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@102132 0039d316-1c4b-4281-b951-d872f2087c98
* Reland r101418: Fix aspect ratio and clarify video frame dimensionsvrk@google.com2011-09-193-31/+27
| | | | | | | | | | | | Fixes shared build errors for windows and linux. BUG=18941,94861 TEST=shared builds compile TBR=acolwell Review URL: http://codereview.chromium.org/7932005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@101808 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 101418 - Fix aspect ratio and clarify video frame dimensionsvrk@google.com2011-09-163-27/+31
| | | | | | | | | | | | BUG=18941,94861 TEST=video-aspect-ratio.html Review URL: http://codereview.chromium.org/7864009 TBR=vrk@google.com Review URL: http://codereview.chromium.org/7919006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@101420 0039d316-1c4b-4281-b951-d872f2087c98
* Fix aspect ratio and clarify video frame dimensionsvrk@google.com2011-09-163-31/+27
| | | | | | | | | BUG=18941,94861 TEST=video-aspect-ratio.html Review URL: http://codereview.chromium.org/7864009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@101418 0039d316-1c4b-4281-b951-d872f2087c98
* add workarounds for some popular webcam.wjia@chromium.org2011-09-083-48/+65
| | | | | | | | | | | | | 1. bring device back to normal from bad state by querying all controls. 2. call ioctl with VIDIOC_TRY_FMT twice. 3. incorporate change from http://codereview.chromium.org/7753002/. "Improve stability when using a Logitech 9000 camera on Linux. Changed so that the device driver is opened and closed on the same thread as all the v4l2 calls." BUG=94134 TEST= Review URL: http://codereview.chromium.org/7743002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100165 0039d316-1c4b-4281-b951-d872f2087c98
* Replace the use of an int32* with an explicit profile for decoder configuration.fischman@chromium.org2011-09-011-145/+22
| | | | | | | | | | | | | | Replaces the error-prone, overly-general, error-containing, and brittle manually-terminated array-of-ints holding name/value pairs (except for names that don't take values) with a simple profile parameter (specifying only information we actually use today). BUG=none TEST=trybots, ovdatest, gles2 Review URL: http://codereview.chromium.org/7779001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@99111 0039d316-1c4b-4281-b951-d872f2087c98
* Disables camera hardware dependent tests.perkj@chromium.org2011-08-291-19/+29
| | | | | | | | | | | Also increase the time the test is allowed to wait for a video frame. TEST= BUG=94134 Review URL: http://codereview.chromium.org/7757002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@98691 0039d316-1c4b-4281-b951-d872f2087c98
* Create media.dll / libmedia.so.darin@chromium.org2011-08-297-9/+13
| | | | | | | | | This is a re-attempt at landing http://codereview.chromium.org/7572040, now with a DEPS roll to pick up the changes made to ffmpeg by http://codereview.chromium.org/7778004/ so that media_unittests run properly on Linux. TBR=fischman Review URL: http://codereview.chromium.org/7775004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@98600 0039d316-1c4b-4281-b951-d872f2087c98
* Completely random typo fixes and style corrections cleanup.fischman@chromium.org2011-08-251-3/+3
| | | | | | | | | | | | | Accumulated these while spelunking into the fast-forward issue, but now that's on hold figured I'd send these out for cleanup. BUG=none TEST=trybots Review URL: http://codereview.chromium.org/7726002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@98194 0039d316-1c4b-4281-b951-d872f2087c98
* Reland r97895: Use surface_width/height instead of coded_width/coded_height ↵vrk@chromium.org2011-08-242-8/+8
| | | | | | | | | | | | | | | for VideoFrame size The media_unittests needed to be adjusted to reflect the change made in the dimensions used to create VideoFrames in FFmpegVideoDecodeEngine. BUG=91506 TEST=media_unittests Review URL: http://codereview.chromium.org/7714016 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@98002 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 97892 - Use surface_width/height instead of coded_width/coded_height ↵vrk@google.com2011-08-231-2/+2
| | | | | | | | | | | | | | | | for VideoFrame size It appears that "surface" height/width refers to the visible height/width of the frame. FFmpegVideoDecodeEngine should use these dimensions to create VideoFrames rather than the coded width/height. BUG=91506 TEST=NONE Review URL: http://codereview.chromium.org/7624055 TBR=vrk@google.com Review URL: http://codereview.chromium.org/7719005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@97895 0039d316-1c4b-4281-b951-d872f2087c98