summaryrefslogtreecommitdiffstats
path: root/media
Commit message (Collapse)AuthorAgeFilesLines
* Small refactor to media architecture in order to allow end-to-end tests.primiano@chromium.org2012-07-125-9/+199
| | | | | | | | | | | | | - Introduced the class MockAudioManager that will be used by upcoming speech recognition tests. - Added interface TestAudioInputController::Delegate, which allows to intercept Record/Close events. - Added audio_manager_for_tests_ static field in SpeechRecognizer to inject the MockAudioManager during tests. BUG=116954 TEST=none Review URL: https://chromiumcodereview.appspot.com/10704154 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@146316 0039d316-1c4b-4281-b951-d872f2087c98
* Fix SplitRange() logic in SourceBufferRange to transfer next buffer position ↵vrk@chromium.org2012-07-122-4/+59
| | | | | | | | | | | | | | | appropriately Adds logic to transfer the waiting-for-next-keyframe state to the split range if necessary. BUG=NONE TEST=media_unittests Review URL: https://chromiumcodereview.appspot.com/10703125 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@146274 0039d316-1c4b-4281-b951-d872f2087c98
* Add MultiChannelResampler wrapper for SincResampler.dalecurtis@chromium.org2012-07-124-0/+295
| | | | | | | | | | | | | | | | | Wraps the SincResampler for multichannel use. Can't land until after https://chromiumcodereview.appspot.com/10702050/ lands. Since the SincResampler unit tests focus on accuracy, the multi channel resampling tests just ensure the wrapper functions as expected with a single resampling frequency and a constant fill value. BUG=133637 TEST=MultiChannelResampler Unittests. Review URL: https://chromiumcodereview.appspot.com/10701049 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@146254 0039d316-1c4b-4281-b951-d872f2087c98
* Allocate one extra line to allow for H264 chroma MC overreads.rbultje@chromium.org2012-07-121-2/+9
| | | | | | Review URL: https://chromiumcodereview.appspot.com/10736040 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@146250 0039d316-1c4b-4281-b951-d872f2087c98
* Add SincResampler ported from WebKit.dalecurtis@chromium.org2012-07-114-0/+554
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is a partial port of WebAudio's SincResampler from WebKit formatted and culled for use by Chrome Media. We can't directly use the one in WebKit as it's layed under a ton of abstraction and is tightly coupled with WebKit objects. Test generates a swept sine wave and calculates the RMS error for common sample rates (via UMA stats). MultiChannelResampler and AudioRenderMixer changes to support resampling will come in later CLs. The 1000 ft view is that MultiChannelResampler will implement SincResampler:: AudioSourceProvider and AudioRendererMixer will implement a new MultiChannelResampler::MultiChannelAudioSourceProvider interface. When resampling is necessary AudioRenderMixer will feed itself into a MultiChannelResampler instance which will poll data as necessary and feed it channel by channel into a set of SincResamplers (one for each channel). We want to resample post-mixing since resampling is a much more expensive operation. Original for reference: http://git.chromium.org/gitweb/?p=external/Webkit.git&a=blob&f=Source/WebCore/platform/audio/SincResampler.cpp Visual plot of 44100 to 48000 for reference; red line is the resampled signal and the green line is the reference signal. Ideally only the pure signal (green) should be seen. Any bit of the resampled signal showing (red) is where the resampling algorithm is incorrect: http://i.imgur.com/1vsaI.png BUG=133637 TEST=New unittests. Review URL: https://chromiumcodereview.appspot.com/10702050 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@146219 0039d316-1c4b-4281-b951-d872f2087c98
* Reland r145965 - Make sure previous seek points in SourceBufferStream are ↵vrk@google.com2012-07-113-13/+193
| | | | | | | | | | | ignored during overlaps BUG=NONE TEST=asan bots are now happy Review URL: https://chromiumcodereview.appspot.com/10736021 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@146181 0039d316-1c4b-4281-b951-d872f2087c98
* Fix buffered range reporting for ChunkDemuxeracolwell@chromium.org2012-07-113-10/+25
| | | | | | | | | BUG=None TEST=PipelineIntegrationTest, BasicPlayback_MediaSource Review URL: https://chromiumcodereview.appspot.com/10701130 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@146127 0039d316-1c4b-4281-b951-d872f2087c98
* Remove #pragma once from mediaajwong@chromium.org2012-07-119-12/+3
| | | | | | Review URL: https://chromiumcodereview.appspot.com/10701135 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@146076 0039d316-1c4b-4281-b951-d872f2087c98
* Remove stale TODO + unnecessary if-guard in Pipeline::DoSeek().scherkus@chromium.org2012-07-111-10/+2
| | | | | | | | | | | | If we call DoSeek() without a demuxer we've clearly got issues as we shouldn't be seeking without a demuxer so crashing is preferable. BUG=none TEST=none TBR=acolwell Review URL: https://chromiumcodereview.appspot.com/10694125 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@146024 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 145965 - Make sure previous seek points in SourceBufferStream are ↵vrk@google.com2012-07-103-190/+13
| | | | | | | | | | | | | | ignored during overlaps BUG=136463 TEST=media_unittests,seek-sourceabort-crash.html Review URL: https://chromiumcodereview.appspot.com/10766021 TBR=vrk@google.com Review URL: https://chromiumcodereview.appspot.com/10690124 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@145968 0039d316-1c4b-4281-b951-d872f2087c98
* Make sure previous seek points in SourceBufferStream are ignored during overlapsvrk@google.com2012-07-103-13/+190
| | | | | | | | | BUG=136463 TEST=media_unittests,seek-sourceabort-crash.html Review URL: https://chromiumcodereview.appspot.com/10766021 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@145965 0039d316-1c4b-4281-b951-d872f2087c98
* Reintroduce kFramePadBytes in frame allocation.rbultje@chromium.org2012-07-101-2/+4
| | | | | | | | | | | | | Some FFmpeg SIMD functions overread/write slightly to have more efficient buffer access code. Not adding padding thus can lead to overreads/bytes (and crashes) when the buffer pointer points near the end of the frame boundaries. BUG=136231 Review URL: https://chromiumcodereview.appspot.com/10765015 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@145900 0039d316-1c4b-4281-b951-d872f2087c98
* Remove buffer duration calculation from WebMClusterParser and update ↵vrk@chromium.org2012-07-109-475/+614
| | | | | | | | | | | | | | | SourceBufferStream accordingly This removes the work to calculate buffers' durations in WebMClusterParser and modifies SourceBufferStream to no longer need duration information to work. This allows us to fix several bugs. BUG=131695,136418 TEST=media_unittests, go/oiedsb no longer needs Abort()s to work Review URL: https://chromiumcodereview.appspot.com/10692053 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@145839 0039d316-1c4b-4281-b951-d872f2087c98
* Remove default implementations for media::Filter methods.scherkus@chromium.org2012-07-0915-154/+158
| | | | | | | | | | | | | Another step towards eliminating the Filter class altogether! While attempting to refactor AudioRenderer out of Filter I noticed that Filter's default implementations caused Pipeline tests to not set appropriate expectations leading to unexpected gmock warnings. With this change we should be able to continue refactoring with no changes to unit tests. BUG=108341, 108342 TEST=none Review URL: https://chromiumcodereview.appspot.com/10749003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@145771 0039d316-1c4b-4281-b951-d872f2087c98
* Add HE AAC support to ISO BMFF.xiaomings@google.com2012-07-0916-33/+1037
| | | | | | | | | | | | | | | | Also abstract common code in H264BitReader into BitReader for reusing. Was: https://chromiumcodereview.appspot.com/10710002/ In the submitted patch of the last issue, the media.gyp included a non-existing file. This error hadn't been caught by try but later got caught by the build process and reverted the commit. So I create this issue to submit the fixed patch. BUG=134445 TEST=BitReaderTest, AACTest Review URL: https://chromiumcodereview.appspot.com/10753005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@145769 0039d316-1c4b-4281-b951-d872f2087c98
* Remove CompositeFilter::RemoveFilter() and Filter::clear_host().scherkus@chromium.org2012-07-095-40/+2
| | | | | | | | | | | | They are no longer used and will not be needed in a post-Filter world. BUG=126069 TEST=none Review URL: https://chromiumcodereview.appspot.com/10752003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@145715 0039d316-1c4b-4281-b951-d872f2087c98
* Switch to TimeDelta interfaces for TestTimeouts in media code.tedvessenes@gmail.com2012-07-092-12/+6
| | | | | | | | | | R=acolwell@chromium.org BUG=108171 Review URL: https://chromiumcodereview.appspot.com/10692117 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@145692 0039d316-1c4b-4281-b951-d872f2087c98
* Fix COM initialization on media thread.grt@chromium.org2012-07-091-0/+1
| | | | | | | | | | BUG=134624,136238 TEST=none TBR=tommi@chromium.org Review URL: https://chromiumcodereview.appspot.com/10744002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@145682 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 145617 - Add HE AAC support to ISO BMFF.cpu@chromium.org2012-07-0616-1038/+33
| | | | | | | | | | | | | | | | | | 1. Parse esds box to get HE AAC config. 2. Send audio config from AAC header to decoder instead of the one embedded in SampleDescription. 3. Convert raw audio data into ADTS before sending to decoder. 4. Abstract general bit stream code from H264BitReader into media::BitReader. BUG=134445 TEST=BitReaderTest, AACTest Review URL: https://chromiumcodereview.appspot.com/10710002 TBR=xiaomings@google.com Review URL: https://chromiumcodereview.appspot.com/10693115 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@145621 0039d316-1c4b-4281-b951-d872f2087c98
* Add HE AAC support to ISO BMFF.xiaomings@google.com2012-07-0616-33/+1038
| | | | | | | | | | | | | | | 1. Parse esds box to get HE AAC config. 2. Send audio config from AAC header to decoder instead of the one embedded in SampleDescription. 3. Convert raw audio data into ADTS before sending to decoder. 4. Abstract general bit stream code from H264BitReader into media::BitReader. BUG=134445 TEST=BitReaderTest, AACTest Review URL: https://chromiumcodereview.appspot.com/10710002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@145617 0039d316-1c4b-4281-b951-d872f2087c98
* Pepper needs to repeatedly call enumeration API in order to get a ↵xians@chromium.org2012-07-062-5/+15
| | | | | | | | | | | | | synchronous up-to-date device list. This heavily loads the IO thread since the enumeration is done on IO thread. This patch moves the device thread from VideoCaptureManager to MediaStreamManager, so that audio and video can share one device thread, and also allow running audio device API on this shared device thread. BUG=132701,130113 TEST=content_unittests Review URL: https://chromiumcodereview.appspot.com/10662049 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@145584 0039d316-1c4b-4281-b951-d872f2087c98
* Move the callback out of the Read method and into Initialize to make Read ↵tommi@chromium.org2012-07-044-45/+118
| | | | | | | | | | loops simpler. TEST=Run media_unittests --gtest_filter=*AsyncSocketIoHandlerTest* Review URL: https://chromiumcodereview.appspot.com/10697069 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@145453 0039d316-1c4b-4281-b951-d872f2087c98
* Coverity: Initialize a member variable.jhawkins@chromium.org2012-07-021-1/+2
| | | | | | | | | | | | | | CID_COUNT=1 CID=104390 BUG=none TEST=none R=gbillock TBR=vrk Review URL: https://chromiumcodereview.appspot.com/10702066 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@145207 0039d316-1c4b-4281-b951-d872f2087c98
* Add support to WebMClusterParser to accept clusters without blocksvrk@chromium.org2012-06-294-22/+42
| | | | | | | | | | BUG=NONE TEST=media_unittests Review URL: https://chromiumcodereview.appspot.com/10673012 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@145030 0039d316-1c4b-4281-b951-d872f2087c98
* Fix address casting in audio hashing.dalecurtis@chromium.org2012-06-291-2/+3
| | | | | | | | | | | | Per basictypes.h address-casting is bad! Instead use bit_cast<>. BUG=none TEST=BasicPlaybackHashed still passes without hash update. Review URL: https://chromiumcodereview.appspot.com/10702028 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@144856 0039d316-1c4b-4281-b951-d872f2087c98
* Lays the base work and initial unit tests for renderer side audio mixing! Rightdalecurtis@chromium.org2012-06-289-0/+896
| | | | | | | | | | | | | | | | | | | | | | | | | | | now nothing except unit tests are plumbed since mixing has poor performance with the large buffer sizes we're currently using; I.e., for a 140ms clip we attempt to render the full 170ms buffer, adding 30ms of overhead to the pipeline. Once another CL introduces resampling we can switch over to the low latency path for audio and turn on mixing at the same time. Chris also has some other ideas on how we might fix this prior to introducing resampling. Design revolves around each renderer creating an AudioRendererMixerInput which funnels into a common AudioRendererMixer (based on the AudioParameters each AudioRendererMixerInput is configured with). Each AudioRendererMixer owns a single AudioDevice which output from each AudioRendererMixerInput is mixed into. As mentioned above, for the initial landing of this CL the code which creates an AudioRendererMixerInput in RenderAudioSourceProvider has been disabled. Please look at https://chromiumcodereview.appspot.com/10636036/ if you're interested in what this looks like. BUG=133637 TEST=unit tests. audio_latency_perf. drum machine. manual testing. asan. Review URL: https://chromiumcodereview.appspot.com/10544130 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@144819 0039d316-1c4b-4281-b951-d872f2087c98
* The ISO BMFF Media Source parser supplied decode timestamps instead of ↵strobe@google.com2012-06-2811-37/+111
| | | | | | | | | | | | | presentation timestamps, resulting in incorrect playback. This patch makes the source buffer explicitly reference only decode timestamps as an intermediate step toward making it aware of buffer reordering. Additionally, FFmpegVideoDecoder will attempt to recover a framerate from the first sample duration it sees if the decoder config has a framerate of zero. BUG=134262 TEST=SourceBufferStreamTest, manual playback tests Review URL: https://chromiumcodereview.appspot.com/10669005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@144724 0039d316-1c4b-4281-b951-d872f2087c98
* Avoid undefined behaviour in OpenSLESOutputStream::FillBufferQueue()steveblock@chromium.org2012-06-281-1/+1
| | | | | | | | | The existing code produces undefined behaviour because it attempts to make multiple modifications to active_queue_ between two sequence points. Review URL: https://chromiumcodereview.appspot.com/10699022 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@144697 0039d316-1c4b-4281-b951-d872f2087c98
* Avoid undefined behaviour in OpenSLESInputStream::ReadBufferQueue()steveblock@chromium.org2012-06-281-1/+1
| | | | | | | | | | The existing code produces undefined behaviour because it attempts to make multiple modifications to active_queue_ between two sequence points. This generates warnings with -Wsequence-point in some versions of gcc. Review URL: https://chromiumcodereview.appspot.com/10689009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@144690 0039d316-1c4b-4281-b951-d872f2087c98
* Reland r143115: Provide a Chrome-owned buffer to FFmpeg for video decoding ↵scherkus@chromium.org2012-06-285-51/+121
| | | | | | | | | | | | (round 3). TEST=none BUG=none TBR=jam Review URL: https://chromiumcodereview.appspot.com/10703018 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@144640 0039d316-1c4b-4281-b951-d872f2087c98
* Temporary fix to allow ISO BMFF quality switching in Media Source.strobe@google.com2012-06-283-8/+34
| | | | | | | | | | BUG=122913 TEST=MP4StreamParserTest Review URL: https://chromiumcodereview.appspot.com/10656022 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@144618 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 144484 - Reland r143115: Provide a Chrome-owned buffer to FFmpeg for ↵dmichael@chromium.org2012-06-275-121/+51
| | | | | | | | | | | | | | video decoding. TEST=none BUG=none Review URL: https://chromiumcodereview.appspot.com/10678010 TBR=scherkus@chromium.org Review URL: https://chromiumcodereview.appspot.com/10704014 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@144539 0039d316-1c4b-4281-b951-d872f2087c98
* Recover pixel aspect ratio in ISO BMFF parser.strobe@google.com2012-06-273-3/+28
| | | | | | | | | | BUG=134801 TEST=Manual (non-public media file) Review URL: https://chromiumcodereview.appspot.com/10689003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@144537 0039d316-1c4b-4281-b951-d872f2087c98
* Reland r143115: Provide a Chrome-owned buffer to FFmpeg for video decoding.scherkus@chromium.org2012-06-275-51/+121
| | | | | | | | | TEST=none BUG=none Review URL: https://chromiumcodereview.appspot.com/10678010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@144484 0039d316-1c4b-4281-b951-d872f2087c98
* Flush sample buffers when reading new segments in Media Source BMFF parser.strobe@google.com2012-06-273-11/+35
| | | | | | | | | | BUG=134274 TEST=MP4StreamParserTest Review URL: https://chromiumcodereview.appspot.com/10660005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@144410 0039d316-1c4b-4281-b951-d872f2087c98
* Remove MockReader.acolwell@chromium.org2012-06-273-188/+53
| | | | | | | | | | BUG=None TEST=All FFmpegDemuxerTest test cases that used to rely on MockReader still pass. Review URL: https://chromiumcodereview.appspot.com/10687003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@144366 0039d316-1c4b-4281-b951-d872f2087c98
* Move java files into src parent directory.navabi@google.com2012-06-263-2/+2
| | | | | | | | | | BUG=134704 TEST= TBR=jam@chromium.org Review URL: https://chromiumcodereview.appspot.com/10668039 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@144272 0039d316-1c4b-4281-b951-d872f2087c98
* Make chrome compile with the win8 sdk cpu@chromium.org2012-06-261-0/+4
| | | | | | | | | | | | | So we can move the metro bits to \src FACILITY_VISUALCPP is now defined in the sdk BUG=127799 TEST= it builds Review URL: https://chromiumcodereview.appspot.com/10656036 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@144121 0039d316-1c4b-4281-b951-d872f2087c98
* RefCounted types should not have public destructors, delegate cleanuprsleevi@chromium.org2012-06-269-17/+32
| | | | | | | | | | | | | | | | | | | | | | | | For Delegate/Observer-type classes that specify an interface but do not have any particular lifetime requirements, make their destructors protected. This is to allow their interfaces to be implemented safely by RefCounted types. With public destructors, it's possible to do "scoped_ptr<Delegate> foo", and then assign a RefCountedDelegateImpl, which would lead to a double free. As none of these Delegates actually need public destructors (ownership of the Delegate* is not transferred during a function call / class constructor), mark the destructors protected so that it becomes a compile warning to try to delete them via the Delegate*. BUG=123295 TEST=it compiles Review URL: https://chromiumcodereview.appspot.com/10383262 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@144086 0039d316-1c4b-4281-b951-d872f2087c98
* Enable MP4 support when we have Chrome branding.acolwell@chromium.org2012-06-233-8/+8
| | | | | | | | | | BUG=133849 TEST=None Review URL: https://chromiumcodereview.appspot.com/10661015 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@143794 0039d316-1c4b-4281-b951-d872f2087c98
* Ensure media's buffered ranges always have a range that includes currentTime.fischman@chromium.org2012-06-2319-45/+144
| | | | | | | | | | | Avoids buffering bar disappearing/reappearing when the bytes are distributed unevenly throughout the media. BUG=133567,131444 Review URL: https://chromiumcodereview.appspot.com/10581050 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@143765 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 143725 - Fix MP4 support so it properly appears in Canary builds.sky@chromium.org2012-06-222-6/+6
| | | | | | | | | | | | | BUG=133849 TEST=None Review URL: https://chromiumcodereview.appspot.com/10636011 TBR=acolwell@chromium.org Review URL: https://chromiumcodereview.appspot.com/10656009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@143736 0039d316-1c4b-4281-b951-d872f2087c98
* Refactor SourceBufferStream to fix DCHECK when overlapping the selected rangevrk@google.com2012-06-223-175/+313
| | | | | | | | | | | | | The bug was made possible because the return value for GetNextTimestamp() was used incorrectly to check whether or not a range was seeked. This CL simplifies the purpose of GetNextTimestamp() and adds HasNextBufferPosition(). BUG=132930 TEST=media_unittests Review URL: https://chromiumcodereview.appspot.com/10573015 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@143734 0039d316-1c4b-4281-b951-d872f2087c98
* Fix MP4 support so it properly appears in Canary builds.acolwell@chromium.org2012-06-222-6/+6
| | | | | | | | | | BUG=133849 TEST=None Review URL: https://chromiumcodereview.appspot.com/10636011 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@143725 0039d316-1c4b-4281-b951-d872f2087c98
* Fix an uninteresting call in ↵xhwang@chromium.org2012-06-221-4/+4
| | | | | | | | | | | | | | FFmpegVideoDecoderTest.DecodeEncryptedFrame_WrongKey. This is a follow up CL of r142356. BUG=124741 TEST=This CL fixes an uninteresting call in the test. Review URL: https://chromiumcodereview.appspot.com/10640022 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@143680 0039d316-1c4b-4281-b951-d872f2087c98
* Add support for namespace in static JNI methods.bulach@chromium.org2012-06-221-7/+9
| | | | | | | | | | | | | | | | | | | This patch introduces two new annotations for the JNI: @JNINamespace: class-level annotation that indicates all natives methods are in the specified namespace. @NativeClassQualifiedName: method-level annotation that indicates the native method should be bound to the specific class (for instance, OuterContainer::InnerDelegate). This annotation removes the previously used feature of commenting the p0 param with the class name. BUG= TEST=jni_generator_tests Review URL: https://chromiumcodereview.appspot.com/10578035 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@143569 0039d316-1c4b-4281-b951-d872f2087c98
* Disable MP4StreamParser code for chromium builds.acolwell@chromium.org2012-06-213-27/+46
| | | | | | | | | BUG=133849 TEST=None Review URL: https://chromiumcodereview.appspot.com/10597005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@143477 0039d316-1c4b-4281-b951-d872f2087c98
* Fix crash caused by calling GetBufferedRanges() before init segment gets ↵acolwell@chromium.org2012-06-212-2/+15
| | | | | | | | | | | | appended. BUG=132933 TEST=ChunkDemuxerTest.TestGetBufferedRangesBeforeInitSegment Review URL: https://chromiumcodereview.appspot.com/10574047 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@143324 0039d316-1c4b-4281-b951-d872f2087c98
* Coverity: Fix a pass-by-value.jhawkins@chromium.org2012-06-201-1/+1
| | | | | | | | | | | | | | CID_COUNT=1 CID=104236 BUG=none TEST=none R=groby TBR=vrk Review URL: https://chromiumcodereview.appspot.com/10578050 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@143267 0039d316-1c4b-4281-b951-d872f2087c98
* Coverity: Fix uninitialized scalar variables.kmadhusu@chromium.org2012-06-204-21/+85
| | | | | | | | | | | CID=104262, 104261, 104260, 104259, 104258, 104257, 104256, 104255, 104254, 104253, 104252, 104251, 104250, 104249, 104248, 104247, 104246, 104245, 104244, 104243, 104242 BUG=none TEST=none TBR=acolwell@chromium.org Review URL: https://chromiumcodereview.appspot.com/10579031 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@143229 0039d316-1c4b-4281-b951-d872f2087c98