summaryrefslogtreecommitdiffstats
path: root/media
Commit message (Collapse)AuthorAgeFilesLines
* media bench report fpsfbarchard@chromium.org2010-08-032-18/+28
| | | | | | | | | | | also handles videos with 0 frames the same as videos with more than 0 frames, for consistency and throwing in a comment in yuv conversion BUG=none TEST=none Review URL: http://codereview.chromium.org/3073019 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@54842 0039d316-1c4b-4281-b951-d872f2087c98
* Adding two test cases to ffmpeg_unittest from user bug reports.scherkus@chromium.org2010-08-031-1/+10
| | | | | | | | | | | | Issue 47761 (short Ogg files don't play) is currently failing and requires fixing FFmpeg. Issue 50045 (MP4 moof atom playback issue) is currently passing as expected. BUG=47761,50045 TEST=ffmpeg_unittests Review URL: http://codereview.chromium.org/3043049 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@54796 0039d316-1c4b-4281-b951-d872f2087c98
* base: Rename EnvVarGetter to Environment.tfarina@chromium.org2010-08-032-3/+2
| | | | | | | | | | | | | | | Now EnvVarGetter do much more than getting environment variables. Per suggestion from Pawel in http://codereview.chromium.org/3043018/. BUG=None TEST=trybots Signed-off-by: Thiago Farina <tfarina@chromium.org> Review URL: http://codereview.chromium.org/3052034 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@54696 0039d316-1c4b-4281-b951-d872f2087c98
* Remove FFmpegDemuxer::first_seek_hack_ since underlying bugs have been fixed ↵scherkus@chromium.org2010-08-023-32/+3
| | | | | | | | | | | in FFmpeg. BUG=50460 TEST=layout tests, media tests Review URL: http://codereview.chromium.org/3073004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@54574 0039d316-1c4b-4281-b951-d872f2087c98
* This tool demonstrates the use of the Media Foundation H.264 decoder as a ↵imcheng@chromium.org2010-08-028-0/+1366
| | | | | | | | | | standalone Media Foundation Transform (MFT). TEST=none BUGS=none git-svn-id: svn://svn.chromium.org/chrome/trunk/src@54571 0039d316-1c4b-4281-b951-d872f2087c98
* Uninitialized member in AudioInputController.finnur@chromium.org2010-08-021-0/+1
| | | | | | | | | | BUG=None TEST=None CID=11839 Review URL: http://codereview.chromium.org/3078012 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@54546 0039d316-1c4b-4281-b951-d872f2087c98
* Add undeclared virtual destructors part 5ziadh@chromium.org2010-08-021-0/+2
| | | | | | | | | | | | | | | Preventative maintenance for abstract classes that do not declare virtual destructors. Base classes that do not declare their destructors as virtual could potentially lead to memory leaks. These files were discovered using the -Wnon-virtual-dtor flag in g++. r=jar BUG=47469 Review URL: http://codereview.chromium.org/3080013 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@54540 0039d316-1c4b-4281-b951-d872f2087c98
* fix another compile error on windows.nsylvain@chromium.org2010-07-311-0/+1
| | | | | | Review URL: http://codereview.chromium.org/3052032 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@54470 0039d316-1c4b-4281-b951-d872f2087c98
* Fix compile on windows.nsylvain@chromium.org2010-07-311-0/+1
| | | | | | Review URL: http://codereview.chromium.org/3020052 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@54467 0039d316-1c4b-4281-b951-d872f2087c98
* Convert more callers of the integer/string functions to usingbrettw@chromium.org2010-07-315-20/+24
| | | | | | | | | | string_number_conversions.h TEST=it compiles BUG=none Review URL: http://codereview.chromium.org/3013046 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@54454 0039d316-1c4b-4281-b951-d872f2087c98
* Revert "This tool demonstrates the use of the Media Foundation H.264 decoder ↵evan@chromium.org2010-07-298-1366/+0
| | | | | | | | as a standalone Media Foundation Transform (MFT). See README for more information." This reverts commit r54078, due to link failures. git-svn-id: svn://svn.chromium.org/chrome/trunk/src@54084 0039d316-1c4b-4281-b951-d872f2087c98
* This tool demonstrates the use of the Media Foundation H.264 decoder as a ↵imcheng@chromium.org2010-07-298-0/+1366
| | | | | | | | standalone Media Foundation Transform (MFT). See README for more information. Review URL: http://codereview.chromium.org/3044019 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@54078 0039d316-1c4b-4281-b951-d872f2087c98
* Fixing crash in buffer bar painting codevrk@google.com2010-07-281-0/+13
| | | | | | | | | | | | | | | Crash was because of inconsistent state caused by threading issues. The state of the buffer bar isn't so critical to need to be 100% accurate at all times, so this is fixed by capping the buffered_bytes_ and current_bytes to always be at legal values. BUG=49870 TEST=media_unittests TBR=hclam Review URL: http://codereview.chromium.org/3045021 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@54055 0039d316-1c4b-4281-b951-d872f2087c98
* ffmpeg roll for chromium source and windows binariesfbarchard@chromium.org2010-07-288-10/+11
| | | | | | | | | | Includes fix for XP Perf by reducing webm library size by 1.6 MB BUG=48037,50507 TEST=XP Perf and layout tests should pass Review URL: http://codereview.chromium.org/3030024 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@54053 0039d316-1c4b-4281-b951-d872f2087c98
* base/ header cleanup. Forward declaration instead of including.erg@google.com2010-07-286-1/+7
| | | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/3068004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@53969 0039d316-1c4b-4281-b951-d872f2087c98
* Revert r53885 "deps roll for new ffmpeg and switch to av_register_protocol2"jeremy@chromium.org2010-07-288-11/+10
| | | | | | | | Trying to track down cause of XP Perf bot regression. Review URL: http://codereview.chromium.org/3081002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@53926 0039d316-1c4b-4281-b951-d872f2087c98
* Updated some comments to point at bugs in ffmpeg_unittest.cc and ↵scherkus@chromium.org2010-07-282-1/+19
| | | | | | | | | | | | | | ffmpeg_audio_decoder.cc. No code change. BUG=49709,50457 TEST=none TBR=hclam Review URL: http://codereview.chromium.org/3077004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@53896 0039d316-1c4b-4281-b951-d872f2087c98
* deps roll for new ffmpeg and switch to av_register_protocol2fbarchard@chromium.org2010-07-288-10/+11
| | | | | | | | | BUG=48037 TEST=layout tests should still pass for media Review URL: http://codereview.chromium.org/3005034 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@53885 0039d316-1c4b-4281-b951-d872f2087c98
* Implement accurate seeking by decoding audio/video until we reach our ↵scherkus@chromium.org2010-07-285-5/+33
| | | | | | | | | | | | | | | desired timestamp. This fixes a host of seeking, synchronization and layout test issues but does incur a CPU performance hit while seeking. In the worst case a seek could take quite some time if the content was high resolution H.264 and we attempted to seek to right before a keyframe. We'll see how people like the new behaviour and back this change out if needed. BUG=48984 TEST=layout tests, media_unittests Review URL: http://codereview.chromium.org/3032030 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@53872 0039d316-1c4b-4281-b951-d872f2087c98
* Checking in FFmpeg regression testing framework.scherkus@chromium.org2010-07-272-0/+661
| | | | | | | | | | | Basically verifies that our assumptions on how FFmpeg works opening/seeking/parsing files remain intact. BUG=19865 TEST=run ffmpeg_unittests :) Review URL: http://codereview.chromium.org/3058004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@53818 0039d316-1c4b-4281-b951-d872f2087c98
* GTTF: more FRIEND_TEST_ALL_PREFIXESphajdan.jr@chromium.org2010-07-277-45/+56
| | | | | | | | | TEST=compile BUG=44549 Review URL: http://codereview.chromium.org/3064007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@53793 0039d316-1c4b-4281-b951-d872f2087c98
* media: player_wtl renderer forget to putCurrentFrame.jiesun@google.com2010-07-271-0/+1
| | | | | | | | forget to putCurrentFrame for this renderer. Review URL: http://codereview.chromium.org/3018021 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@53730 0039d316-1c4b-4281-b951-d872f2087c98
* use_system_ffmpeg broken on gentoo side.jiesun@google.com2010-07-261-2/+0
| | | | | | | | | | | | because if use "system" ffmpeg, AVFrame has no member named "owner". we need use #ifdef to use the single codec context, if system ffmpeg is used. but what if "system" is ffmpeg-mt? Review URL: http://codereview.chromium.org/2873068 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@53695 0039d316-1c4b-4281-b951-d872f2087c98
* ffmpeg seek to keyframe on oggfbarchard@chromium.org2010-07-261-4/+0
| | | | | | | | BUG=none TEST=do a seek in chrome on ogg. it should appear clean, not garbled. Review URL: http://codereview.chromium.org/3043021 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@53656 0039d316-1c4b-4281-b951-d872f2087c98
* media: preparation for recycle buffer.jiesun@google.com2010-07-227-58/+273
| | | | | | | | | | | | | | 1. add ffmpeg_video_allocator.cc/h 2. add omx_bufferheader type and av_frame type of video_frame. please see http://codereview.chromium.org/2992002/show for full commit. this is used to split that change to reviewable size of changelist. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=53348 Review URL: http://codereview.chromium.org/3006001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@53418 0039d316-1c4b-4281-b951-d872f2087c98
* Remove FFmpegDemuxer::current_timestamp_ and always pass ↵scherkus@chromium.org2010-07-223-17/+11
| | | | | | | | | | | | | | | AVSEEK_FLAG_BACKWARD when seeking. Previously we falsely assumed that AVSEEK_FLAG_BACKWARD was needed when seeking backwards in the media file. Turns out all this flag does is seek to a point in the file that is less-than-or-equal to the desired timestamp -- regardless of the current position in the file! Furthermore, the value of current_timestamp_ was never guaranteed to be valid as poorly muxed files tend to contain AV_NOPTS_VALUE packets. This all resulted in non-deterministic seeking behaviour depending on whether current_timestamp_ was valid or contained AV_NOPTS_VALUE. BUG=48722 TEST=covered by layout tests Review URL: http://codereview.chromium.org/3050011 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@53404 0039d316-1c4b-4281-b951-d872f2087c98
* Reverting 53348...bradnelson@google.com2010-07-227-273/+58
| | | | | | | | | | | | | Broke webkit build. BUG=None TEST=Nont TBR=jiesun Review URL: http://codereview.chromium.org/3015018 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@53352 0039d316-1c4b-4281-b951-d872f2087c98
* media: preparation for recycle buffer.jiesun@google.com2010-07-227-58/+273
| | | | | | | | | | | | 1. add ffmpeg_video_allocator.cc/h 2. add omx_bufferheader type and av_frame type of video_frame. please see http://codereview.chromium.org/2992002/show for full commit. this is used to split that change to reviewable size of changelist. Review URL: http://codereview.chromium.org/3006001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@53348 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 53266 - Testing a revert works removing a harmless comment CLfbarchard@chromium.org2010-07-221-4/+1
| | | | | | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/2883032 TBR=fbarchard@chromium.org Review URL: http://codereview.chromium.org/2832068 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@53293 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 53281 - ffmpeg update with vp3 patches.fbarchard@chromium.org2010-07-227-10/+9
| | | | | | | | | | | | | | switch to av_register_protocol2 video-played-collapse layout test fails and will be fixed later. BUG=48037 TEST=ffmpeg should still build/run Review URL: http://codereview.chromium.org/3010016 TBR=fbarchard@chromium.org Review URL: http://codereview.chromium.org/2805098 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@53285 0039d316-1c4b-4281-b951-d872f2087c98
* ffmpeg update with vp3 patches.fbarchard@chromium.org2010-07-227-9/+10
| | | | | | | | | | | switch to av_register_protocol2 video-played-collapse layout test fails and will be fixed later. BUG=48037 TEST=ffmpeg should still build/run Review URL: http://codereview.chromium.org/3010016 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@53281 0039d316-1c4b-4281-b951-d872f2087c98
* comment for what linearscale isfbarchard@chromium.org2010-07-211-1/+4
| | | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/2883032 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@53266 0039d316-1c4b-4281-b951-d872f2087c98
* Remove deprecated CommandLine::GetLooseValues(), rename to args().evan@chromium.org2010-07-215-53/+66
| | | | | | | | | | | | | | | It returned a wstring, when really we wanted the native encoded strings. Fixing the majority of callers actually simplified them in many cases because the callers wanted native strings too. Since I'm touching every caller, I gave it a more useful name. I'm not sure where "loose" came from but it never made sense to me. BUG=24672 Review URL: http://codereview.chromium.org/3028010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@53193 0039d316-1c4b-4281-b951-d872f2087c98
* Fix check_deps problem in my last commithclam@chromium.org2010-07-201-1/+1
| | | | | | | | | | Ouch I typed in the wrong path. Now it is correct. TBR=awong Review URL: http://codereview.chromium.org/3035013 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@53110 0039d316-1c4b-4281-b951-d872f2087c98
* Fix check_deps failure in my last commithclam@chromium.org2010-07-201-0/+1
| | | | | | | | | | Added "gfx/" to media/DEPS TBR=pinkerton Review URL: http://codereview.chromium.org/2861068 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@53104 0039d316-1c4b-4281-b951-d872f2087c98
* A tool using Source Reader in Media Foundation for H264 decodinghclam@chromium.org2010-07-205-0/+1040
| | | | | | | | | Reviewed: http://codereview.chromium.org/2812050/show Submitted for: imcheng@google.com Review URL: http://codereview.chromium.org/3003002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@53090 0039d316-1c4b-4281-b951-d872f2087c98
* media seek to any frame for ffmpegfbarchard@chromium.org2010-07-201-0/+4
| | | | | | | | | BUG=none TEST=should help layout tests when combined with a roll Review URL: http://codereview.chromium.org/2809061 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@53004 0039d316-1c4b-4281-b951-d872f2087c98
* file_util: deprecate remaining wstring functionsevan@chromium.org2010-07-168-53/+72
| | | | | | | | | | | This removes the last wstring-accepting functions from file_util on non-Windows platforms. BUG=24672 Review URL: http://codereview.chromium.org/3005005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@52755 0039d316-1c4b-4281-b951-d872f2087c98
* temporary fix for OMX port setting change eventwjia@google.com2010-07-162-7/+15
| | | | | | | | | BUG=none TEST=compiles Review URL: http://codereview.chromium.org/2868050 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@52702 0039d316-1c4b-4281-b951-d872f2087c98
* Add an AudioInputController to help with audio recording.satish@chromium.org2010-07-164-0/+409
| | | | | | | | | | | This is similar to the existing AudioOutputController class used for playback. Also adds a unit test which uses the new fake audio input stream, so this patch is dependant on http://codereview.chromium.org/2909015 TEST=no new user visible change to test, just some backend recording additions which are unused at the moment. BUG=none Review URL: http://codereview.chromium.org/2905010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@52617 0039d316-1c4b-4281-b951-d872f2087c98
* This callback is created byjiesun@google.com2010-07-151-5/+4
| | | | | | | | void PipelineImpl::FilterStateTransitionTask() and the ownership is transferred to Filter, filter should delete this callback. Review URL: http://codereview.chromium.org/2845044 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@52565 0039d316-1c4b-4281-b951-d872f2087c98
* Audio pause and seek betterhclam@chromium.org2010-07-152-19/+69
| | | | | | | | | | | | | BUG=39885 Pause for audio is now in effect in less than 200ms. Seek for audio is also much better as we play also very short amount after seek. Review URL: http://codereview.chromium.org/2931014 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@52560 0039d316-1c4b-4281-b951-d872f2087c98
* AudioController sets volume to 1.0 by defaulthclam@chromium.org2010-07-151-1/+1
| | | | | | | | | | | | | | NaCl audio stop working because the demo NaCl plugin doesn't set volume. We revert to the original behavior of AudioController by setting volume to 1.0 by default. TBR=neb BUG=49807 TEST=NaCl audio plays Review URL: http://codereview.chromium.org/2966014 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@52548 0039d316-1c4b-4281-b951-d872f2087c98
* Attempt four at landing the "histogram.h removed from message_loop.h" patch.erg@chromium.org2010-07-155-0/+5
| | | | | | | | | | | | | | Previously committed as r52349 and r52336. Related commits: r52367, r52364 and r52343. Rerunning trybots due to previous trybot breakage. TEST=none BUG=none Review URL: http://codereview.chromium.org/2965015 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@52496 0039d316-1c4b-4281-b951-d872f2087c98
* Add a fake audio input stream for testing purposes.satish@chromium.org2010-07-157-1/+201
| | | | | | | | | | This will be used by unit tests in subsequent patches. BUG=none TEST=no new features added and nothing new to test Review URL: http://codereview.chromium.org/2909015 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@52464 0039d316-1c4b-4281-b951-d872f2087c98
* Fixed ended event when playing audio on linux.sergeyu@chromium.org2010-07-143-43/+3
| | | | | | | | | | | AudioRendererBase::FillBuffers() sends ended event only when playback_delay == 0, but with ALSA output the delay was never set to 0 because it includes hardware delay. Changed ALSA output to include only internal buffers in the delay value: this matches windows and mac versions. BUG=45074 TEST=<audio> fires ended event when necessary Review URL: http://codereview.chromium.org/2978005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@52407 0039d316-1c4b-4281-b951-d872f2087c98
* media: refactoring video_render_base to recycle buffersjiesun@google.com2010-07-146-132/+205
| | | | | | | | | | | | | | To make recycle work, we had to define the usage scope of current frame. otherwise we are introducing tearing because we will begin to decode into the buffer before it is done by the renderer/painter/compositor. current mechanism depends on hold reference of a copied picture. we had no that luxury if we do not copy output buffers. we had to compromise by 1. in pause() ( which is not the sense of pipeline->pause(), which is implemented by set playrate = 0 ) in filter->pause() ( or in the future flush() ), which is part of seeking ( and in the future, part of stop() too) , we had to return all the buffers to owner. we had no current buffer to display. we use NULL as current frame in this case. 2. remove black frame from render base, this is only valid for system memory based video frame, even that should we use color fill instead of color conversion and scale. 3. pause and stop has to wait for pending read (actually flush) and pending paint. 4. we only advance frame when there are two or more frames in ready queue. Review URL: http://codereview.chromium.org/2836038 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@52398 0039d316-1c4b-4281-b951-d872f2087c98
* Revert everything related to the relanding.erg@chromium.org2010-07-145-5/+0
| | | | | | Review URL: http://codereview.chromium.org/2982009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@52367 0039d316-1c4b-4281-b951-d872f2087c98
* Reland r52336.erg@chromium.org2010-07-145-0/+5
| | | | | | | | | | | | | | (I accidentally committed a previous version of the patch (the fixed version was in codereview since I had switched to my mac to address this exact compile failure.) BUG=none TEST=none TBR=evan Review URL: http://codereview.chromium.org/2937010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@52349 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 52336 - More header cleanup:erg@chromium.org2010-07-145-5/+0
| | | | | | | | | | | | | | | | - Some cleanup to url_request.h - Remove task.h from headers that don't need it. - Remove histogram.h from message_loop.h BUG=none TEST=none Review URL: http://codereview.chromium.org/2986002 TBR=erg@chromium.org Review URL: http://codereview.chromium.org/2969006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@52343 0039d316-1c4b-4281-b951-d872f2087c98