| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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 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
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
| |
Review URL: https://chromiumcodereview.appspot.com/10876023
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@152924 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
| |
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 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
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
| |
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
|