Xxhwang@chromium.orgCall Decryptor::Stop() in FFmpegVideoDecoder::Reset().
| 文件 | 最后提交记录 | 最后更新时间 |
|---|---|---|
Update FFmpegAudioDecoder to support multi-frame packets. 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 | 13 年前 | |
Update FFmpegAudioDecoder to support multi-frame packets. 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 | 13 年前 | |
Remove media::AudioRendererImpl::SignalEndOfStream() and some other minor cleanup. BUG=none TEST=none Review URL: https://chromiumcodereview.appspot.com/10389138 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@138161 0039d316-1c4b-4281-b951-d872f2087c98 | 13 年前 | |
Remove media::AudioRendererImpl::SignalEndOfStream() and some other minor cleanup. BUG=none TEST=none Review URL: https://chromiumcodereview.appspot.com/10389138 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@138161 0039d316-1c4b-4281-b951-d872f2087c98 | 13 年前 | |
Rename AudioRendererBase and AudioRendererAlgorithmBase Neither of these classes are base classes anymore, so rename to AudioRendererImpl and AudioRendererAlgorithm respectively BUG=116645 TEST=media_unittests; chrome compiles and runs audio/video Review URL: http://codereview.chromium.org/9826028 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@131481 0039d316-1c4b-4281-b951-d872f2087c98 | 14 年前 | |
Upgrade AudioBus to support wrapping, interleaving. 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 | 13 年前 | |
Don't use an unretained read callback when calling AudioDecoder::Read(). 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 | 13 年前 | |
Replace Pipeline::kEnded state and HasEnded() methods with renderer-specific bools. The kEnded state was used to protect against the case where both renderers executed their ended callbacks at the same time, resulting in two tasks posted to the pipeline message loop but the first task would poll each renderer's HasEnded() method and execute the ended logic before the second task executed. Review URL: https://chromiumcodereview.appspot.com/10834236 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@150689 0039d316-1c4b-4281-b951-d872f2087c98 | 13 年前 | |
Add is_encrypted() in VideoDecoderConfig. This is needed so that decoders can check if the video stream is encrypted or not and decide if it can support decrypting and/or decodeing the stream. BUG=141784 TEST=media_unittest, encrypted media demo. Review URL: https://chromiumcodereview.appspot.com/10910293 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@157324 0039d316-1c4b-4281-b951-d872f2087c98 | 13 年前 | |
Move ChunkDemuxer handling from WMPProxy to WMPI and remove ChunkDemuxerClient BUG=148662 TEST=ChunkDemuxerTest, PipelineIntegrationTest, and Media Source LayoutTests all still pass. Review URL: https://codereview.chromium.org/10905236 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@156425 0039d316-1c4b-4281-b951-d872f2087c98 | 13 年前 | |
Add is_encrypted() in VideoDecoderConfig. This is needed so that decoders can check if the video stream is encrypted or not and decide if it can support decrypting and/or decodeing the stream. BUG=141784 TEST=media_unittest, encrypted media demo. Review URL: https://chromiumcodereview.appspot.com/10910293 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@157324 0039d316-1c4b-4281-b951-d872f2087c98 | 13 年前 | |
Dead code elimination: scythe.chrome_functions:segment.path %media% edition, round 1. Internal-only site: http://go/videostack/engineering/dead-code-elimination Review URL: https://chromiumcodereview.appspot.com/10837118 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@150129 0039d316-1c4b-4281-b951-d872f2087c98 | 13 年前 | |
Dead code elimination: scythe.chrome_functions:segment.path %media% edition, round 1. Internal-only site: http://go/videostack/engineering/dead-code-elimination Review URL: https://chromiumcodereview.appspot.com/10837118 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@150129 0039d316-1c4b-4281-b951-d872f2087c98 | 13 年前 | |
Update FFmpegAudioDecoder to support multi-frame packets. 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 | 13 年前 | |
Update FFmpegAudioDecoder to support multi-frame packets. 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 | 13 年前 | |
Update FFmpegAudioDecoder to support multi-frame packets. 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 | 13 年前 | |
Create video and audio decoder threads on demand. Instead of creating these worker threads when the decoder objects are instantiated, we now do it when Initialize is called. Doing this avoids spinning a video decoder thread for an audio tag. BUG=114699,116873 TEST=Thread count in the renderer process goes up by 3 and not 4 when an audio element is created. Review URL: https://chromiumcodereview.appspot.com/9632024 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@126077 0039d316-1c4b-4281-b951-d872f2087c98 | 14 年前 | |
Revert changes from r152523 and split bitstream converter creation and enabling. 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 | 13 年前 | |
Revert changes from r152523 and split bitstream converter creation and enabling. 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 | 13 年前 | |
Use enum instead of string in MessageLoopFactory::GetMessageLoop* This reinforces that only certain thread/message_loop can be created, which also makes thread/message_loop access easier and more reliable. Also use scoped_refptr<MessageLoopProxy> instead of raw MessageLoop pointer in various places. BUG=none TEST=media_unittest, content_unittest and normal <video> playback. Review URL: https://chromiumcodereview.appspot.com/10855051 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@151204 0039d316-1c4b-4281-b951-d872f2087c98 | 13 年前 | |
Move VideoRenderer out of Filter heirarchy. As a result the following classes no longer exist: Filter, FilterHost, CompositeFilter. I guess you could say we no longer have filters! BUG=108342,126069 TEST=none Review URL: https://chromiumcodereview.appspot.com/10796074 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@147834 0039d316-1c4b-4281-b951-d872f2087c98 | 13 年前 | |
RefCounted types should not have public destructors, delegate cleanup 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 | 13 年前 | |
Second attempt to land ffmpeg roll. Attempt 2 at landing http://codereview.chromium.org/9317096/ Same as before except for fixes in checkperms/ However, fixes have landed elsewhere for: - mp3 decode issue. - FrameRateNoVsyncCanvasInternalTest.fishbowl/0 BUG=110776 TEST=unittests, layouttests, trybots, perf tests... Review URL: https://chromiumcodereview.appspot.com/9447029 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@124501 0039d316-1c4b-4281-b951-d872f2087c98 | 14 年前 | |
Reorganize media bitstream converter classes. - Remove IdentityBitstreamConverter which is obsolete. - Remove FFmpegBitstreamConverter as it's not really used. Currently GpuVideoDecoder is the only place where EnableBitstreamConverter is called, and EnableBitstreamConverter is the only caller of FFmpegBitstreamConverter, but only when decoding mpeg4, whereas GVD only works with h264 ATM. - Remove BitstreamConverter since after removal of FFmpegBitstreamConverter, we only have one implementation of BitstreamConverter. - Remove BitstreamConverterTest. - Rename FFmpegH264BitstreamConverter to FFmpegH264ToAnnexBBitstreamConverter. - Rename H264BitstreamConverter to H264ToAnnexBBitstreamConverter. - Move H264ToAnnexBBitstreamConverter from media/base to media/filters. BUG=none TEST=media_unittests Review URL: https://chromiumcodereview.appspot.com/10690140 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@146420 0039d316-1c4b-4281-b951-d872f2087c98 | 13 年前 | |
Move VideoRenderer out of Filter heirarchy. As a result the following classes no longer exist: Filter, FilterHost, CompositeFilter. I guess you could say we no longer have filters! BUG=108342,126069 TEST=none Review URL: https://chromiumcodereview.appspot.com/10796074 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@147834 0039d316-1c4b-4281-b951-d872f2087c98 | 13 年前 | |
Reorganize media bitstream converter classes. - Remove IdentityBitstreamConverter which is obsolete. - Remove FFmpegBitstreamConverter as it's not really used. Currently GpuVideoDecoder is the only place where EnableBitstreamConverter is called, and EnableBitstreamConverter is the only caller of FFmpegBitstreamConverter, but only when decoding mpeg4, whereas GVD only works with h264 ATM. - Remove BitstreamConverter since after removal of FFmpegBitstreamConverter, we only have one implementation of BitstreamConverter. - Remove BitstreamConverterTest. - Rename FFmpegH264BitstreamConverter to FFmpegH264ToAnnexBBitstreamConverter. - Rename H264BitstreamConverter to H264ToAnnexBBitstreamConverter. - Move H264ToAnnexBBitstreamConverter from media/base to media/filters. BUG=none TEST=media_unittests Review URL: https://chromiumcodereview.appspot.com/10690140 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@146420 0039d316-1c4b-4281-b951-d872f2087c98 | 13 年前 | |
Call Decryptor::Stop() in FFmpegVideoDecoder::Reset(). To avoid freeze when user trying to seek() when a DecrptCB is pending. It also fix an issue that the stop_cb isn't fired when FFmpegVideoDecoder::Stop() is called when a DecryptCB is pending. BUG=151708 TEST=added new unittest to catch this Review URL: https://chromiumcodereview.appspot.com/10964055 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@158457 0039d316-1c4b-4281-b951-d872f2087c98 | 13 年前 | |
Add support for config changes during playback to 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/10828425 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@152666 0039d316-1c4b-4281-b951-d872f2087c98 | 13 年前 | |
Call Decryptor::Stop() in FFmpegVideoDecoder::Reset(). To avoid freeze when user trying to seek() when a DecrptCB is pending. It also fix an issue that the stop_cb isn't fired when FFmpegVideoDecoder::Stop() is called when a DecryptCB is pending. BUG=151708 TEST=added new unittest to catch this Review URL: https://chromiumcodereview.appspot.com/10964055 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@158457 0039d316-1c4b-4281-b951-d872f2087c98 | 13 年前 | |
Remove use of PipelineStatus from BufferedDataSource and FileDataSource. You know what's better than passing back PIPELINE_OK and not-PIPELINE_OK? A boolean. BUG=126067 TEST=none Review URL: https://chromiumcodereview.appspot.com/10808087 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@147978 0039d316-1c4b-4281-b951-d872f2087c98 | 13 年前 | |
Remove use of PipelineStatus from BufferedDataSource and FileDataSource. You know what's better than passing back PIPELINE_OK and not-PIPELINE_OK? A boolean. BUG=126067 TEST=none Review URL: https://chromiumcodereview.appspot.com/10808087 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@147978 0039d316-1c4b-4281-b951-d872f2087c98 | 13 年前 | |
Remove use of PipelineStatus from BufferedDataSource and FileDataSource. You know what's better than passing back PIPELINE_OK and not-PIPELINE_OK? A boolean. BUG=126067 TEST=none Review URL: https://chromiumcodereview.appspot.com/10808087 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@147978 0039d316-1c4b-4281-b951-d872f2087c98 | 13 年前 | |
Workaround for DXVA incorrectly claiming to be done with Decode() too soon. BUG=150824,150925,108121 Review URL: https://chromiumcodereview.appspot.com/10948051 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@157705 0039d316-1c4b-4281-b951-d872f2087c98 | 13 年前 | |
Workaround for DXVA incorrectly claiming to be done with Decode() too soon. BUG=150824,150925,108121 Review URL: https://chromiumcodereview.appspot.com/10948051 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@157705 0039d316-1c4b-4281-b951-d872f2087c98 | 13 年前 | |
Reorganize media bitstream converter classes. - Remove IdentityBitstreamConverter which is obsolete. - Remove FFmpegBitstreamConverter as it's not really used. Currently GpuVideoDecoder is the only place where EnableBitstreamConverter is called, and EnableBitstreamConverter is the only caller of FFmpegBitstreamConverter, but only when decoding mpeg4, whereas GVD only works with h264 ATM. - Remove BitstreamConverter since after removal of FFmpegBitstreamConverter, we only have one implementation of BitstreamConverter. - Remove BitstreamConverterTest. - Rename FFmpegH264BitstreamConverter to FFmpegH264ToAnnexBBitstreamConverter. - Rename H264BitstreamConverter to H264ToAnnexBBitstreamConverter. - Move H264ToAnnexBBitstreamConverter from media/base to media/filters. BUG=none TEST=media_unittests Review URL: https://chromiumcodereview.appspot.com/10690140 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@146420 0039d316-1c4b-4281-b951-d872f2087c98 | 13 年前 | |
Reorganize media bitstream converter classes. - Remove IdentityBitstreamConverter which is obsolete. - Remove FFmpegBitstreamConverter as it's not really used. Currently GpuVideoDecoder is the only place where EnableBitstreamConverter is called, and EnableBitstreamConverter is the only caller of FFmpegBitstreamConverter, but only when decoding mpeg4, whereas GVD only works with h264 ATM. - Remove BitstreamConverter since after removal of FFmpegBitstreamConverter, we only have one implementation of BitstreamConverter. - Remove BitstreamConverterTest. - Rename FFmpegH264BitstreamConverter to FFmpegH264ToAnnexBBitstreamConverter. - Rename H264BitstreamConverter to H264ToAnnexBBitstreamConverter. - Move H264ToAnnexBBitstreamConverter from media/base to media/filters. BUG=none TEST=media_unittests Review URL: https://chromiumcodereview.appspot.com/10690140 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@146420 0039d316-1c4b-4281-b951-d872f2087c98 | 13 年前 | |
Move VideoRenderer out of Filter heirarchy. As a result the following classes no longer exist: Filter, FilterHost, CompositeFilter. I guess you could say we no longer have filters! BUG=108342,126069 TEST=none Review URL: https://chromiumcodereview.appspot.com/10796074 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@147834 0039d316-1c4b-4281-b951-d872f2087c98 | 13 年前 | |
Change Read() size and return value to a size_t for clarity BUG=100641 TEST=N/A Review URL: http://codereview.chromium.org/8352019 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@106533 0039d316-1c4b-4281-b951-d872f2087c98 | 14 年前 | |
Add OVERRIDE to media/. BUG=104314 TEST=no change Review URL: http://codereview.chromium.org/8520033 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@110304 0039d316-1c4b-4281-b951-d872f2087c98 | 14 年前 | |
We are removing checksum/HMAC in WebM spec for encrypted content. xhwang's CL http://codereview.chromium.org/10918276 has been merged into this one. 10918276 CL updates the CDM interface, ClearKeyCdm, CdmWrapper and PPAPI code to reflect this change. - Remove checksum from DecryptConfig. - Remove HMAC from AesDecryptor. - Update AesDecryptorTest as all decrypts will not fail but the data checks will. - Remove FakeCheckSum from ffmpeg_video_decoder_unittest. - Remove HMAC from WebMClusterParser. - Remove checksum from ppapi_plugin_instance. BUG=150014 TEST=All media_unittests pass. TBR=dmichael Review URL: https://chromiumcodereview.appspot.com/10917308 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@157322 0039d316-1c4b-4281-b951-d872f2087c98 | 13 年前 | |
Move ChunkDemuxer handling from WMPProxy to WMPI and remove ChunkDemuxerClient BUG=148662 TEST=ChunkDemuxerTest, PipelineIntegrationTest, and Media Source LayoutTests all still pass. Review URL: https://codereview.chromium.org/10905236 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@156425 0039d316-1c4b-4281-b951-d872f2087c98 | 13 年前 | |
Move ChunkDemuxer handling from WMPProxy to WMPI and remove ChunkDemuxerClient BUG=148662 TEST=ChunkDemuxerTest, PipelineIntegrationTest, and Media Source LayoutTests all still pass. Review URL: https://codereview.chromium.org/10905236 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@156425 0039d316-1c4b-4281-b951-d872f2087c98 | 13 年前 | |
Remove lingering frames of old data when appending a new segment This CL adds a check in SourceBufferStream to delete old frames whose timestamps are in between the timestamps of the new frames overlapping it. BUG=146791 Review URL: https://chromiumcodereview.appspot.com/10952034 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@157688 0039d316-1c4b-4281-b951-d872f2087c98 | 13 年前 | |
Remove lingering frames of old data when appending a new segment This CL adds a check in SourceBufferStream to delete old frames whose timestamps are in between the timestamps of the new frames overlapping it. BUG=146791 Review URL: https://chromiumcodereview.appspot.com/10952034 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@157688 0039d316-1c4b-4281-b951-d872f2087c98 | 13 年前 | |
Remove lingering frames of old data when appending a new segment This CL adds a check in SourceBufferStream to delete old frames whose timestamps are in between the timestamps of the new frames overlapping it. BUG=146791 Review URL: https://chromiumcodereview.appspot.com/10952034 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@157688 0039d316-1c4b-4281-b951-d872f2087c98 | 13 年前 | |
Use enum instead of string in MessageLoopFactory::GetMessageLoop* This reinforces that only certain thread/message_loop can be created, which also makes thread/message_loop access easier and more reliable. Also use scoped_refptr<MessageLoopProxy> instead of raw MessageLoop pointer in various places. BUG=none TEST=media_unittest, content_unittest and normal <video> playback. Review URL: https://chromiumcodereview.appspot.com/10855051 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@151204 0039d316-1c4b-4281-b951-d872f2087c98 | 13 年前 | |
Use enum instead of string in MessageLoopFactory::GetMessageLoop* This reinforces that only certain thread/message_loop can be created, which also makes thread/message_loop access easier and more reliable. Also use scoped_refptr<MessageLoopProxy> instead of raw MessageLoop pointer in various places. BUG=none TEST=media_unittest, content_unittest and normal <video> playback. Review URL: https://chromiumcodereview.appspot.com/10855051 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@151204 0039d316-1c4b-4281-b951-d872f2087c98 | 13 年前 | |
Remove VideoDecoder::PrepareForShutdownHack() and friends. As of r156011 the pause/flush/stop shutdown dance is no more. Now that Pipeline immediately calls Stop() during teardown we can remove a pile of hacks that were used to signal that teardown was starting. BUG=110228 Review URL: https://chromiumcodereview.appspot.com/10918172 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@156525 0039d316-1c4b-4281-b951-d872f2087c98 | 13 年前 | |
Remove VideoDecoder::PrepareForShutdownHack() and friends. As of r156011 the pause/flush/stop shutdown dance is no more. Now that Pipeline immediately calls Stop() during teardown we can remove a pile of hacks that were used to signal that teardown was starting. BUG=110228 Review URL: https://chromiumcodereview.appspot.com/10918172 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@156525 0039d316-1c4b-4281-b951-d872f2087c98 | 13 年前 | |
Address post commit nits for r151132 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 | 13 年前 |