summaryrefslogtreecommitdiffstats
path: root/media
Commit message (Collapse)AuthorAgeFilesLines
* Fix VideoRendererBase so it doesn't trigger a DCHECK during pipeline teardown.acolwell@chromium.org2011-07-143-20/+100
| | | | | | | | | BUG=89095 TEST=VideoRendererBaseTest.GetCurrentFrame_AfterError, VideoRendererBaseTest.Error_DuringPaint, VideoRendererBaseTest.GetCurrentFrame_AfterStop Review URL: http://codereview.chromium.org/7360005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@92542 0039d316-1c4b-4281-b951-d872f2087c98
* Rename CommandLine::GetArgs(), update callers.msw@chromium.org2011-07-135-6/+6
| | | | | | | | | BUG=73195 TEST=none Review URL: http://codereview.chromium.org/7352006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@92448 0039d316-1c4b-4281-b951-d872f2087c98
* Added extending forward_capacity for large reads.slock@chromium.org2011-07-132-2/+6
| | | | | | | | | | BUG=42956 TEST=test_shell_tests Review URL: http://codereview.chromium.org/7227016 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@92435 0039d316-1c4b-4281-b951-d872f2087c98
* Overhauled OmxVideoDecodeAccelerator's state machine implementation and ↵fischman@chromium.org2011-07-132-30/+28
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | related code. This implementation makes explicit the transitions being effected and the states expected to be involved in each transition, instead of the previous implementation's use of implicit information (e.g. "we just went from Idle to Loaded, therefore we must be Abort()'ing"). This is not only much more readable but also enables implementation of Reset() (for Seek) and replaces Abort() (which never had a well-defined purpose that was consistent in documentation and implementation, nor a customer use-case). DID some TODOs in OmxVideoDecodeAccelerator: - Replaced error-code checking boilerplate w/ a failure-handling macro. - Simplified the component-name-getting dance in CreateComponent(). Updated the PPAPI interface: - Removed GetConfigs & AssignSysmemBuffers (neither of which ever came into its own) - Replaced Abort with Reset, and added Destroy (which still nobody calls, but that'll change shortly). - Cleaned up the interface's documentation. BUG=86122 TEST=OVDATest, gles2 sample plugin Review URL: http://codereview.chromium.org/7338010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@92383 0039d316-1c4b-4281-b951-d872f2087c98
* Remove the comments setting emands and vim tab width and expansion variables.tony@chromium.org2011-07-111-6/+0
| | | | | | | | | | | | | | | | | These were added in r28089 (http://codereview.chromium.org/256059) but are unnecessary bloat for everyone to carry around, even those that don't use emacs or vim. In an earlier change, I added editor config files in src/tools/emacs/ and src/tools/vim/ so users of the appropriate editor can source those instead. BUG=none TEST=none Review URL: http://codereview.chromium.org/7310019 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@92046 0039d316-1c4b-4281-b951-d872f2087c98
* Refactor ChunkDemuxer initialization so INFO & TRACKS elements are passed ↵acolwell@chromium.org2011-07-115-193/+186
| | | | | | | | | | | via appendData(). BUG=86536 TEST=ChunkDemuxerTest.* Review URL: http://codereview.chromium.org/7329026 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@92011 0039d316-1c4b-4281-b951-d872f2087c98
* Change audio renderer to communicate with host using low latency codepath.enal@chromium.org2011-07-072-1/+5
| | | | | | | | | | | | | | | | | | | | | | I.e. now host signals that it is ready to accept new data using sockets, not IPC messages. That should reduce audio latency. Right now we are using high latency code path by default, low latency would be used only when flag --enable-low-latency-audio is specified. Changed unit tests to test low latency code path, as it should become default soon. http://codereview.chromium.org/7253003 BUG=http://code.google.com/p/chromium/issues/detail?id=87640 TEST=Everything should just work. I changed audio_renderer_impl_unittest.cc to explicitly force low latency code path. Review URL: http://codereview.chromium.org/7253003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@91727 0039d316-1c4b-4281-b951-d872f2087c98
* The RTCVideoDecoder will eventially need to depend on third_party lib ↵wjia@chromium.org2011-07-077-556/+15
| | | | | | | | | | | | (webrtc, libjinle etc), which is not allowed in media. So move rtc_video_decoder* from media/filter/ to content/renderer/media/. patch by ronghuawu@google.com BUG=none TEST=unit_tests, media_unittests Review URL: http://codereview.chromium.org/7193001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@91721 0039d316-1c4b-4281-b951-d872f2087c98
* Fix chromeos build buster.acolwell@chromium.org2011-07-011-2/+2
| | | | | | TBR=scherkus@chromium.org git-svn-id: svn://svn.chromium.org/chrome/trunk/src@91328 0039d316-1c4b-4281-b951-d872f2087c98
* Fix heapchecker failure due to not deleting AVStream*[].scherkus@chromium.org2011-07-011-33/+39
| | | | | | | | | | | Refactored ChunkDemuxerTest to include a ShutdownDemuxer() that cleans up memory. BUG=none TEST=heapcheck passes for media_unittests Review URL: http://codereview.chromium.org/7290030 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@91267 0039d316-1c4b-4281-b951-d872f2087c98
* Glue ffmpeg to Chromeihf@chromium.org2011-07-0118-123/+98
| | | | | | | | | | | | | | | | | | | | 1) works for Linux, OSX, Windows 2) mostly renaming #defines 3) avcodec_thread_init is deprecated and has been removed, see http://ffmpeg.org/pipermail/ffmpeg-cvslog/2011-February/034326.html 4) will require a rebaseline of platform/chromium-mac/media/video-currentTime-set-expected.txt platform/chromium-win/media/video-currentTime-set-expected.txt with a change of -video.currentTime.toFixed(2) == '5.82' +video.currentTime.toFixed(2) == '5.81' BUG=87831 TESTS=ran Theora Testsuite, media tests, WebKit media tests Review URL: http://codereview.chromium.org/6993042 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@91259 0039d316-1c4b-4281-b951-d872f2087c98
* Remove app from a bunch of DEPS files.tfarina@chromium.org2011-06-301-4/+0
| | | | | | | | | | | BUG=72317 TEST=None R=rsesek@chromium.org Review URL: http://codereview.chromium.org/7218065 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@91179 0039d316-1c4b-4281-b951-d872f2087c98
* This is the VideoCaptureDevice implementation for windows.perkj@google.com2011-06-3013-0/+1998
| | | | | | | | | | DirectShow is used for controlling cameras and receiving video frames. It implements all COM functions needed to support cameras using DirectShow and don't depend on Microsofts base classes for DirectShow filters. It supersedes the patch http://codereview.chromium.org/6929064/. Review URL: http://codereview.chromium.org/7229013 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@91096 0039d316-1c4b-4281-b951-d872f2087c98
* Implement proper synchronization between HW video decode IPC and CommandBuffer.fischman@chromium.org2011-06-291-9/+3
| | | | | | | | | | | | | | | | | This is done by inserting tokens into the command-buffer stream when synchronization is needed, and adding a last-read/last-written token pair to each IPC message. This allowed me to remove the bogus FinishGL() calls from the gles2 sample pepper plugin. As part of this CL, the return value for VideoDecodeAccelerator::{Decode,Flush,Abort} changed from bool to void. These are all async methods so errors ought to be signaled using callbacks. BUG=none TEST=gles2 works, no crashes; trybots Review URL: http://codereview.chromium.org/7260008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90971 0039d316-1c4b-4281-b951-d872f2087c98
* Fix uninitialized variable error on official builds.acolwell@chromium.org2011-06-291-1/+1
| | | | | | TBR=scherkus@chromium.org git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90970 0039d316-1c4b-4281-b951-d872f2087c98
* Add ~ClusterBuilder() to make clang happy.acolwell@chromium.org2011-06-292-0/+2
| | | | | | | | BUG=86536 TEST=None TBR=scherkus@chromium.org git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90967 0039d316-1c4b-4281-b951-d872f2087c98
* Adding ChunkDemuxer implementation.acolwell@chromium.org2011-06-2927-84/+2804
| | | | | | | | | BUG=86536 TEST=ChunkDemuxerTest.* Review URL: http://codereview.chromium.org/7203002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90966 0039d316-1c4b-4281-b951-d872f2087c98
* GCC 4.6 -Wunused-but-set-variable cleanup.pph34r@gmail.com2011-06-282-1/+7
| | | | | | | | | | BUG=87490 TEST=None Review URL: http://codereview.chromium.org/7230023 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90818 0039d316-1c4b-4281-b951-d872f2087c98
* Added last_modified field to TemplateURL and database. Updated unit tests, ↵stevet@chromium.org2011-06-281-46/+19
| | | | | | | | | | including refactoring MockTimeProvider out of media/. BUG=None TEST=No visible changes. Ensure that changed unittests all pass. Review URL: http://codereview.chromium.org/7232023 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90764 0039d316-1c4b-4281-b951-d872f2087c98
* Do not depend on liballocator on Mac.glider@chromium.org2011-06-281-3/+7
| | | | | | | TBR=scherkus Review URL: http://codereview.chromium.org/7253040 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90750 0039d316-1c4b-4281-b951-d872f2087c98
* Link media_unittests with liballocator.glider@chromium.org2011-06-281-0/+7
| | | | | | | | | This should hotfix the issues with the heap checker bot. TBR=scherkus Review URL: http://codereview.chromium.org/7277037 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90746 0039d316-1c4b-4281-b951-d872f2087c98
* Removing defunct OpenMAX code (round 2).scherkus@chromium.org2011-06-2823-4616/+5
| | | | | | | | | | | All of this code hasn't been used in over a year and has been replaced by VideoDecodeAccelerator and it's corresponding OpenMAX implementation OmxVideoDecodeAccelerator. BUG=none TEST=the world still compiles Review URL: http://codereview.chromium.org/7066071 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90697 0039d316-1c4b-4281-b951-d872f2087c98
* Remove media::{Base,GLES,Sysmem}Buffer(const PP_{Base,GLES,SysmemBuffer}&) ↵fischman@chromium.org2011-06-271-9/+0
| | | | | | | | | | | | | | | ctors. Removed these pseudo-copy-constructors, since they look a lot like violations of dependency rules and since they're not necessary given the explicit constructors. BUG=none TEST=gles2.cc builds & works with this patched in. Review URL: http://codereview.chromium.org/7276005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90628 0039d316-1c4b-4281-b951-d872f2087c98
* Refactor the glib message-pump, and use it as the base for a gtk message ↵sadrul@chromium.org2011-06-241-0/+1
| | | | | | | | | | | | | | | | | | | pump and an X message pump. The changes: * Rename MessagePumpGlibX to MessagePumpX. * Rename MessagePumpForUI to MessagePumpGlib. * Move some stuff out of MessagePumpGlib, and into MessagePumpGtk and MessagePumpX. * Rename MessagePumpForUI::Observer to MessageObserver, moved the platform-specific implementations into MessagePumpGtk and MessagePumpX. Ditto for MessagePumpForUI::Dispatcher. MessagePumpX is independent of MessagePumpGtk. At the moment, MessagePumpX does process some GDK event, but once we have a complete native_widget_x, we can take out the GDK processing and things should continue to work. BUG=none TEST=existing message-pump tests. Review URL: http://codereview.chromium.org/7250001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90418 0039d316-1c4b-4281-b951-d872f2087c98
* Fix clang.jhawkins@chromium.org2011-06-221-2/+2
| | | | | | | | | | | BUG=none TEST=none R=vandebo@chromium.org Review URL: http://codereview.chromium.org/7241004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90123 0039d316-1c4b-4281-b951-d872f2087c98
* Coverity: Pass parameters by reference.jhawkins@chromium.org2011-06-2230-54/+68
| | | | | | | | | | | | | CID=13686,13687,13688,13709,13710,13711,13712,13713,13714,13718,13715,13716, 13717,13796,13797,13798,13811,13812,13813,14708,16329 BUG=none TEST=none R=vandebo@chromium.org Review URL: http://codereview.chromium.org/7218031 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90112 0039d316-1c4b-4281-b951-d872f2087c98
* Coverity: Initialize member variables.jhawkins@chromium.org2011-06-223-0/+4
| | | | | | | | | | | | CID=13721,14709,16392,17192,17585 BUG=none TEST=none R=kmadhusu@chromium.org Review URL: http://codereview.chromium.org/7206066 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90086 0039d316-1c4b-4281-b951-d872f2087c98
* Remove unused testing code from media/base/media_win.ccscherkus@chromium.org2011-06-221-19/+1
| | | | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/7230003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90051 0039d316-1c4b-4281-b951-d872f2087c98
* Coverity: Fix a leak in VideoCaptureDevice::Create.thestig@chromium.org2011-06-221-1/+4
| | | | | | | | | | BUG=none TEST=none CID=16736 Review URL: http://codereview.chromium.org/7217024 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90001 0039d316-1c4b-4281-b951-d872f2087c98
* Coverity: Initialize member variables.jhawkins@chromium.org2011-06-211-0/+1
| | | | | | | | | | | | | CID=9299,13805,14203,14284,14459,15435,15897,16023,16614,16773,16819,16820, 16908,16916 BUG=none TEST=none R=kmadhusu@chromium.org Review URL: http://codereview.chromium.org/7215027 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@89897 0039d316-1c4b-4281-b951-d872f2087c98
* Not allow compression when requesting multimediaenal@chromium.org2011-06-211-2/+3
| | | | | | | | | | | | | | | | because ffmpeg does not expect compressed files. (It does not make sense to compress audio/video anyways...) http://codereview.chromium.org/7044092 BUG=47381 TEST=http://www.deafmac.org/html5/grinchsample.mp4 (Please clear browser cache, otherwise it can contain original GZIP-ed file requested by old browser) Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=89532 Review URL: http://codereview.chromium.org/7044092 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@89858 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 89532 - Not allow compression when requesting multimediaakalin@chromium.org2011-06-171-3/+2
| | | | | | | | | | | | | | | | | because ffmpeg does not expect compressed files. (It does not make sense to compress audio/video anyways...) http://codereview.chromium.org/7044092 BUG=47381 TEST=http://www.deafmac.org/html5/grinchsample.mp4 (Please clear browser cache, otherwise it can contain original GZIP-ed file requested by old browser) Review URL: http://codereview.chromium.org/7044092 TBR=enal@chromium.org Review URL: http://codereview.chromium.org/7205011 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@89546 0039d316-1c4b-4281-b951-d872f2087c98
* Adding media/OWNERS.scherkus@chromium.org2011-06-171-0/+9
| | | | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/7173001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@89533 0039d316-1c4b-4281-b951-d872f2087c98
* Not allow compression when requesting multimediaenal@chromium.org2011-06-171-2/+3
| | | | | | | | | | | | | | because ffmpeg does not expect compressed files. (It does not make sense to compress audio/video anyways...) http://codereview.chromium.org/7044092 BUG=47381 TEST=http://www.deafmac.org/html5/grinchsample.mp4 (Please clear browser cache, otherwise it can contain original GZIP-ed file requested by old browser) Review URL: http://codereview.chromium.org/7044092 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@89532 0039d316-1c4b-4281-b951-d872f2087c98
* Remove msvs_guids from ipc, media, net, ppapi, printing, sandbox,tony@chromium.org2011-06-161-3/+0
| | | | | | | | | | sdch, skia, and testing. BUG=28727 Review URL: http://codereview.chromium.org/7165009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@89369 0039d316-1c4b-4281-b951-d872f2087c98
* Move ProvidePictureBuffers out of initialization and into the decode path.fischman@chromium.org2011-06-151-2/+5
| | | | | | | | | | | | | | | | | | | | | | | | | This is required for various reasons: - Clients don't necessarily know the dimensions of the video they feed to the decoder. It's the decoder's job to figure it out from the encoded stream, but it only gets a crack at that once Decode() is called, after initialization is done. - PPAPI plugins get to call PPB_* functions at will, which means that (to avoid having to code re-entrancy into everything on the browser side) PPB_* functions aren't allowed to call into PPP_* functions. So the previous call sequence of Initialize->ProvidePictureBuffers->CreateTextures would deadlock (requiring plugins to pre-create any textures they were going to get asked for in the future). Also a bit of minor cleanup & added TODOs for future cleanup, and changed GetConfigs()' signature to match the PPAPI version of itself (return bool instead of void). BUG=none TEST=OVDATest passes on kaen. Review URL: http://codereview.chromium.org/7147006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@89251 0039d316-1c4b-4281-b951-d872f2087c98
* Fix bug when unplugging an audio input device whilst using speech input on ↵allanwoj@chromium.org2011-06-154-5/+90
| | | | | | | | | | | | | Windows. Added a three second timeout for not receiving an OnData callback in AudioInputController and a one second timeout for not receiving any callback in PCMWaveInAudioInputStream. Also added a unit test fixture for it. BUG=79936 TEST=Unplug microphone while using speech input, browser should no longer hang. Review URL: http://codereview.chromium.org/7129057 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@89197 0039d316-1c4b-4281-b951-d872f2087c98
* Improve and unify Mac OS X run-time version checks.mark@chromium.org2011-06-141-4/+2
| | | | | | | | | | | | Don't use base::SysInfo::OperatingSystemVersionNumbers, because it calls Gestalt, which has a few bad properties. Introduce new functions that perform specific version checks. BUG=85972 TEST=base_unittests MacUtilTest.IsOSEllipsis Review URL: http://codereview.chromium.org/7144007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@89028 0039d316-1c4b-4281-b951-d872f2087c98
* Fix two bugs found by a new clang warning I'm currently testing:thakis@chromium.org2011-06-131-3/+4
| | | | | | | | | | | | | | | | | | | | | /Users/thakis/src/chrome-git/src/base/md5.cc:250:24: error: The sizeof expression in 'memset' has type 'struct Context *', the same type that the first argument has. The sizeof expression should probably have type 'struct Context' instead. [-Werror] memset(ctx, 0, sizeof(ctx)); /* In case it's sensitive */ ^ 1 error generated. /Users/thakis/src/chrome-git/src/media/video/capture/fake_video_capture_device.cc:75:32: error: The sizeof expression in 'memset' has type 'unsigned char *', the same type that the first argument has. The sizeof expression should probably have type 'unsigned char' instead. [-Werror] memset(fake_frame_.get(), 0, sizeof(fake_frame_.get())); ^ 1 error generated. The warning is up for review here: http://lists.cs.uiuc.edu/pipermail/cfe-commits/Week-of-Mon-20110613/042719.html BUG=none TEST=none Review URL: http://codereview.chromium.org/7003140 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@88855 0039d316-1c4b-4281-b951-d872f2087c98
* Uninitialized member in FakeVideoCaptureDevice.finnur@chromium.org2011-06-121-0/+1
| | | | | | | | | BUG=None TEST=None CID=16744 Review URL: http://codereview.chromium.org/7044110 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@88786 0039d316-1c4b-4281-b951-d872f2087c98
* Uninitialized member in VideoFrameBuffer.finnur@chromium.org2011-06-121-1/+6
| | | | | | | | | BUG=None TEST=None CID=17000 Review URL: http://codereview.chromium.org/6995134 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@88785 0039d316-1c4b-4281-b951-d872f2087c98
* Implement AssignGLESBuffers for VideoDecode PPAPIvrk@google.com2011-06-102-12/+7
| | | | | | | | | | | | | | Fills in implementation for AssignGLESBuffers where it was missing. Also updates OmxVideoDecodeAccelerator to reflect the changes. BUG=NONE TEST=NONE Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=88698 Review URL: http://codereview.chromium.org/6965010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@88733 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 88698 - Implement AssignGLESBuffers for VideoDecode PPAPIdavemoore@chromium.org2011-06-102-7/+12
| | | | | | | | | | | | | | | Fills in implementation for AssignGLESBuffers where it was missing. Also updates OmxVideoDecodeAccelerator to reflect the changes. BUG=NONE TEST=NONE Review URL: http://codereview.chromium.org/6965010 TBR=vrk@google.com Review URL: http://codereview.chromium.org/7044115 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@88702 0039d316-1c4b-4281-b951-d872f2087c98
* Implement AssignGLESBuffers for VideoDecode PPAPIvrk@google.com2011-06-102-12/+7
| | | | | | | | | | | | Fills in implementation for AssignGLESBuffers where it was missing. Also updates OmxVideoDecodeAccelerator to reflect the changes. BUG=NONE TEST=NONE Review URL: http://codereview.chromium.org/6965010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@88698 0039d316-1c4b-4281-b951-d872f2087c98
* Simplify AlsaPcmOutputStream and AudioManagerLinux. Code was thread-safe, ↵enal@chromium.org2011-06-085-273/+159
| | | | | | | | | | | | | | but now client and audio threads are the same, so we don't need locks/ref counts/etc. http://codereview.chromium.org/7117001 BUG=62588 Review URL: http://codereview.chromium.org/7117001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@88369 0039d316-1c4b-4281-b951-d872f2087c98
* 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
* Fix for CoreAudio stereo problem for unknown speakersannacc@chromium.org2011-06-081-2/+22
| | | | | | | | | | BUG=85100 TEST=HTML5 videos with stereo channel layout on Mac with unconfigured speakers can be heard. Review URL: http://codereview.chromium.org/7134001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@88339 0039d316-1c4b-4281-b951-d872f2087c98
* Random cleanup & improvements to OmxVideoDecodeAccelerator:fischman@chromium.org2011-06-061-3/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Rip out support for HW decode to sysmem buffers from OVDA (which doesn't work and for which we don't have a need (yet)). - Deliver on the promise of Picture to provide a bitstream buffer id from which the picture was decoded by propagating the bitstream buffer id from input buffers (BitstreamBuffer) to output buffers (Picture) through OMX_BUFFERHEADERTYPE's nTimeStamp field (HACK!). Now checking for this in the test. - Free output buffers uniformly: both dismiss from OMX & from client at the same time, during OnStateChangeExecutingToIdle, instead of freeing from OMX during OnStateChangeExecutingToIdle and from client during OnPortCommandFlush. - Clean up OVDA implementation: - Replaced unnecessary OMX_AllocateBuffer with OMX_UseBuffer. - Unify two parallel vector<>s (one of which is of pairs) into a single map<id, struct> to ease readability and to increase robustness (should be harder now to have these data items fall out of sync with each other). - Removed unnecessary output_buffer_count_ field. - Removed unused field output_buffer_size_. - Replaced use of explicitly-managed omx_buff_ids_ map with implicit use of pAppPrivate field on input buffer headers. BUG=none TEST=run the test per instructions in http://codereview.chromium.org/7112019 Review URL: http://codereview.chromium.org/6993051 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@87996 0039d316-1c4b-4281-b951-d872f2087c98
* Removing defunct GpuVideoDecoder and IpcVideoDecoder.scherkus@chromium.org2011-06-032-4/+0
| | | | | | | | | | | | These haven't been used in quite some time and have been replaced by the newer VideoDecoderAccelerator set of classes. BUG=none TEST=the world still compiles Review URL: http://codereview.chromium.org/6993016 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@87841 0039d316-1c4b-4281-b951-d872f2087c98
* Follow up CL to 6990065fgalligan@chromium.org2011-06-032-2/+8
| | | | | | | | | | | | Follow up CL to http://codereview.chromium.org/6990065/ BUG=75458 TEST=Open file out29367209.webm from Bug#75458 and video playback should be more than one frame. Review URL: http://codereview.chromium.org/7058020 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@87839 0039d316-1c4b-4281-b951-d872f2087c98