summaryrefslogtreecommitdiffstats
path: root/media/tools/player_x11
Commit message (Collapse)AuthorAgeFilesLines
* Replace Pipeline::SetNetworkActivity() with BufferedDataSource -> ↵fischman@chromium.org2012-06-121-2/+1
| | | | | | | | | | | | | WebMediaPlayerImpl callback. This removes a bunch of plumbing that's completely unnecessary. Concretely: Before: BufferedResourceLoader -> BufferedDataSource -> DemuxerHost -> DataSourceHost -> Pipeline -> WebMediaPlayerImpl After: BufferedResourceLoader -> BufferedDataSource -> WebMediaPlayerImpl Review URL: https://chromiumcodereview.appspot.com/10535101 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@141619 0039d316-1c4b-4281-b951-d872f2087c98
* RefCounted types should not have public destructors, Linux fixesrsleevi@chromium.org2012-05-182-3/+5
| | | | | | | | | | BUG=123295 TEST=none Review URL: https://chromiumcodereview.appspot.com/10392152 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@137893 0039d316-1c4b-4281-b951-d872f2087c98
* ui: Move gl/ directory out of gfx/, up to ui/.tfarina@chromium.org2012-05-122-3/+3
| | | | | | | | | | BUG=104040 R=ben@chromium.org TBR=tony@chromium.org Review URL: https://chromiumcodereview.appspot.com/10392068 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@136777 0039d316-1c4b-4281-b951-d872f2087c98
* Reland r136679, reverted in r136753: Delete MockDataSource, remove ↵fischman@chromium.org2012-05-121-2/+1
| | | | | | | | | | | | | | | IsLocalSource & IsSeekable from Demuxer/Pipeline. MockDataSource was completely unused, and Is{LocalSource,Seekable} being in the Demuxer is leftover craxy from the DataSourceFactory / DemuxerFactory / DownloadRateMonitor days, may they forever RIP. BUG=120426 TBR=scherkus Review URL: https://chromiumcodereview.appspot.com/10392069 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@136770 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 136679 - Delete MockDataSource, remove IsLocalSource & IsSeekable ↵tkent@chromium.org2012-05-121-1/+2
| | | | | | | | | | | | | | from Demuxer/Pipeline. It broke LayoutTests/http/tests/media/media-source/webm/video-media-source-seek.html and video-media-source-state-changes.html. http://test-results.appspot.com/dashboards/flakiness_dashboard.html#showExpectations=true&showLargeExpectations=true&tests=http%2Ftests%2Fmedia%2Fmedia-source%2Fwebm%2Fvideo-media-source-seek.html%2Chttp%2Ftests%2Fmedia%2Fmedia-source%2Fwebm%2Fvideo-media-source-state-changes.html Review URL: https://chromiumcodereview.appspot.com/10378100 TBR=fischman@chromium.org Review URL: https://chromiumcodereview.appspot.com/10378119 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@136753 0039d316-1c4b-4281-b951-d872f2087c98
* Delete MockDataSource, remove IsLocalSource & IsSeekable from Demuxer/Pipeline.fischman@chromium.org2012-05-111-2/+1
| | | | | | | | | | | | | MockDataSource was completely unused, and Is{LocalSource,Seekable} being in the Demuxer is leftover craxy from the DataSourceFactory / DemuxerFactory / DownloadRateMonitor days, may they forever RIP. BUG=120426 Review URL: https://chromiumcodereview.appspot.com/10378100 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@136679 0039d316-1c4b-4281-b951-d872f2087c98
* Rename AudioRendererBase and AudioRendererAlgorithmBasevrk@chromium.org2012-04-091-2/+2
| | | | | | | | | | | | | Neither of these classes are base classes anymore, so rename to AudioRendererImpl and AudioRendererAlgorithm respectively BUG=116645 TEST=media_unittests; chrome compiles and runs audio/video Review URL: http://codereview.chromium.org/9826028 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@131481 0039d316-1c4b-4281-b951-d872f2087c98
* Merge AudioRendererImpl and AudioRendererBase; add NullAudioSinkvrk@chromium.org2012-04-061-2/+4
| | | | | | | | | | | | | | | This CL removes AudioRendererImpl and replaces it with AudioRendererBase. NullAudioRenderer is also removed and replaced with NullAudioSink. Also, a subtle bug is fixed in AudioRendererBase to allow for smooth video playback when running Chrome with the --disable-audio flag. BUG=119549,116645 TEST=media_unittests, playing video on Chrome/content_shell with and without --disable-audio flag should look identical Review URL: http://codereview.chromium.org/9826023 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@131089 0039d316-1c4b-4281-b951-d872f2087c98
* Move media/audio files into media namespace (relanding)vrk@google.com2012-04-031-2/+2
| | | | | | | | | | BUG=115187 TEST=compiles and runs without breaking audio tag; media_unittests, content_unittests TBR=scherkus,jam Review URL: https://chromiumcodereview.appspot.com/9968054 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@130288 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 130180 - Move media/audio files into media namespacevrk@google.com2012-04-021-2/+2
| | | | | | | | | | | | BUG=115187 TEST=compiles and runs without breaking audio tag; media_unittests, content_unittests Review URL: https://chromiumcodereview.appspot.com/9805001 TBR=vrk@google.com Review URL: https://chromiumcodereview.appspot.com/9965076 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@130182 0039d316-1c4b-4281-b951-d872f2087c98
* Move media/audio files into media namespacevrk@google.com2012-04-021-2/+2
| | | | | | | | | BUG=115187 TEST=compiles and runs without breaking audio tag; media_unittests, content_unittests Review URL: https://chromiumcodereview.appspot.com/9805001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@130180 0039d316-1c4b-4281-b951-d872f2087c98
* Remove DemuxerFactory and URL parameter from Pipeline.scherkus@chromium.org2012-04-021-4/+4
| | | | | | | | | | | | Since Demuxers still require asynchronous initialization DemuxerFactory::Build() has been replaced with Demuxer::Initialize(). Since FFmpegDemuxer is the only Demuxer requiring a DataSource it is now passed in via FFmpegDemuxer's constructor. Now that Demuxer::set_host() is guaranteed to be called prior to initialization we're able to tighten up some code inside ChunkDemuxer. We should still nuke set_host() (see bug 111585) but I'll leave that for a future CL. BUG=110804, 110809 Review URL: https://chromiumcodereview.appspot.com/9860027 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@130165 0039d316-1c4b-4281-b951-d872f2087c98
* Move DataSource::SetPreload() to BufferedDataSource.scherkus@chromium.org2012-03-312-6/+0
| | | | | | | | | | Preload is a concept that only applies to HTML media elements and is only implemented by BufferedDataSource. Since the preload value is known when BufferedDataSource is created by WebMediaPlayerImpl, we can remove a lot of unnecessary plumbing in the media code. TEST=amount of data buffered (either visible on controls or via buffered attribute) is the same for each preload value (none, metadata, and auto). http://mastodon.sea/demos/preload contains a test page. Review URL: https://chromiumcodereview.appspot.com/9864022 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@130004 0039d316-1c4b-4281-b951-d872f2087c98
* Don't report errors twice to pipeline clients.fischman@chromium.org2012-03-271-19/+3
| | | | | | | | Also convert Pipeline::Stop() to take a Closure instead of a PipelineStatusCB since the status is always OK for a Stop. Review URL: http://codereview.chromium.org/9863028 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@129256 0039d316-1c4b-4281-b951-d872f2087c98
* Replace size_t with int in a few media classes.scherkus@chromium.org2012-03-272-4/+4
| | | | | | | | This addressed TODOs in DataSource::Read() and AudioRendererAlgorithmBase, which led to converting Buffer and SeekableBuffer as well. Review URL: https://chromiumcodereview.appspot.com/9854031 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@129140 0039d316-1c4b-4281-b951-d872f2087c98
* Add frame hashing to PipelineIntegrationTests.dalecurtis@chromium.org2012-03-171-1/+2
| | | | | | | | | | | | | | | | | | | | Also adds support for disabling frame dropping in the video renderer so we can be sure we've seen every frame coming out. Testing shows that there are issues with threading in the FFmpeg H264 decoder. Running with --video-threads=1 will yield different results than running multithreaded. We should probably increase the breadth of our basic playback testing once we've got a better handle on threaded decoding in FFmpeg. BUG=118688 TEST=media_unittests, ffmpeg_regression_tests. Review URL: http://codereview.chromium.org/9716008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@127320 0039d316-1c4b-4281-b951-d872f2087c98
* Create video and audio decoder threads on demand.tommi@chromium.org2012-03-111-2/+6
| | | | | | | | | | | | | Instead of creating these worker threads when the decoder objects are instantiated, we now do it when Initialize is called. Doing this avoids spinning a video decoder thread for an audio tag. BUG=114699,116873 TEST=Thread count in the renderer process goes up by 3 and not 4 when an audio element is created. Review URL: https://chromiumcodereview.appspot.com/9632024 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@126077 0039d316-1c4b-4281-b951-d872f2087c98
* Rename StatisticsCallback to StatisticsCB and DataSource::ReadCallback to ↵xhwang@chromium.org2012-03-102-5/+5
| | | | | | | | | | | | | | | ReadCB. To comply with naming convention in media code. Renaming only, no other changes. In rtc_video_decoder.cc, explicitly use media::VideoFrame so not to confuse with cricket::VideoFrame. BUG=none TEST=none Review URL: http://codereview.chromium.org/9634012 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@125959 0039d316-1c4b-4281-b951-d872f2087c98
* Replace use of XKeycodeToKeysym() with XkbKeycodeToKeysym() in player_x11.scherkus@chromium.org2012-03-081-2/+2
| | | | | | | | | | BUG=117274 TEST=player_x11 comples with gcc 4.6.3 / ubuntu 12.04 TBR=bungeman@chromium.org Review URL: https://chromiumcodereview.appspot.com/9632006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@125653 0039d316-1c4b-4281-b951-d872f2087c98
* Fold media::MessageLoopFactoryImpl into media::MessageLoopFactory.scherkus@chromium.org2012-03-061-2/+2
| | | | | | | | We haven't had a second implementation of the interface since MessageLoopFactory was added in r71548. Review URL: https://chromiumcodereview.appspot.com/9597016 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@125064 0039d316-1c4b-4281-b951-d872f2087c98
* Convert uses of int ms to TimeDelta in media.tedvessenes@gmail.com2012-03-051-3/+7
| | | | | | | | | | R=acolwell@chromium.org BUG=108171 Review URL: http://codereview.chromium.org/9582042 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@124971 0039d316-1c4b-4281-b951-d872f2087c98
* Make AudioManager non-refcounted.tommi@chromium.org2012-02-151-2/+2
| | | | | | | | | BUG=107087 TEST=media and content unit tests Review URL: https://chromiumcodereview.appspot.com/9382040 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@122102 0039d316-1c4b-4281-b951-d872f2087c98
* Add DataSource logging to player_x11.scherkus@chromium.org2012-02-153-30/+144
| | | | | | | | | | We issue the same read patterns regardless of whether the file lives on disk or over the network. Logging all DataSource operations helps us spot patterns that cause additional round trips when accessed over HTTP. DataSource logging can be enabled by passing --v=1 to player_x11. You can also force streaming read mode by passing --streaming to player_x11. Review URL: https://chromiumcodereview.appspot.com/9378016 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@122051 0039d316-1c4b-4281-b951-d872f2087c98
* Remove "high"-latency audio code pathvrk@chromium.org2012-02-031-8/+3
| | | | | | | | | | | | | | Chrome only uses the low-latency code path. This CL removes the unused, alternative "high"-latency code path. player_x11 and player_wtl use the high-latency code path, so sound is disabled from those players for now. BUG=NONE TEST=builds; content_unittests Review URL: http://codereview.chromium.org/9121062 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@120436 0039d316-1c4b-4281-b951-d872f2087c98
* Fold Pipeline::Init() into Pipeline::Start().scherkus@chromium.org2012-01-301-1/+4
| | | | | | | | Instead of having clients optionally call Init() (i.e., player_x11, player_wtl) they are forced to pass null callbacks into Start(). Review URL: https://chromiumcodereview.appspot.com/9269022 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@119723 0039d316-1c4b-4281-b951-d872f2087c98
* Replace DataSourceFactory with explicitly initialized DataSources.scherkus@chromium.org2012-01-191-5/+10
| | | | | | | | | BUG=107324 TEST=layout tests Review URL: https://chromiumcodereview.appspot.com/9243016 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@118338 0039d316-1c4b-4281-b951-d872f2087c98
* Fold media::PipelineImpl into media::Pipeline as there is only one ↵scherkus@chromium.org2012-01-181-5/+5
| | | | | | | | | | implementation of the interface. Three years ago I thought that having a separate interface would be A Good Thing. Needless to say there has never been a case where having an interface defintion of Pipeline came in handy as all clients create and access PipelineImpl objects directly. Review URL: http://codereview.chromium.org/9243025 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@118082 0039d316-1c4b-4281-b951-d872f2087c98
* Take advantage of the new Pass() machinery on scoped_ptr{,_malloc}.fischman@chromium.org2012-01-101-3/+4
| | | | | | | | | | | | | | | | | | | | | Pass() was announced in https://groups.google.com/a/chromium.org/d/topic/chromium-dev/RTd7rNxHjqk/discussion This CL replaces comments about ownership transfer (in all files whose paths contain media/) with the explicit passing of the appropriate scoper. The exceptions that are not touched by this CL: - scoped_refptr<> doesn't support Pass() and so is untouched. - media/audio code defines its own callback machinery, mimicking the old-style callbacks (pass by pointer, explicit deletes). I think that whole pile needs to be replaced with new-style (Bind) callbacks, so left it alone for now. BUG=none TEST=trybots Review URL: http://codereview.chromium.org/9015015 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@117009 0039d316-1c4b-4281-b951-d872f2087c98
* Fix invalid use of Base::Bind(,this) in ~ReferenceAudioRender.dalecurtis@chromium.org2012-01-063-12/+22
| | | | | | | | | | | | Also converts ~AudioOutputController to be protected and fixes an exit crash in player_x11. BUG=107864 TEST=Ran player_x11 --audio. Review URL: http://codereview.chromium.org/9111003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@116599 0039d316-1c4b-4281-b951-d872f2087c98
* Delete media::VideoFrame::planes() and ::planes_ since they can be derived ↵fischman@chromium.org2011-12-292-9/+6
| | | | | | | | | | | | | | | from format(). This lets us delete craxy code that first asks for format() and then ensures that the world hasn't gone crazy and given an RGB frame 4 planes (for example). BUG=107043 TEST=trybots. Review URL: http://codereview.chromium.org/8989067 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@115942 0039d316-1c4b-4281-b951-d872f2087c98
* Set opaque on the WebMediaPlayerClient based on the decoderdanakj@chromium.org2011-12-231-2/+6
| | | | | | | | | | BUG= TEST= Review URL: http://codereview.chromium.org/8989038 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@115720 0039d316-1c4b-4281-b951-d872f2087c98
* Replace subclass interface from VideoRendererBase with a single paint callback.scherkus@chromium.org2011-12-225-183/+150
| | | | | | | | | | As a result, many of the existing VideoRendererBase subclasses have been simplified or even removed entirely. BUG=28208 Review URL: http://codereview.chromium.org/8999029 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@115583 0039d316-1c4b-4281-b951-d872f2087c98
* Remove the AudioManager singleton.tommi@chromium.org2011-12-121-7/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Unit tests now instantiate their own AudioManager and can choose to use the default one or provide their own mock implementation without having to worry about conflicting with the singleton. The teardown sequence of the AudioManager and its thread has been cleaned up significantly and I don't think it has been completely tested before as the audio thread was terminated before all objects that belonged to the thread had a chance to do cleanup. The AudioManager unit tests do not use the actual audio thread, so this part seems to have been left out. In Chrome, the AudioManager instance is now owned by BrowserProcessImpl and always constructed on the UI thread. This instance is then shared in the same way that several other 'manager' type objects are shared to 'content' code, via content::ResourceContext. Audio specific classes do though receive a direct pointer to the AudioManager and are required to do proper reference counting if they need to hold onto the instance. I chose to use the ResourceContext rather than direct use of g_browser_process to avoid requiring another singleton when writing relatively simple tests that touch the AudioManager. I added a couple of safeguards to guard against future regressions: - Not more than one instance of the AudioManager should be created. - The AudioManager should not be addrefed by its own thread. This can basically become a circular reference and prevent deterministic shutdown. Reviewers: Of course you're free to review everything, but here's the breakdown in terms of the bare minimum from the standpoint of "Owners approval". I'm asking Henrik to be the main reviewer of the entire patch (sorry!). Henrik: Everything minus the below, but it would be great if you could take a look at the whole thing, specifically media/audio. Pawel: I'd like you to take a generic look at this approach. The key areas as far as the singleton itself goes are in media/audio/audio_manager[_base].* and chrome/browser/browser_process*.* Satish: content/browser/speech/* media/audio/audio_manager_base.* (new reference counting code) Andrew: content/browser/renderer_host/media/* content/renderer/media/webrtc_audio_device_unittest.cc (Owner) Avi: content/browser/renderer_host/render_process_host_impl.cc content/browser/resource_context.* William: chrome/browser/profiles/profile_io_data.cc chrome/browser/browser_process*.* Robert: This is basically a heads up. I hope that I didn't break the OpenBSD implementation, but unfortunately I have no way of knowing for sure. Shijing: Please take a look at AudioManagerLinux. I replaced the set of active streams with a simple counter. BUG=105249 TEST=content_unittests, media_unittests, browser_tests. Review URL: http://codereview.chromium.org/8818012 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@114084 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
* Add OVERRIDE to media/.avi@chromium.org2011-11-232-6/+8
| | | | | | | | | BUG=104314 TEST=no change Review URL: http://codereview.chromium.org/8634013 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@111364 0039d316-1c4b-4281-b951-d872f2087c98
* Callback API Change: Reimplement Bind(); support IgnoreResult, full ↵ajwong@chromium.org2011-11-211-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | currying, and use less types. The main API change IgnoreResult() and fully currying. See unittest for what the new API looks like. The rest of the changes are done to support that. Previously, IgnoreReturn could not be used with WeakPtr<> Bind()s as it was applied after the fact to the Callback object. Now, IgnoreResult() wraps the function like Unretained(). As an incidental benefit, the new implementation gave us fully currying for free. Also, the new implementation scales better when supporting higher arities of functions. The new type growth is: (n^2 + 20n) / 2 as opposed to (3n^2 + 17n) / 2 where n == arity. For n = 6 and n=10, the new implementation has 81 and 155 templates respectively. The old implementation had 105 and 235 templates respectively. BUG=35233,98919,98542 TEST=existing unittests Review URL: http://codereview.chromium.org/8483003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@110975 0039d316-1c4b-4281-b951-d872f2087c98
* Remove media::VideoDecodeContext as it has no implementation and is unused.scherkus@chromium.org2011-10-281-2/+1
| | | | | | Review URL: http://codereview.chromium.org/8417016 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@107796 0039d316-1c4b-4281-b951-d872f2087c98
* Rename media::AudioRendererImpl to media::ReferenceAudioRendererImpl.fischman@chromium.org2011-10-071-2/+2
| | | | | | | | | | | | This removes the name-collision with ::AudioRendererImpl, which lives in content/renderer/media, and is actually the implementation used in chrome. BUG=none TEST=trybots Review URL: http://codereview.chromium.org/8201018 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@104587 0039d316-1c4b-4281-b951-d872f2087c98
* Convert all of media/ over to the new base::{Bind,Callback} hotness.fischman@chromium.org2011-09-295-23/+18
| | | | | | | | | | | | | | | | | | | 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
* Reland r101418: Fix aspect ratio and clarify video frame dimensionsvrk@google.com2011-09-192-3/+4
| | | | | | | | | | | | 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-162-4/+3
| | | | | | | | | | | | 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-162-3/+4
| | | | | | | | | 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
* Migrate Pipeline & PipelineImpl to PipelineStatusCB.acolwell@chromium.org2011-08-091-4/+7
| | | | | | | | | BUG=90214 TEST=PipelineImplTest.* Review URL: http://codereview.chromium.org/7484054 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@96096 0039d316-1c4b-4281-b951-d872f2087c98
* Log PipelineImpl events to MediaLog.scottfr@chromium.org2011-08-091-1/+2
| | | | | | | | | | BUG= TEST= Review URL: http://codereview.chromium.org/7584013 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@96090 0039d316-1c4b-4281-b951-d872f2087c98
* Replace VideoDecoder::media_format() with significantly simpler ↵scherkus@chromium.org2011-07-202-26/+45
| | | | | | | | | | | | | | | | | width()/height() methods. Clients don't really care about the surface type during initialization, hence its removal. Also removed a few methods from VideoRendererBase that didn't need to be there. Technically it's possible to do without width()/height() methods entirely, forcing clients to inspect VideoFrame objects for changes in dimensions and react accordingly, but that'll be a change for a different day. BUG=28206 TEST=media_unittests, unit_tests, layout_tests Review URL: http://codereview.chromium.org/7461016 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@93278 0039d316-1c4b-4281-b951-d872f2087c98
* Remove Gles2VideoRenderer from player_x11 and make X11 vs GL rendering a ↵scherkus@chromium.org2011-07-197-701/+52
| | | | | | | | | | | run-time option. Also removed AdaptiveDemuxer since it causes run-time checks at the moment. Review URL: http://codereview.chromium.org/7410008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@93074 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
* Removing defunct OpenMAX code (round 2).scherkus@chromium.org2011-06-281-20/+3
| | | | | | | | | | | 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
* Revert 87790 - Removing defunct OpenMAX code.scherkus@chromium.org2011-06-031-3/+20
| | | | | | | | | | | | | | 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 TBR=scherkus@chromium.org Review URL: http://codereview.chromium.org/7065060 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@87793 0039d316-1c4b-4281-b951-d872f2087c98
* Removing defunct OpenMAX code.scherkus@chromium.org2011-06-031-20/+3
| | | | | | | | | | | 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@87790 0039d316-1c4b-4281-b951-d872f2087c98