summaryrefslogtreecommitdiffstats
path: root/webkit/glue/media
Commit message (Collapse)AuthorAgeFilesLines
* Log BufferedResourceLoader events to MediaLog.scottfr@chromium.org2011-08-0910-21/+85
| | | | | | | | | | BUG= TEST= Review URL: http://codereview.chromium.org/7591001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@96014 0039d316-1c4b-4281-b951-d872f2087c98
* Fixes video to canvas drawing. The draw operationwas not updating the videojunov@chromium.org2011-08-041-0/+1
| | | | | | | | | | | | | past the first frame. Added a call to notifyPixelsChanged on the bitmap to invalidate the associated texture chache entry, so that the frame will update correctly. BUG=57695 TEST=http://www.craftymind.com/factory/html5video/CanvasVideo.html Review URL: http://codereview.chromium.org/7572027 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@95535 0039d316-1c4b-4281-b951-d872f2087c98
* First pass at removal of unneeded dependencies on skia\ext\platform_canvas.htwiz@google.com2011-08-021-1/+0
| | | | | | | | | | All 'low-hanging' platform_canvas.h dependencies have been removed, and replaced with skia-specific includes. BUG=None TEST=None Review URL: http://codereview.chromium.org/7517020 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@95083 0039d316-1c4b-4281-b951-d872f2087c98
* Change call to WebFrame::createAssociatedURLLoader to use overload that ↵bbudge@chromium.org2011-07-292-4/+18
| | | | | | | | takes WebURLLoaderOptions, in preparation for WebKit change that removes parameterless overload. Review URL: http://codereview.chromium.org/7519013 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@94675 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 93723 - Migrate DataSourceFactory to new callback system.acolwell@chromium.org2011-07-279-83/+128
| | | | | | | | | | | | BUG=90214 TEST=BufferedDataSourceTest.*, SimpleDataSourceTest.* Review URL: http://codereview.chromium.org/7461035 TBR=acolwell@chromium.org Review URL: http://codereview.chromium.org/7482029 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@94376 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 94316 - Fix invalid pointer dereference in WebDataSourceFactory when ↵acolwell@chromium.org2011-07-275-310/+35
| | | | | | | | | | | | | | | | playback is cancelled during init. Removed Clone() from DemuxerFactory since it isn't being used. BUG=90393 TEST=WebDataSourceFactoryTest.* Review URL: http://codereview.chromium.org/7465048 TBR=acolwell@chromium.org Review URL: http://codereview.chromium.org/7482028 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@94373 0039d316-1c4b-4281-b951-d872f2087c98
* Fix invalid pointer dereference in WebDataSourceFactory when playback is ↵acolwell@chromium.org2011-07-275-35/+310
| | | | | | | | | | | | | cancelled during init. Removed Clone() from DemuxerFactory since it isn't being used. BUG=90393 TEST=WebDataSourceFactoryTest.* Review URL: http://codereview.chromium.org/7465048 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@94316 0039d316-1c4b-4281-b951-d872f2087c98
* Migrate DataSourceFactory to new callback system.acolwell@chromium.org2011-07-229-128/+83
| | | | | | | | | BUG=90214 TEST=BufferedDataSourceTest.*, SimpleDataSourceTest.* Review URL: http://codereview.chromium.org/7461035 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@93723 0039d316-1c4b-4281-b951-d872f2087c98
* Remove MediaFormat once and for all.scherkus@chromium.org2011-07-204-24/+1
| | | | | | | | | BUG=28206 TEST=media_unittests, unit_tests, layout_tests Review URL: http://codereview.chromium.org/7452016 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@93282 0039d316-1c4b-4281-b951-d872f2087c98
* Replace VideoDecoder::media_format() with significantly simpler ↵scherkus@chromium.org2011-07-201-9/+6
| | | | | | | | | | | | | | | | | 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
* reland patch http://codereview.chromium.org/7204008 after webrtc build has ↵wjia@chromium.org2011-07-201-0/+33
| | | | | | | | | | been fixed. BUG=none TEST=try bots Review URL: http://codereview.chromium.org/7458006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@93235 0039d316-1c4b-4281-b951-d872f2087c98
* Fix media code to use initializer-list style for inheritance.scherkus@chromium.org2011-07-182-5/+6
| | | | | | Review URL: http://codereview.chromium.org/7400013 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@92826 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 92590 - add CaptureVideoDecoder for local preview and ↵sky@chromium.org2011-07-141-33/+0
| | | | | | | | | | | | | VideoCaptureModuleImpl for rtc encoding channel. BUG=none TEST=try bots Review URL: http://codereview.chromium.org/7204008 TBR=wjia@chromium.org Review URL: http://codereview.chromium.org/7375007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@92592 0039d316-1c4b-4281-b951-d872f2087c98
* add CaptureVideoDecoder for local preview and VideoCaptureModuleImpl for rtc ↵wjia@chromium.org2011-07-141-0/+33
| | | | | | | | | | encoding channel. BUG=none TEST=try bots Review URL: http://codereview.chromium.org/7204008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@92590 0039d316-1c4b-4281-b951-d872f2087c98
* Added extending forward_capacity for large reads.slock@chromium.org2011-07-133-19/+104
| | | | | | | | | | 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
* Adding ChunkDemuxer implementation.acolwell@chromium.org2011-06-291-4/+6
| | | | | | | | | 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
* Not allow compression when requesting multimediaenal@chromium.org2011-06-213-5/+33
| | | | | | | | | | | | | | | | 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-173-33/+5
| | | | | | | | | | | | | | | | | 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
* Not allow compression when requesting multimediaenal@chromium.org2011-06-173-5/+33
| | | | | | | | | | | | | | 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
* Adding retry limit for BufferedResourceLoader cache misses.acolwell@chromium.org2011-06-143-2/+77
| | | | | | | | | | | | This allows BufferedDataSource::Read() to fail when bad servers close the connection right after sending the headers. BUG=78218 TEST=BufferedDataSourceTest::BoundedCacheMisses Review URL: http://codereview.chromium.org/7056047 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@89010 0039d316-1c4b-4281-b951-d872f2087c98
* Fix BufferedResourceLoader's kReadThenDefer strategy.scherkus@chromium.org2011-06-033-7/+44
| | | | git-svn-id: svn://svn.chromium.org/chrome/trunk/src@87752 0039d316-1c4b-4281-b951-d872f2087c98
* Remove MockWebFrame in favour of MockWebFrameClient and update corresponding ↵scherkus@chromium.org2011-05-133-28/+42
| | | | | | | | | | | | | tests. This should help reduce WebKit gardening pain since all WebFrame implementations will reside in WebKit itself. BUG=none TEST=test_shell_tests Review URL: http://codereview.chromium.org/6973051 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@85343 0039d316-1c4b-4281-b951-d872f2087c98
* iwyu: Include stringprintf.h where appropriate, part 1.jhawkins@chromium.org2011-05-101-0/+1
| | | | | | | | | | | BUG=82098 TEST=none R=thakis@chromium.org Review URL: http://codereview.chromium.org/6997006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@84754 0039d316-1c4b-4281-b951-d872f2087c98
* iwyu: Use callback_old.h where appropriate, part 1.jhawkins@chromium.org2011-05-101-0/+1
| | | | | | | | | | | BUG=none TEST=none R=thakis@chromium.org Review URL: http://codereview.chromium.org/6985009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@84725 0039d316-1c4b-4281-b951-d872f2087c98
* Partial revert of 82061 so we keep the initial unbounded range request.scherkus@chromium.org2011-05-033-13/+60
| | | | | | | | | | | Also loosened restrictions for detecting servers that support range requests. Sending "Accept-Ranges: bytes" is optional so if we attempt a range request and a server replies correctly we should assume that it does indeed support range requests. BUG=80187 TEST=test_shell_tests --gtest_filter=Buffered* Review URL: http://codereview.chromium.org/6889019 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@83942 0039d316-1c4b-4281-b951-d872f2087c98
* Remove the watchdog timer and some member variables from BufferedDataSource.scherkus@chromium.org2011-04-223-149/+0
| | | | | | | | | | | | | | | | | | | | | | | The root cause is that the WatchDogTask will always think the read has timed out since read_submitted_time_ only gets set during ReadTask. What this means is that while we may start the WatchDogTask inside InitializeTask... read_submitted_time_ remains 0 which means the initial loader can get killed if it takes longer than 2500ms to respond and for the initial ReadTask to get executed. In other words, a pretty likely scenario on laggier connections. Sequence of events: 1) Initial loader created and watch dog timer started in BDS::InitializeTask 2) The initial loader receives a response and BDS and rest of pipeline continues to initialize 3) BDS::WatchDogTask gets posted to render thread 4) FFmpegDemuxer issues a read, setting read_callback_ and posting BDS::ReadTask to render thread 5) BDS::WatchDogTask is executed, but since read_callback_ isn't NULL and read_submitted_time_ is zero the loader is recreated 6) BDS::ReadTask is executed and calls BRL::Read ***on the new loader*** 7) The new loader receives a response, which calls BDS::PartialReadStartCallback and calls BRL::Read 8) DCHECK(!read_callback_.get()) is hit inside BRL::Read BUG=74473 TEST=layout tests under valgrind Review URL: http://codereview.chromium.org/6899004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@82683 0039d316-1c4b-4281-b951-d872f2087c98
* Fix BufferedResourceLoader so it only makes Range requests when a subset of ↵acolwell@chromium.org2011-04-195-50/+111
| | | | | | | | | | | the file is needed. BUG=74975 TEST=none Review URL: http://codereview.chromium.org/6815012 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@82061 0039d316-1c4b-4281-b951-d872f2087c98
* Renamed raw_data_length to encoded_data_lengthvsevik@chromium.org2011-04-134-4/+4
| | | | | | | | | BUG=None TEST=None Review URL: http://codereview.chromium.org/6838021 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@81464 0039d316-1c4b-4281-b951-d872f2087c98
* Enabled actual transfer size in chromiumvsevik@chromium.org2011-04-082-2/+2
| | | | | | | | | BUG=40502 TEST=Open DevTools, open site having gzip/chunked encoding, ensure transfer size is correct. Review URL: http://codereview.chromium.org/6771043 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@80965 0039d316-1c4b-4281-b951-d872f2087c98
* Eliminate skia::PlatformCanvas, a subclass of SkCanvas. Skia provides ↵alokp@chromium.org2011-04-083-12/+12
| | | | | | | | | | | | | | | | multiple types of SkCanvas classes that we would like to use. Unfortunately these classes are implemented as subclasses of SkCanvas. Subclassing SkCanvas in both Skia and Chromium makes it impossible to dynamically use any SkCanvas. There is also no reason for chromium to subclass SkCanvas. Most of the extra functionalities can be implemented by hanging meta-data from SkCanvas. We cannot eliminate skia::PlatformCanvas in one step due to WebKit's dependency on skia::PlatformCanvas. WebKit::WebCanvas is typedef as skia::PlatformDevice. It should be SkCanvas. So we need to do it in multiple steps: 1. Prepare Chromium tree for the change in WebKit::WebCanvas tyepdef. This basically means adding a couple of static_cast<skia::PlatformCanvas>(WebCanvas). 2. Change WebKit::WebCanvas typedef from skia::PlatformCanvas to SkCanvas 3. Eliminate skia::PlatformCanvas in chromium This CL accomplishes the first step on windows. WebKit BUG=https://bugs.webkit.org/show_bug.cgi?id=57563 Review URL: http://codereview.chromium.org/6783023 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@80955 0039d316-1c4b-4281-b951-d872f2087c98
* Fix clang build.pfeldman@chromium.org2011-04-073-3/+5
| | | | | | TBR=vsevik git-svn-id: svn://svn.chromium.org/chrome/trunk/src@80774 0039d316-1c4b-4281-b951-d872f2087c98
* WebKit roll 83160:83167.pfeldman@chromium.org2011-04-073-21/+5
| | | | | | TBR=tkent git-svn-id: svn://svn.chromium.org/chrome/trunk/src@80766 0039d316-1c4b-4281-b951-d872f2087c98
* Implementing preload=metadata for videovrk@google.com2011-04-057-146/+216
| | | | | | | | | | | | | 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 deadlock during player shutdown when a Seek() and an Abort() are racing.fischman@chromium.org2011-04-011-13/+7
| | | | | | | | | BUG=74031 TEST=trybots, repro site from bug report, and a layouttest which I probably won't be able to submit since it requires a large (14MB) video file to repro, and webkit's bugzilla assplodes on patches over 10MB. Review URL: http://codereview.chromium.org/6690065 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@80163 0039d316-1c4b-4281-b951-d872f2087c98
* Migrated chromium to new didReceiveData method in WebURLLoaderClientvsevik@chromium.org2011-03-313-3/+23
| | | | | | | | BUG=40502 Review URL: http://codereview.chromium.org/6737026 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@80039 0039d316-1c4b-4281-b951-d872f2087c98
* Move some files from base to base/memory.levin@chromium.org2011-03-283-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
* PipelineError is dead. Long live PipelineStatus!fischman@chromium.org2011-03-165-13/+15
| | | | | | | | | | | | | | | | | | | | 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
* Refactoring code to use factories to create DataSource objects.acolwell@chromium.org2011-03-1410-180/+519
| | | | | | | | | 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
* Remove the silly notion of MIME types from media code.scherkus@chromium.org2011-03-092-4/+0
| | | | | | | | | | | 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
* Cleaning up src/media to be consistent with static versus anonymous namespaces.scherkus@chromium.org2011-03-076-68/+43
| | | | | | | | | | | | | 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
* Revert code that prevents redirecting to different origins because it breaks ↵acolwell@chromium.org2011-03-014-18/+3
| | | | | | | | | | | Vimeo & YouTube. BUG=74451,72578 TEST=BufferedResourceLoaderTest.HasSingleOrigin(), SimpleDataSourceTest.HasSingleOrigin() Review URL: http://codereview.chromium.org/6596059 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@76334 0039d316-1c4b-4281-b951-d872f2087c98
* Make playback fail if redirected to a different origin.acolwell@chromium.org2011-02-254-35/+27
| | | | | | | | | | | | Fixing spec compliance. http://dev.w3.org/html5/spec/Overview.html#concept-media-load-resource BUG=72578 TEST=BufferedResourceLoaderTest.HasSingleOrigin(), SimpleDataSourceTest.HasSingleOrigin() Review URL: http://codereview.chromium.org/6580014 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@76064 0039d316-1c4b-4281-b951-d872f2087c98
* Fix HasSingleOrigin() so it properly handles double redirect & added unit test.acolwell@chromium.org2011-02-147-14/+157
| | | | | | | | | BUG=72625 TEST=BufferedResourceLoaderTest.HasSingleOrigin, SimpleDataSourceTest.HasSingleOrigin Review URL: http://codereview.chromium.org/6488008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@74829 0039d316-1c4b-4281-b951-d872f2087c98
* Change includes of gfx/* to ui/gfx/*sail@chromium.org2011-02-051-2/+2
| | | | | | | | | BUG=71063 TEST=compiled Review URL: http://codereview.chromium.org/6312156 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@73890 0039d316-1c4b-4281-b951-d872f2087c98
* Fix a teardown hang caused by an Abort() call while there is a pending read.acolwell@chromium.org2011-01-263-22/+118
| | | | | | | | | BUG=64754 TEST=BufferedDataSourceTest.StopDoesNotUseMessageLoopForCallback , BufferedDataSourceTest.AbortDuringPendingRead Review URL: http://codereview.chromium.org/6342018 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@72668 0039d316-1c4b-4281-b951-d872f2087c98
* Fix memory leaks in BufferedDataSource, BufferedResourceLoader and ↵scherkus@chromium.org2011-01-243-4/+0
| | | | | | | | | | | SimpleDataSource unit tests. BUG=65664 TEST=test_shell_tests + valgrind/heapcheck Review URL: http://codereview.chromium.org/6246017 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@72316 0039d316-1c4b-4281-b951-d872f2087c98
* Remove obsolete base/lock.h and fix up callers to use the new header file andbrettw@chromium.org2011-01-215-18/+17
| | | | | | | | | | | the base namespace. Fix several files including lock.h unnecessarily. BUG=none TEST=none Original review=http://codereview.chromium.org/6142009/ Patch by leviw@chromium.org git-svn-id: svn://svn.chromium.org/chrome/trunk/src@72106 0039d316-1c4b-4281-b951-d872f2087c98
* Add some more OWNERS files.darin@chromium.org2011-01-201-0/+1
| | | | | | | | TBR=ben Review URL: http://codereview.chromium.org/6306005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@71964 0039d316-1c4b-4281-b951-d872f2087c98
* Roll WebKit DEPS past WebKit move. Update gyp files and include paths to ↵abarth@chromium.org2011-01-179-29/+29
| | | | | | reflect the move. Consolidate how we DEPS in WebKit source files. Cross fingers. git-svn-id: svn://svn.chromium.org/chrome/trunk/src@71586 0039d316-1c4b-4281-b951-d872f2087c98
* Replace MockFilterCallback with MockCallback and simplify unit tests.scherkus@chromium.org2011-01-152-38/+23
| | | | | | | | | | | We had a ton of unnecessarily duplicated code, most of which could be solved by introducing NewExpectedCallback(). BUG=none TEST=media_unittests Review URL: http://codereview.chromium.org/6350001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@71537 0039d316-1c4b-4281-b951-d872f2087c98