summaryrefslogtreecommitdiffstats
path: root/media
Commit message (Collapse)AuthorAgeFilesLines
* use lock to make sure device will not deliver any frame after it's stopped.wjia@chromium.org2012-09-062-2/+12
| | | | | | | BUG=139004 Review URL: https://chromiumcodereview.appspot.com/10905134 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@155250 0039d316-1c4b-4281-b951-d872f2087c98
* Combine audio decoding and video decoding onto a single thread.scherkus@chromium.org2012-09-065-14/+10
| | | | | | | | | | | Our use of multithreaded video decoding results in a video decoding thread that is idle most of the time. Instead of being idle it can be doing real work like decoding audio. The end result is one less thread created per media element. BUG=61293 TBR=brettw Review URL: https://chromiumcodereview.appspot.com/10915091 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@155179 0039d316-1c4b-4281-b951-d872f2087c98
* Ports the WebCore::AudioFIFO in WebKit to Chrome style.henrika@chromium.org2012-09-064-0/+415
| | | | | | | | | | | This class will be use in an upcoming AudioPullFifo class. BUG=none TEST=media_unittests --gtest_filter=AudioFifoTest.* Review URL: https://chromiumcodereview.appspot.com/10912079 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@155171 0039d316-1c4b-4281-b951-d872f2087c98
* Make initial reset period configurable for audio input no data timer.milanb@chromium.org2012-09-062-13/+22
| | | | | | | | | | | | iOS needs extra initial delay in the reset timer to avoid getting killed because there is a background music fading out. BUG=b/6754065 Review URL: https://chromiumcodereview.appspot.com/10911067 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@155082 0039d316-1c4b-4281-b951-d872f2087c98
* Exclude audio output dispatcher for iOS.milanb@chromium.org2012-09-051-0/+11
| | | | | | | | | | | | Upstreaming iOS. iOS uses only audio input and does not link against the audio output classes. BUG=b/6754065 Review URL: https://chromiumcodereview.appspot.com/10914082 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@155011 0039d316-1c4b-4281-b951-d872f2087c98
* Build supported media targets on iOS.blundell@chromium.org2012-09-052-19/+81
| | | | | | | | | | | | Makes the changes required to get media and media_unittests to build successfully for iOS, building only the limited set of files that iOS requires. BUG=None Review URL: https://chromiumcodereview.appspot.com/10919092 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@154996 0039d316-1c4b-4281-b951-d872f2087c98
* Fix unitialized reads in AudioOutputController unittest.dalecurtis@google.com2012-09-052-9/+13
| | | | | | | | | | | | | | | | | http://crrev.com/154951 introduced a bug where these buffers were used, but the mocked methods were not initializing the data. Also removes the suppression and fixes scherkus's comments from: http://codereview.chromium.org/10920092/ BUG=114700 TEST=media_unittests + valgrind. TBR=timurrr,scherkus Review URL: https://chromiumcodereview.appspot.com/10916120 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@154968 0039d316-1c4b-4281-b951-d872f2087c98
* Fix unitialized data access in AlsaOutputUnittest.dalecurtis@google.com2012-09-051-4/+10
| | | | | | | | | | | | | http://crrev.com/154951 introduced a bug where these buffers were used, but the mocked methods were not initializing the data. BUG=none TEST=media_unittests + valgrind. TBR=timurrrr, scherkus Review URL: https://chromiumcodereview.appspot.com/10920092 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@154954 0039d316-1c4b-4281-b951-d872f2087c98
* Switch OnMoreData() to use AudioBus.dalecurtis@chromium.org2012-09-0539-553/+444
| | | | | | | | | | | | | | | | | | | As titled, with this change we're now piping float data around the pipeline from end to end. This change is in preparation for browser side channel remixing and resampling. As a consequence of this change the shared memory now represents the contents of an AudioBus object, which is essentially audio data in a float planar format. BUG=114700 TEST=Should be no audible change. Ran all existing tests. Compiled ran WebAudio/HTML5/WebRTC on all platforms and PPAPI on Linux. Review URL: https://chromiumcodereview.appspot.com/10832285 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@154951 0039d316-1c4b-4281-b951-d872f2087c98
* Add MainHook to media unittests.milanb@chromium.org2012-09-051-0/+2
| | | | | | | | | | | | | | MainHooks are in particular used on iOS to prevent the system from killing the test application at startup. See https://chromiumcodereview.appspot.com/10690161/ for more context. BUG=b/6754065 Review URL: https://chromiumcodereview.appspot.com/10915061 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@154946 0039d316-1c4b-4281-b951-d872f2087c98
* Move various media.gyp targets into an OS!=iOS blockstuartmorgan@chromium.org2012-09-041-216/+223
| | | | | | | | | | | | | | In preparation for iOS support in media.gyp, move all the targets that won't be used on iOS into a !iOS block. Future CLs will get the remaning targets (media, media_unittests, media_test_support) building for iOS. BUG=None Review URL: https://chromiumcodereview.appspot.com/10887008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@154790 0039d316-1c4b-4281-b951-d872f2087c98
* Add missing break to WinAudioTest.dalecurtis@google.com2012-09-041-0/+1
| | | | | | | | | | | Fixes unittest failures on Windows. BUG=none TEST=none Review URL: https://chromiumcodereview.appspot.com/10918044 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@154734 0039d316-1c4b-4281-b951-d872f2087c98
* Update FFmpegAudioDecoder to support multi-frame packets.dalecurtis@chromium.org2012-08-305-116/+188
| | | | | | | | | | | | | | | | | | Per the FFmpeg docs, avcodec_decode_audio4 should be repeatedly called until all data from a given packet is exhausted. Ran into this issue while running the FFmpeg regression tests after acolwell added a DCHECK to ensure we were always fully exhausting the packet after decode. BUG=141020 TEST=ffmpeg_regression_tests TBR=jam Review URL: https://chromiumcodereview.appspot.com/10869085 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@154220 0039d316-1c4b-4281-b951-d872f2087c98
* Convert WebAudio file handlers to use AudioBus.dalecurtis@chromium.org2012-08-307-137/+145
| | | | | | | | | | | | | | | Lets us remove AudioUtil::DeinterleaveAudioChannel(). Modifies AudioBus to add a new FromInterleavedPartial() function which allows for streaming deinterleave. Also adds supporting method: ZeroFramesPartial(). BUG=114700, 120319 TEST=unittests + WebAudio test page. Review URL: https://chromiumcodereview.appspot.com/10871051 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@154099 0039d316-1c4b-4281-b951-d872f2087c98
* Rename shared memory method parameters for consistency.dalecurtis@google.com2012-08-302-16/+16
| | | | | | | | | | | | | Brings internal consistency to the shared memory support methods since it doesn't look like we'll be able to get rid of them anytime soon. BUG=none TEST=none Review URL: https://chromiumcodereview.appspot.com/10873071 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@154033 0039d316-1c4b-4281-b951-d872f2087c98
* Fix encrypted AAC in BMFF by adding subsample info.strobe@google.com2012-08-294-17/+43
| | | | | | | | | | | | | | AAC audio is sent to the Source Buffer in ADTS format, which requires adding a header to each media sample. Encrypted subsample information was not being adjusted to compensate for the extra clear bytes. TEST=Manual, pending audio decryption support in decoder BUG=132351 Review URL: https://chromiumcodereview.appspot.com/10886022 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@153854 0039d316-1c4b-4281-b951-d872f2087c98
* Add a bool return value to media::Decryptor::GenerateKeyRequest().xhwang@chromium.org2012-08-296-7/+12
| | | | | | | | | | | | Add a bool return value in media::Decryptor::GenerateKeyRequest() call and add code in WebMeidaPlayerImpl to check this value. If GenerateKeyRequest() fails, prevent further calls to AddKey() and CancelKeyRequest(). BUG=145322 TEST=Called AddKey() after GenerateKeyRequest() fails and no crash. Review URL: https://chromiumcodereview.appspot.com/10896014 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@153818 0039d316-1c4b-4281-b951-d872f2087c98
* Support setting an explicit duration on MediaSource objectsvrk@chromium.org2012-08-285-6/+204
| | | | | | | | | | | This is the Chromium-side change to support MediaSource's duration attribute. BUG=NONE Review URL: https://chromiumcodereview.appspot.com/10879056 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@153613 0039d316-1c4b-4281-b951-d872f2087c98
* Disable AudioOutputMixer.dalecurtis@chromium.org2012-08-243-8/+43
| | | | | | | | | | | | | | | | | | | Removes AudioOutputMixer from compilation in preperation for converting the pipeline over to floats. Plan is as follows: 1. Remove AudioOutputMixer from compilation. 2. Convert all OnMoreData(uint8*, uint32, ...) calls to OnMoreData(AudioBus*, ...) 3. Refactor AudioRendererMixing->AudioMixerUtil (Or VectorMath). 4. Convert AudioOutputMixer to use float mixing and re-enable. BUG=114700 TEST=media_unittests Review URL: https://chromiumcodereview.appspot.com/10855086 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@153347 0039d316-1c4b-4281-b951-d872f2087c98
* use manual release for QTCaptureDecompressedVideoOutput.wjia@chromium.org2012-08-242-6/+11
| | | | | | | | | Need to make sure when setCaptureDevice:nil returns, QTCaptureDecompressedVideoOutput won't call captureOutput of VideoCaptureDeviceQTKit. BUG=139004 Review URL: https://chromiumcodereview.appspot.com/10875038 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@153307 0039d316-1c4b-4281-b951-d872f2087c98
* Introduce shared_memory_support media target for PPAPI.dalecurtis@chromium.org2012-08-2413-94/+207
| | | | | | | | | | | | | | | | | | Allows PPAPI to directly utilize shared memory utility code instead of duplicating it and CHECK'ing for equivalency. Required to eventually convert PPAPI to using an AudioBus and floats in PPB_Audio_Shared::Run(). http://crbug.com/114700 BUG=123203 TEST=unittests Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=152406 Review URL: https://chromiumcodereview.appspot.com/10826296 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@153262 0039d316-1c4b-4281-b951-d872f2087c98
* Revert r152236 since we found the culprit for bug 143440.scherkus@chromium.org2012-08-241-6/+6
| | | | | | | | | | | Turns out non-null media::Pipeline::pending_callbacks_ wasn't it -- on top the that we haven't received a single crash as a result of the CHECKs(). BUG=143539 TBR=jar Review URL: https://chromiumcodereview.appspot.com/10867054 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@153216 0039d316-1c4b-4281-b951-d872f2087c98
* Revert changes from r152523 and split bitstream converter creation and enabling.acolwell@chromium.org2012-08-243-19/+15
| | | | | | | | | | | | | Bitstream converter now gets created when we know the FFmpeg structs are valid. Enabling the converter no longer needs access to these structs so the crash is avoided. BUG=144432 TEST=Manual testing with GPU video decoder init failing and falling back to software decode. Review URL: https://chromiumcodereview.appspot.com/10879057 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@153204 0039d316-1c4b-4281-b951-d872f2087c98
* Split AudioRendererMixer::VectorFMAC into VectorMath library.dalecurtis@chromium.org2012-08-248-196/+267
| | | | | | | | | | | | | | | | Prepare the future home for more optimized vector math routines, specifically FMUL for volume adjustment. Additionally the FMAC operation will be used for browser side mixing as well as channel upmixing and downmixing. BUG=none TEST=unittests. Review URL: https://chromiumcodereview.appspot.com/10868037 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@153122 0039d316-1c4b-4281-b951-d872f2087c98
* Guard against ALSA returning insane frame counts for current delay.fischman@chromium.org2012-08-232-4/+4
| | | | | | | | | | | | | | | One video observed generated an alleged delay of 9222246136947932171 frames near the end of stream, which when converted to bytes and passed through a chain of uint32/int32 conversions became -4052 bytes, and then -46ms, triggering the DCHECK in the bug below. BUG=144281 Originally landed as r152919 and reverted in r152923. Review URL: https://chromiumcodereview.appspot.com/10869019 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@152977 0039d316-1c4b-4281-b951-d872f2087c98
* Use scoped_refptr::swap() instead of accident-prone scoped_refptr::release().fischman@chromium.org2012-08-231-7/+4
| | | | | | Review URL: https://chromiumcodereview.appspot.com/10876023 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@152924 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 152919 - Guard against ALSA returning insane frame counts for current ↵rsleevi@chromium.org2012-08-232-4/+4
| | | | | | | | | | | | | | | | | | | delay. One video observed generated an alleged delay of 9222246136947932171 frames near the end of stream, which when converted to bytes and passed through a chain of uint32/int32 conversions became -4052 bytes, and then -46ms, triggering the DCHECK in the bug below. BUG=144281 Review URL: https://chromiumcodereview.appspot.com/10869019 TBR=fischman@chromium.org Review URL: https://chromiumcodereview.appspot.com/10879027 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@152923 0039d316-1c4b-4281-b951-d872f2087c98
* Guard against ALSA returning insane frame counts for current delay.fischman@chromium.org2012-08-232-4/+4
| | | | | | | | | | | | | | One video observed generated an alleged delay of 9222246136947932171 frames near the end of stream, which when converted to bytes and passed through a chain of uint32/int32 conversions became -4052 bytes, and then -46ms, triggering the DCHECK in the bug below. BUG=144281 Review URL: https://chromiumcodereview.appspot.com/10869019 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@152919 0039d316-1c4b-4281-b951-d872f2087c98
* Remove start time logic since the Media Source spec now states the start ↵acolwell@chromium.org2012-08-227-108/+41
| | | | | | | | | | | time must always be 0. BUG=144240 Review URL: https://chromiumcodereview.appspot.com/10877022 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@152832 0039d316-1c4b-4281-b951-d872f2087c98
* Don't leak render thread on Stop().dalecurtis@google.com2012-08-222-7/+8
| | | | | | | | | | | Noticed this while refactoring. BUG=none TEST=drmemory windows. Review URL: https://chromiumcodereview.appspot.com/10866015 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@152786 0039d316-1c4b-4281-b951-d872f2087c98
* Add support for config changes during playback to FFmpegVideoDecoder.acolwell@chromium.org2012-08-219-105/+281
| | | | | | | | | | | Also fixed a minor config change related bug in SourceBufferStream. BUG=122913 TEST=PipelineIntegrationTest.MediaSource_MP4ConfigChange, SourceBufferStreamTest.ConfigChange_Basic, PipelineIntegrationTest.MediaSource_Seek Review URL: https://chromiumcodereview.appspot.com/10828425 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@152666 0039d316-1c4b-4281-b951-d872f2087c98
* clear QTCaptureDecompressedVideoOutput's delegate when CaptureSession is ↵wjia@chromium.org2012-08-211-0/+2
| | | | | | | | | | | released. When CaptureSession is released, it's needed to remove it as QTCaptureDecompressedVideoOutput's delegate. BUG=138326 Review URL: https://chromiumcodereview.appspot.com/10827458 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@152665 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 152622 - Add support for config changes during playback to ↵acolwell@chromium.org2012-08-219-277/+103
| | | | | | | | | | | | | | | | FFmpegVideoDecoder. Also fixed a minor config change related bug in SourceBufferStream. BUG=122913 TEST=PipelineIntegrationTest.MediaSource_MP4ConfigChange, SourceBufferStreamTest.ConfigChange_Basic, PipelineIntegrationTest.MediaSource_Seek Review URL: https://chromiumcodereview.appspot.com/10836304 TBR=acolwell@chromium.org Review URL: https://chromiumcodereview.appspot.com/10866002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@152629 0039d316-1c4b-4281-b951-d872f2087c98
* Add support for config changes during playback to FFmpegVideoDecoder.acolwell@chromium.org2012-08-219-103/+277
| | | | | | | | | | | Also fixed a minor config change related bug in SourceBufferStream. BUG=122913 TEST=PipelineIntegrationTest.MediaSource_MP4ConfigChange, SourceBufferStreamTest.ConfigChange_Basic, PipelineIntegrationTest.MediaSource_Seek Review URL: https://chromiumcodereview.appspot.com/10836304 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@152622 0039d316-1c4b-4281-b951-d872f2087c98
* Remux bear-640x360.webm so it is a valid WebM file & update the ChunkDemuxer ↵acolwell@chromium.org2012-08-213-24/+24
| | | | | | | | | | | | tests that rely on it. FFmpeg generated the existing file with negative timestamps at the beginning which violates the WebM spec. 26ms had to be added to all the timestamps in the file to make it valid. This slightly shifted the file layout and cluster boundaries so ChunkDemuxer tests needed to be updated. BUG=122913 Review URL: https://chromiumcodereview.appspot.com/10861030 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@152585 0039d316-1c4b-4281-b951-d872f2087c98
* Avoid race between initialization and shutdown of AVStream* underpinnings of ↵acolwell@chromium.org2012-08-212-4/+12
| | | | | | | | | | | | | | DemuxerStream. This is a hack to GpuVideoDecoder which should both fix the problem and be easily mergeable to 22. The real fix will have to wait for 23. BUG=143460 Review URL: https://chromiumcodereview.appspot.com/10827423 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@152523 0039d316-1c4b-4281-b951-d872f2087c98
* Upgrade AudioBus to support wrapping, interleaving.dalecurtis@chromium.org2012-08-2110-149/+406
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Introduces AudioBus::WrapMemory(...) for constructing an AudioBus object via an existing block of memory. Introduces AudioBus::CalculateMemorySize(...) for sizing the shared memory without having to create an empty AudioBus. Introduces AudioBus::FromInterleaved(...) and AudioBus:: ToInterleaved() for converting between formats. Fixes off by one errors in our existing audio_util implmentations. Removes InterleaveFloatToInt from audio_util. Removes data(), data_size() methods in favor of CopyTo() helper method combined with AudioBus::WrapMemory(). And of course adds new tests for all of the above. This CL is an extraction of all AudioBus functionality required to land the WIP CL which converts browser side to use AudioBus: http://codereview.chromium.org/10832285/ BUG=114700, 120319 TEST=unit tests under ASAN. Review URL: https://chromiumcodereview.appspot.com/10824304 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@152494 0039d316-1c4b-4281-b951-d872f2087c98
* Remove unused AudioTrackOutput.dalecurtis@google.com2012-08-215-423/+0
| | | | | | | | | | | | | Per qimin, this is no longer used. It doesn't look like it even compiles in the current state. BUG=none TEST=compile TBR=scherkus,qinmin Review URL: https://chromiumcodereview.appspot.com/10825465 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@152479 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 152406 - Introduce shared_memory_support media target for PPAPI.dalecurtis@google.com2012-08-2013-207/+94
| | | | | | | | | | | | | | | | | | | Allows PPAPI to directly utilize shared memory utility code instead of duplicating it and CHECK'ing for equivalency. Required to eventually convert PPAPI to using an AudioBus and floats in PPB_Audio_Shared::Run(). http://crbug.com/114700 BUG=123203 TEST=unittests Review URL: https://chromiumcodereview.appspot.com/10826296 TBR=dalecurtis@chromium.org Review URL: https://chromiumcodereview.appspot.com/10837350 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@152431 0039d316-1c4b-4281-b951-d872f2087c98
* Introduce shared_memory_support media target for PPAPI.dalecurtis@chromium.org2012-08-2013-94/+207
| | | | | | | | | | | | | | | | Allows PPAPI to directly utilize shared memory utility code instead of duplicating it and CHECK'ing for equivalency. Required to eventually convert PPAPI to using an AudioBus and floats in PPB_Audio_Shared::Run(). http://crbug.com/114700 BUG=123203 TEST=unittests Review URL: https://chromiumcodereview.appspot.com/10826296 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@152406 0039d316-1c4b-4281-b951-d872f2087c98
* Update dash test files.acolwell@chromium.org2012-08-202-0/+0
| | | | | | | | | BUG=122913 TBR=acolwell@chromium.org Review URL: https://chromiumcodereview.appspot.com/10854236 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@152386 0039d316-1c4b-4281-b951-d872f2087c98
* On mono mode, SL_SPEAKER_FRONT_CENTER is the channel mask;xians@chromium.org2012-08-202-2/+12
| | | | | | | | | | | | | On Stereo mode, SL_SPEAKER_FRONT_LEFT | SL_SPEAKER_FRONT_RIGHT should be used. For channel more than 2, we don't support it yet. BUG=143352 Review URL: https://chromiumcodereview.appspot.com/10855218 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@152313 0039d316-1c4b-4281-b951-d872f2087c98
* media::BindToLoop() is born, with example uses to slim down WebMediaPlayerProxy.fischman@chromium.org2012-08-184-0/+330
| | | | | | | | | | | WMPP could use further slimming-down / flamethrowering, but this pass just removes the lowest-hanging fruit, where WMPP is used as a silent trampoline to WMPI on the render thread. Review URL: https://chromiumcodereview.appspot.com/10855188 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@152239 0039d316-1c4b-4281-b951-d872f2087c98
* Upgrade DCHECK(pending_callbacks_.get()) to CHECK().scherkus@chromium.org2012-08-181-6/+6
| | | | | | | | | | | Collecting data to track down use-after-free crashes when executing callbacks in media::Pipeline. BUG=143440 Review URL: https://chromiumcodereview.appspot.com/10857062 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@152236 0039d316-1c4b-4281-b951-d872f2087c98
* Don't use an unretained read callback when calling AudioDecoder::Read().scherkus@chromium.org2012-08-172-6/+2
| | | | | | | | BUG=143440 Review URL: https://chromiumcodereview.appspot.com/10857066 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@152192 0039d316-1c4b-4281-b951-d872f2087c98
* Allow transitioning to HAVE_METADATA before pipeline initialization completes.acolwell@chromium.org2012-08-178-33/+113
| | | | | | | | | BUG=141429 Review URL: https://chromiumcodereview.appspot.com/10854151 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@151988 0039d316-1c4b-4281-b951-d872f2087c98
* use type std::string instead of integer for MediaLogEvent::TOTAL_BYTES_SETchangbin.shao@intel.com2012-08-163-7/+9
| | | | | | | | | | | | | because overflow happens for integer larger than 2^31. original review url: http://codereview.chromium.org/10828226 BUG=120328 Review URL: https://chromiumcodereview.appspot.com/10832334 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@151861 0039d316-1c4b-4281-b951-d872f2087c98
* Fix FFmpegAudioDecoder monotonically increasing timestamp check to ignore ↵acolwell@chromium.org2012-08-162-7/+14
| | | | | | | | | | | kNoTimestamp() BUG=142738 TEST=Cr142738/FFmpegRegressionTest.BasicPlayback/0 Review URL: https://chromiumcodereview.appspot.com/10857010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@151800 0039d316-1c4b-4281-b951-d872f2087c98
* Address post commit nits for r151132acolwell@chromium.org2012-08-152-5/+4
| | | | | | | | | BUG=141533 TBR=scherkus@chromium.org Review URL: https://chromiumcodereview.appspot.com/10830342 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@151740 0039d316-1c4b-4281-b951-d872f2087c98
* Upgrade Pipeline::Start() NOTREACHED() back into a CHECK().scherkus@chromium.org2012-08-151-5/+4
| | | | | | | | Unlike Seek(), calling Start() while running is a clear programmer error. If the pipeline did stop running and you called Start() again, make sure to no-op inside StartTask(). Review URL: https://chromiumcodereview.appspot.com/10857018 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@151721 0039d316-1c4b-4281-b951-d872f2087c98