summaryrefslogtreecommitdiffstats
path: root/media
Commit message (Collapse)AuthorAgeFilesLines
* Allow audio output on Linux to work with web audio APIcrogers@google.com2011-04-161-2/+3
| | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/6871030 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@81843 0039d316-1c4b-4281-b951-d872f2087c98
* Fix erratic HTML5 audio playbackdavej@chromium.org2011-04-144-21/+77
| | | | | | | | | | | | | | | This was affecting all platforms. The major issue was that he SetPlaybackRate(1) which is used to start playback was being done in the middle of the series of Seek operations. So doing Seek() followed by SetPlaybackRate(1) was done, the Seek could immediately stop the playback. To fix, Seek() was made atomic with regards to SetPlaybackRate() by delaying the execution of SetPlaybackRate until after the Seek sequence has completely finished. To make things run even smoother, a short delay was added before recyclilng a used audio stream in the Dispatcher. This gives the stream time to 'power down' before being reused. Tested on Linux, Chrome-OS (Cr48) and Mac with great results, making HTML5 audio much more usable for games. BUG=73045,59369,59370,65618 TEST=Manual, Quickly playing/stopping HTML5 audio should play cleanly. Review URL: http://codereview.chromium.org/6822019 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@81670 0039d316-1c4b-4281-b951-d872f2087c98
* Fix Linux audio playbackdavej@chromium.org2011-04-142-2/+26
| | | | | | | | | | | | | Playing audio files shorter than about 11000 samples on Linux was either not working at all, or re-playing (starting and stopping on the same audio object) was working only intermittently. It was possible to stop audio playback before the first buffer was played, resulting in no sound at all. Also, if replaying, the buffer may have been dirty from the previous playback on the same device. So on Linux at least, these issues appear fixed. BUG=73045,47761,59367,59369,59370,65618 TEST=Manual, quickly playing short audio files on Linux should work Review URL: http://codereview.chromium.org/6776024 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@81669 0039d316-1c4b-4281-b951-d872f2087c98
* Draft version of the HW video decode tester and few other changes.scherkus@chromium.org2011-04-145-4/+386
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Intention is that this tester can be used to decouple HW decode accelerator integration from running the whole Chrome browser. Features: - Independent GUnit executable, which should be possible to use in autotests. - Mimics Renderer process from Gpu video pipeline perspective. * Test bench contains implementation of FakeClient which essentially mimics Renderer process from the GpuVideoDecodeAccelerator's point of view. * FakeClient runs on it's own thread and will communicate with using the IPC messages that are used also within the real use case. * FakeClient will allocate memories using same SharedMemory stuff as the real Renderer code. * Currently reads H.264 Annex B bitstream from file and parses it to NAL units before feeding to the decoder * TODO: Polish and improving the features and configurability. * TODO: GLES texture allocation for textures. - Allows building various test cases and error behaviour as well both on AcceleratedVideoDecoder interface as well as erroneous behaviour from the client. - Allows also checking expected order of calls if we want to enforce certain behaviour across various implementations. Patch by vmr@chromium.org: http://codereview.chromium.org/6720040/ BUG=none TEST=none git-svn-id: svn://svn.chromium.org/chrome/trunk/src@81663 0039d316-1c4b-4281-b951-d872f2087c98
* Implements AudioInputDevice API for low-latency audio input in the browser ↵scherkus@chromium.org2011-04-124-13/+85
| | | | | | | | | | | | | | process. Also refactors some shared audio code into audio_common.{cc,h}. Patch by henrika@chromim.org: http://codereview.chromium.org/6250196/ BUG=none TEST=none git-svn-id: svn://svn.chromium.org/chrome/trunk/src@81315 0039d316-1c4b-4281-b951-d872f2087c98
* Include base/win/scoped_comptr.h instead of base/scoped_comptr_win.h.tfarina@chromium.org2011-04-124-35/+36
| | | | | | | | | | | | | | | | | Fix up all the callers to use the new location and namespace. Also, delete the stub file since it isn't included by anyone more. (Note: This was a TODO for brettw). BUG=None TEST=None R=brettw@chromium.org Review URL: http://codereview.chromium.org/6825055 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@81303 0039d316-1c4b-4281-b951-d872f2087c98
* Add IPC messages and renderer process side API for video capture. wjia@google.com2011-04-123-1/+121
| | | | | | | | BUG=none TEST=try bots Review URL: http://codereview.chromium.org/6815011 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@81199 0039d316-1c4b-4281-b951-d872f2087c98
* Remove svn:mergeinfo props from many files.pkasting@chromium.org2011-04-082-2/+2
| | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/6814027 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@80878 0039d316-1c4b-4281-b951-d872f2087c98
* Make the windows_version.h functions threadsafe by using a singleton. Add ↵pkasting@chromium.org2011-04-072-2/+4
| | | | | | | | | | | | accessors to the singleton for more values that various code wants, then convert almost everyone using OSVERSIONINFO or SYSTEM_INFO structs to calling these accessors. Declare an AtExitManager in the out-of-process test runner since it didn't have one and that breaks singleton-using code in the test executable (as opposed to in chrome.dll). A few other minor cleanups along the way (binding of "*", shorter code, etc.). Because I ran into problems with it while modifying gcapi.cc, I cleaned up our usage of strsafe.h a bit, so that files that don't need it don't include it and files that do use STRSAFE_NO_DEPRECATE instead of a modified #include order. BUG=none TEST=none Review URL: http://codereview.chromium.org/6816027 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@80851 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 80819 due to failed testsmmenke@chromium.org2011-04-072-4/+2
| | | | | | | TBR=pkasting@chromium.org Review URL: http://codereview.chromium.org/6816024 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@80824 0039d316-1c4b-4281-b951-d872f2087c98
* Make the windows_version.h functions threadsafe by using a singleton. Add ↵pkasting@chromium.org2011-04-072-2/+4
| | | | | | | | | | | | accessors to the singleton for more values that various code wants, then convert almost everyone using OSVERSIONINFO or SYSTEM_INFO structs to calling these accessors. Declare an AtExitManager in the out-of-process test runner since it didn't have one and that breaks singleton-using code in the test executable (as opposed to in chrome.dll). A few other minor cleanups along the way (binding of "*", shorter code, etc.). Because I ran into problems with it while modifying gcapi.cc, I cleaned up our usage of strsafe.h a bit, so that files that don't need it don't include it and files that do use STRSAFE_NO_DEPRECATE instead of a modified #include order. BUG=none TEST=none Review URL: http://codereview.chromium.org/6713107 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@80819 0039d316-1c4b-4281-b951-d872f2087c98
* Fix valgrind messages for uninitialized preload_ value.vrk@google.com2011-04-052-1/+2
| | | | | | | | | | | | | The preload_ field was uninitialized when PipelineImpl was constructed, so PipelineImpl unit tests failed Valgrind tests. CL 80465 introduced the problem. BUG=NONE TEST=NONE TBR=acolwell Review URL: http://codereview.chromium.org/6720052 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@80503 0039d316-1c4b-4281-b951-d872f2087c98
* Implementing preload=metadata for videovrk@google.com2011-04-0511-5/+85
| | | | | | | | | | | | | This patch implements the logic necessary to respect the preload attribute when it is set to MetaData. This also refactors the BufferedResourceLoader to determine its buffering techniques based on a DeferStrategy value. BUG=16482,76555 TEST=media/video-preload.html, test_shell_tests Review URL: http://codereview.chromium.org/6625059 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@80465 0039d316-1c4b-4281-b951-d872f2087c98
* Fix an uninitialized variable compiler warning on some systems.mdm@chromium.org2011-04-031-1/+1
| | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/6677123 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@80279 0039d316-1c4b-4281-b951-d872f2087c98
* Fix samplefmt.h inclusion with newer ffmpegscherkus@chromium.org2011-04-011-1/+0
| | | | | | | | | | | | libavcore is defunct in upstream ffmpeg. Patch by floppymaster@gmail.com: http://codereview.chromium.org/6791004/ BUG=77795 TEST=Build with recent ffmpeg (use_system_ffmpeg=1) git-svn-id: svn://svn.chromium.org/chrome/trunk/src@80200 0039d316-1c4b-4281-b951-d872f2087c98
* Mac platform was not treating 8bit PCM audio as unsigned data.sjl@google.com2011-04-011-4/+15
| | | | | | | | | | | If bits per channel is 8, open the stream as unsigned and use 128 as the silence value. BUG=70730 TEST=media_unittests and manual testing of a bunch of audio files (8 bit and others). Review URL: http://codereview.chromium.org/6689003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@80181 0039d316-1c4b-4281-b951-d872f2087c98
* Fix some style issues in media/video/ffmpeg_video_allocator.{cc,h}.scherkus@chromium.org2011-04-012-14/+16
| | | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/6771037 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@80174 0039d316-1c4b-4281-b951-d872f2087c98
* Create helper methods to process FFmpeg index data.acolwell@chromium.org2011-04-0110-15/+313
| | | | | | | | | BUG=none TEST=FFmpegCommonTest Review URL: http://codereview.chromium.org/6708130 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@80153 0039d316-1c4b-4281-b951-d872f2087c98
* Initialize stderr logging for player_x11 so we can use --v=N.scherkus@chromium.org2011-03-301-0/+7
| | | | | | | | | BUG=none TEST=build player_x11, start it with --v=1 Review URL: http://codereview.chromium.org/6776002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@79889 0039d316-1c4b-4281-b951-d872f2087c98
* Flag-enabled the experimental x-adaptive scheme with --enable-adaptive.fischman@chromium.org2011-03-302-2/+6
| | | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/6749045 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@79886 0039d316-1c4b-4281-b951-d872f2087c98
* Checking in media::BitstreamBuffer and media::VideoDecodeAccelerator interfaces.scherkus@chromium.org2011-03-303-0/+229
| | | | | | | | | | | | | | Also includes stubbed out implementation VideoDecodeAcceleratorHost. Part of a patch by vmr@chromium.org: http://codereview.chromium.org/6541068/ BUG=none TEST=none Review URL: http://codereview.chromium.org/6675040 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@79878 0039d316-1c4b-4281-b951-d872f2087c98
* Explicitly initialize media_format to avoid incorrect uninitialized-variable ↵fischman@chromium.org2011-03-291-1/+1
| | | | | | | | compiler warning. TBR=acolwell git-svn-id: svn://svn.chromium.org/chrome/trunk/src@79752 0039d316-1c4b-4281-b951-d872f2087c98
* AdaptiveDemuxer{,Factory,Stream} are born!fischman@chromium.org2011-03-2910-7/+628
| | | | | | | | | | | | | | | | | | This family of classes enable presenting media combined from multiple URLs (e.g. pulling audio from one URL and video from another). In the future it will also enable switching sources during playback, but this isn't implemented yet. player_{x11,wtl} have been updated to use these new classes, but webmediaplayer_impl hasn't yet, because I don't want to expose my hacky alternative scheme to the world. BUG=none TEST=trybots Review URL: http://codereview.chromium.org/6769001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@79750 0039d316-1c4b-4281-b951-d872f2087c98
* Refactor: Move app/gfx/gl ==> ui/gfx/glbacker@chromium.org2011-03-289-19/+19
| | | | | | | | | | | This is the final patch in the sequence. Note that gl.gyp is introduced because dependency checking on the Mac is done on a per file (rather than per target) basis. BUG=none TEST=trybots Review URL: http://codereview.chromium.org/6722026 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@79599 0039d316-1c4b-4281-b951-d872f2087c98
* Initialize uninitialized members in a couple media classes.finnur@chromium.org2011-03-283-8/+16
| | | | | | | | | BUG=None TEST=None CID=11311, 11977, 11976. Review URL: http://codereview.chromium.org/6730055 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@79546 0039d316-1c4b-4281-b951-d872f2087c98
* Collapse two case statements into one, they do the same thing.finnur@chromium.org2011-03-281-3/+2
| | | | | | | | | BUG=None TEST=None CID=14359 Review URL: http://codereview.chromium.org/6676146 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@79542 0039d316-1c4b-4281-b951-d872f2087c98
* Move some files from base to base/memory.levin@chromium.org2011-03-2864-111/+111
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | raw_scoped_refptr_mismatch_checker.h ref_counted.cc ref_counted.h ref_counted_memory.cc ref_counted_memory.h ref_counted_unittest.cc scoped_callback_factory.h scoped_comptr_win.h scoped_handle.h scoped_native_library.cc scoped_native_library.h scoped_native_library_unittest.cc scoped_nsobject.h scoped_open_process.h scoped_ptr.h scoped_ptr_unittest.cc scoped_temp_dir.cc scoped_temp_dir.h scoped_temp_dir_unittest.cc scoped_vector.h singleton.h singleton_objc.h singleton_unittest.cc linked_ptr.h linked_ptr_unittest.cc weak_ptr.cc weak_ptr.h weak_ptr_unittest.cc BUG=None TEST=Compile Review URL: http://codereview.chromium.org/6714032 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@79524 0039d316-1c4b-4281-b951-d872f2087c98
* Die, DemuxerStream::QueryInterface, die!fischman@chromium.org2011-03-2413-124/+19
| | | | | | | | | | | | | | This appears to have been an attempt at hand-rolling an RTTI system, all for the use of a *single* class (!). Removing it makes for cleaner, simpler, shorter code. It also enables delegating DemuxerStreams, which wasn't possible before (b/c the overridable method was protected, so not visible to a HAS-A wrapper). BUG=none TEST=trybots Review URL: http://codereview.chromium.org/6730006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@79309 0039d316-1c4b-4281-b951-d872f2087c98
* Build a raw video pipeline for the <video> with a special src attribute ↵acolwell@chromium.org2011-03-238-16/+617
| | | | | | | | | | | | | | | | | | | | | (media://...). The raw video pipeline graph only has two filters - one pass-thru decoder filter and one renderer filter. Contributed by ronghuawu@google.com Original code reviews: http://codereview.chromium.org/6658001/ (pipeline changes) http://codereview.chromium.org/6621049/ (pass-thru filter) BUG=none TEST=media_unittests Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=79149 Review URL: http://codereview.chromium.org/6726006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@79185 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 79149 - Build a raw video pipeline for the <video> with a special src ↵acolwell@chromium.org2011-03-238-615/+16
| | | | | | | | | | | | | | | | | | | | | | attribute (media://...). The raw video pipeline graph only has two filters - one pass-thru decoder filter and one renderer filter. Contributed by ronghuawu@google.com Original code reviews: http://codereview.chromium.org/6658001/ (pipeline changes) http://codereview.chromium.org/6621049/ (pass-thru filter) BUG=none TEST=media_unittests Review URL: http://codereview.chromium.org/6726006 TBR=acolwell@chromium.org Review URL: http://codereview.chromium.org/6722031 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@79158 0039d316-1c4b-4281-b951-d872f2087c98
* Build a raw video pipeline for the <video> with a special src attribute ↵acolwell@chromium.org2011-03-238-16/+615
| | | | | | | | | | | | | | | | | | | (media://...). The raw video pipeline graph only has two filters - one pass-thru decoder filter and one renderer filter. Contributed by ronghuawu@google.com Original code reviews: http://codereview.chromium.org/6658001/ (pipeline changes) http://codereview.chromium.org/6621049/ (pass-thru filter) BUG=none TEST=media_unittests Review URL: http://codereview.chromium.org/6726006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@79149 0039d316-1c4b-4281-b951-d872f2087c98
* Add WebKit API to retrieve the audio hardware sample-ratecrogers@google.com2011-03-232-0/+18
| | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/6721015 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@79088 0039d316-1c4b-4281-b951-d872f2087c98
* Use default "output" rather than "input" device when querying hardware ↵crogers@google.com2011-03-221-3/+3
| | | | | | | | | | | sample-rate BUG=none TEST=none (tested locally on Mac OS X to verify that hardware sample-rate is correctly retrieved) Review URL: http://codereview.chromium.org/6721001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@79055 0039d316-1c4b-4281-b951-d872f2087c98
* Simplify the Demuxer interface to get streams by type.fischman@chromium.org2011-03-218-77/+62
| | | | | | | | | | | | | | Before this change Demuxer's interface looked as if it could return multiple streams for a given type, but the implementations all assume/need at most a single stream per type. This change should make the (soon to be written) AdaptiveDemuxer{,Stream} less confusing. BUG=none TEST=trybots Review URL: http://codereview.chromium.org/6685122 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@78908 0039d316-1c4b-4281-b951-d872f2087c98
* Move media library path resolution into Chrome path provider.wez@chromium.org2011-03-173-9/+38
| | | | | | | | | BUG=None TEST=None Review URL: http://codereview.chromium.org/6537022 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@78532 0039d316-1c4b-4281-b951-d872f2087c98
* Stop leaking callbacks in pipeline_impl_unittest.ccfischman@chromium.org2011-03-161-1/+1
| | | | | | | | BUG=none TEST=media_unittests under valgrind stops complaining about leaked memory in PipelineStatusNotification's tests. TBR=acolwell git-svn-id: svn://svn.chromium.org/chrome/trunk/src@78389 0039d316-1c4b-4281-b951-d872f2087c98
* PipelineError is dead. Long live PipelineStatus!fischman@chromium.org2011-03-1624-236/+333
| | | | | | | | | | | | | | | | | | | | PipelineError was a poor naming choice because most of the time variables of that type held the value PIPELINE_OK meaning there was in fact no error. Replaced the idiom of [0-ary callback + GetError()] with [1-ary callback taking PipelineStatus argument] which makes the Pipeline API cleaner and less error-prone. Before, consumers of the API had to make sure to call GetError() at the top of each callback, or risk missing state transitions in the pipeline. Now each callback gets an explicit parameter holding the pipeline status at the moment the callback was invoked so failing to handle error conditions should be more apparent in the code. BUG=none TEST=media_unittests + trybots: {mac,linux,win}{_layout,}, linux_rel, linux_clang (all pass or fail with unrelated errors) Review URL: http://codereview.chromium.org/6686061 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@78379 0039d316-1c4b-4281-b951-d872f2087c98
* DemuxerFactory is born!fischman@chromium.org2011-03-1418-205/+345
| | | | | | | | | | | | | | | PipelineImpl no longer knows about DataSources, with DemuxerFactory taking a DataSourceFactory when appropriate (i.e. for FFmpegDemuxerFactory). This diff is relative to patchset 8 of issue 6480050, which must be submitted first. BUG=none TEST=media_unittests pass; chrome plays videos; trybots passed: mac,linux,linux_clang,win_layout; trybots with only unrelated-looking failures: win,{mac,linux}_layout Review URL: http://codereview.chromium.org/6648004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@78075 0039d316-1c4b-4281-b951-d872f2087c98
* Refactoring code to use factories to create DataSource objects.acolwell@chromium.org2011-03-1428-177/+843
| | | | | | | | | BUG=72485 TEST=None for now. Existing unit tests cover this code. Review URL: http://codereview.chromium.org/6480050 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@78033 0039d316-1c4b-4281-b951-d872f2087c98
* Fold media/ffmpeg/ffmpeg_util.{cc,h} into media/ffmpeg/ffmpeg_common.{cc,h}scherkus@chromium.org2011-03-1211-49/+13
| | | | | | | | | BUG=none TEST=compiles Review URL: http://codereview.chromium.org/6686011 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@77896 0039d316-1c4b-4281-b951-d872f2087c98
* Remove FFmpegVideoDecodeEngine's dependency on AVStream (again).scherkus@chromium.org2011-03-1116-171/+271
| | | | | | | | | | | First step of many towards removing DemuxerStream::QueryInterface, AVStreamProvider, and MediaFormat. This time without memory leaks! BUG=28206 TEST=media_unittests git-svn-id: svn://svn.chromium.org/chrome/trunk/src@77875 0039d316-1c4b-4281-b951-d872f2087c98
* Remove a test which checks for timing of audio recording callbacks.satish@chromium.org2011-03-112-51/+0
| | | | | | | | | | | | This frequently fails in various builders because the machines may be overloaded when the tests run, so I see no point in such a timing test. BUG=49497 TEST=none Review URL: http://codereview.chromium.org/6673022 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@77819 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 77476 - Remove FFmpegVideoDecodeEngine's dependency on AVStream.tyoshino@chromium.org2011-03-1116-262/+171
| | | | | | | | | | | | | | | | | This change seems to have caused Linux Heapcheck bot redness. http://build.chromium.org/p/chromium.memory/builders/Linux%20Heapcheck/builds/4846 First step of many towards removing DemuxerStream::QueryInterface, AVStreamProvider, and MediaFormat. BUG=28206 TEST=media_unittests Review URL: http://codereview.chromium.org/6624062 TBR=scherkus@chromium.org Review URL: http://codereview.chromium.org/6665020 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@77770 0039d316-1c4b-4281-b951-d872f2087c98
* Remove FFmpegVideoDecodeEngine's dependency on AVStream.scherkus@chromium.org2011-03-0916-171/+262
| | | | | | | | | | | First step of many towards removing DemuxerStream::QueryInterface, AVStreamProvider, and MediaFormat. BUG=28206 TEST=media_unittests Review URL: http://codereview.chromium.org/6624062 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@77476 0039d316-1c4b-4281-b951-d872f2087c98
* Remove the silly notion of MIME types from media code.scherkus@chromium.org2011-03-0928-279/+97
| | | | | | | | | | | This was one of the first things I checked into Chromium and I've been wanting to remove it for a long, long time now. BUG=28206 TEST=media_unittests Review URL: http://codereview.chromium.org/6623087 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@77468 0039d316-1c4b-4281-b951-d872f2087c98
* Remove width_ and height_ from various VideoDecoder implementations.scherkus@chromium.org2011-03-086-73/+62
| | | | | | | | | | | Updated unit tests as well. BUG=none TEST=media_unittests Review URL: http://codereview.chromium.org/6624068 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@77336 0039d316-1c4b-4281-b951-d872f2087c98
* Cleaning up src/media to be consistent with static versus anonymous namespaces.scherkus@chromium.org2011-03-0737-377/+222
| | | | | | | | | | | | | We were using a mix of both (and sometimes even static functions *inside* anonymous namespaces!) so we decided to stick to using static. Also moved static/test code into media namespace and removed media:: prefixes. BUG=none TEST=compiles Review URL: http://codereview.chromium.org/6628020 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@77135 0039d316-1c4b-4281-b951-d872f2087c98
* Change other usages of .size() to .empty() when applicable.erg@google.com2011-03-041-4/+4
| | | | | | | | | BUG=carnitas TEST=compiles Review URL: http://codereview.chromium.org/6609008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@76962 0039d316-1c4b-4281-b951-d872f2087c98
* Cleaning up media/base/clock.cc,hscherkus@chromium.org2011-03-042-10/+10
| | | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/6624022 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@76946 0039d316-1c4b-4281-b951-d872f2087c98
* Fold ClockImpl into Clock and update corresponding clients and tests.scherkus@chromium.org2011-03-048-120/+90
| | | | | | | | | BUG=none TEST=media_unittests --gtest_filter=ClockTest.* Review URL: http://codereview.chromium.org/6609035 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@76863 0039d316-1c4b-4281-b951-d872f2087c98
generated by cgit v1.1 at 2025-02-01 01:14:26 +0000