| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
| |
BUG=89095
TEST=VideoRendererBaseTest.GetCurrentFrame_AfterError, VideoRendererBaseTest.Error_DuringPaint, VideoRendererBaseTest.GetCurrentFrame_AfterStop
Review URL: http://codereview.chromium.org/7360005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@92542 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=73195
TEST=none
Review URL: http://codereview.chromium.org/7352006
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@92448 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
BUG=42956
TEST=test_shell_tests
Review URL: http://codereview.chromium.org/7227016
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@92435 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
related code.
This implementation makes explicit the transitions being effected and the states
expected to be involved in each transition, instead of the previous
implementation's use of implicit information (e.g. "we just went from Idle to
Loaded, therefore we must be Abort()'ing"). This is not only much more readable
but also enables implementation of Reset() (for Seek) and replaces Abort()
(which never had a well-defined purpose that was consistent in documentation and
implementation, nor a customer use-case).
DID some TODOs in OmxVideoDecodeAccelerator:
- Replaced error-code checking boilerplate w/ a failure-handling macro.
- Simplified the component-name-getting dance in CreateComponent().
Updated the PPAPI interface:
- Removed GetConfigs & AssignSysmemBuffers (neither of which ever came into its own)
- Replaced Abort with Reset, and added Destroy (which still nobody calls, but
that'll change shortly).
- Cleaned up the interface's documentation.
BUG=86122
TEST=OVDATest, gles2 sample plugin
Review URL: http://codereview.chromium.org/7338010
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@92383 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
These were added in r28089 (http://codereview.chromium.org/256059) but
are unnecessary bloat for everyone to carry around, even those that don't
use emacs or vim.
In an earlier change, I added editor config files in src/tools/emacs/
and src/tools/vim/ so users of the appropriate editor can source those
instead.
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/7310019
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@92046 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
via appendData().
BUG=86536
TEST=ChunkDemuxerTest.*
Review URL: http://codereview.chromium.org/7329026
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@92011 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
I.e. now host signals that it is ready to accept new data using sockets,
not IPC messages.
That should reduce audio latency.
Right now we are using high latency code path by default, low latency
would be used only when flag --enable-low-latency-audio is specified.
Changed unit tests to test low latency code path, as it should become
default soon.
http://codereview.chromium.org/7253003
BUG=http://code.google.com/p/chromium/issues/detail?id=87640
TEST=Everything should just work. I changed audio_renderer_impl_unittest.cc to explicitly force low latency code path.
Review URL: http://codereview.chromium.org/7253003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@91727 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
(webrtc, libjinle etc), which is not allowed in media. So move rtc_video_decoder* from media/filter/ to content/renderer/media/.
patch by ronghuawu@google.com
BUG=none
TEST=unit_tests, media_unittests
Review URL: http://codereview.chromium.org/7193001
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@91721 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
TBR=scherkus@chromium.org
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@91328 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
Refactored ChunkDemuxerTest to include a ShutdownDemuxer() that cleans up memory.
BUG=none
TEST=heapcheck passes for media_unittests
Review URL: http://codereview.chromium.org/7290030
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@91267 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
1) works for Linux, OSX, Windows
2) mostly renaming #defines
3) avcodec_thread_init is deprecated and has been removed, see
http://ffmpeg.org/pipermail/ffmpeg-cvslog/2011-February/034326.html
4) will require a rebaseline of
platform/chromium-mac/media/video-currentTime-set-expected.txt
platform/chromium-win/media/video-currentTime-set-expected.txt
with a change of
-video.currentTime.toFixed(2) == '5.82'
+video.currentTime.toFixed(2) == '5.81'
BUG=87831
TESTS=ran Theora Testsuite, media tests, WebKit media tests
Review URL: http://codereview.chromium.org/6993042
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@91259 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
BUG=72317
TEST=None
R=rsesek@chromium.org
Review URL: http://codereview.chromium.org/7218065
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@91179 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
DirectShow is used for controlling cameras and receiving video frames.
It implements all COM functions needed to support cameras using DirectShow and don't depend on Microsofts base classes for DirectShow filters.
It supersedes the patch http://codereview.chromium.org/6929064/.
Review URL: http://codereview.chromium.org/7229013
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@91096 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This is done by inserting tokens into the command-buffer stream
when synchronization is needed, and adding a
last-read/last-written token pair to each IPC message. This
allowed me to remove the bogus FinishGL() calls from the gles2
sample pepper plugin.
As part of this CL, the return value for VideoDecodeAccelerator::{Decode,Flush,Abort} changed from bool to void. These are all async methods so errors ought to be signaled using callbacks.
BUG=none
TEST=gles2 works, no crashes; trybots
Review URL: http://codereview.chromium.org/7260008
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90971 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
| |
TBR=scherkus@chromium.org
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90970 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=86536
TEST=None
TBR=scherkus@chromium.org
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90967 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=86536
TEST=ChunkDemuxerTest.*
Review URL: http://codereview.chromium.org/7203002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90966 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
BUG=87490
TEST=None
Review URL: http://codereview.chromium.org/7230023
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90818 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
including refactoring MockTimeProvider out of media/.
BUG=None
TEST=No visible changes. Ensure that changed unittests all pass.
Review URL: http://codereview.chromium.org/7232023
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90764 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
| |
TBR=scherkus
Review URL: http://codereview.chromium.org/7253040
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90750 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
This should hotfix the issues with the heap checker bot.
TBR=scherkus
Review URL: http://codereview.chromium.org/7277037
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90746 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
All of this code hasn't been used in over a year and has been replaced by VideoDecodeAccelerator and it's corresponding OpenMAX implementation OmxVideoDecodeAccelerator.
BUG=none
TEST=the world still compiles
Review URL: http://codereview.chromium.org/7066071
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90697 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
ctors.
Removed these pseudo-copy-constructors, since they look a lot like violations of
dependency rules and since they're not necessary given the explicit
constructors.
BUG=none
TEST=gles2.cc builds & works with this patched in.
Review URL: http://codereview.chromium.org/7276005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90628 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
pump and an X message pump.
The changes:
* Rename MessagePumpGlibX to MessagePumpX.
* Rename MessagePumpForUI to MessagePumpGlib.
* Move some stuff out of MessagePumpGlib, and into MessagePumpGtk and MessagePumpX.
* Rename MessagePumpForUI::Observer to MessageObserver, moved the platform-specific implementations into MessagePumpGtk and MessagePumpX. Ditto for MessagePumpForUI::Dispatcher.
MessagePumpX is independent of MessagePumpGtk. At the moment, MessagePumpX does process some GDK event, but once we have a complete native_widget_x, we can take out the GDK processing and things should continue to work.
BUG=none
TEST=existing message-pump tests.
Review URL: http://codereview.chromium.org/7250001
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90418 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
BUG=none
TEST=none
R=vandebo@chromium.org
Review URL: http://codereview.chromium.org/7241004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90123 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
CID=13686,13687,13688,13709,13710,13711,13712,13713,13714,13718,13715,13716,
13717,13796,13797,13798,13811,13812,13813,14708,16329
BUG=none
TEST=none
R=vandebo@chromium.org
Review URL: http://codereview.chromium.org/7218031
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90112 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
CID=13721,14709,16392,17192,17585
BUG=none
TEST=none
R=kmadhusu@chromium.org
Review URL: http://codereview.chromium.org/7206066
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90086 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/7230003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90051 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
BUG=none
TEST=none
CID=16736
Review URL: http://codereview.chromium.org/7217024
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90001 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
CID=9299,13805,14203,14284,14459,15435,15897,16023,16614,16773,16819,16820,
16908,16916
BUG=none
TEST=none
R=kmadhusu@chromium.org
Review URL: http://codereview.chromium.org/7215027
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@89897 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
because ffmpeg does not expect compressed files.
(It does not make sense to compress audio/video anyways...)
http://codereview.chromium.org/7044092
BUG=47381
TEST=http://www.deafmac.org/html5/grinchsample.mp4 (Please clear browser cache, otherwise it can contain original GZIP-ed file requested by old browser)
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=89532
Review URL: http://codereview.chromium.org/7044092
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@89858 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
because ffmpeg does not expect compressed files.
(It does not make sense to compress audio/video anyways...)
http://codereview.chromium.org/7044092
BUG=47381
TEST=http://www.deafmac.org/html5/grinchsample.mp4 (Please clear browser cache, otherwise it can contain original GZIP-ed file requested by old browser)
Review URL: http://codereview.chromium.org/7044092
TBR=enal@chromium.org
Review URL: http://codereview.chromium.org/7205011
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@89546 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/7173001
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@89533 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
because ffmpeg does not expect compressed files.
(It does not make sense to compress audio/video anyways...)
http://codereview.chromium.org/7044092
BUG=47381
TEST=http://www.deafmac.org/html5/grinchsample.mp4 (Please clear browser cache, otherwise it can contain original GZIP-ed file requested by old browser)
Review URL: http://codereview.chromium.org/7044092
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@89532 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
sdch, skia, and testing.
BUG=28727
Review URL: http://codereview.chromium.org/7165009
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@89369 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This is required for various reasons:
- Clients don't necessarily know the dimensions of the video they feed to the
decoder. It's the decoder's job to figure it out from the encoded stream, but
it only gets a crack at that once Decode() is called, after initialization is
done.
- PPAPI plugins get to call PPB_* functions at will, which means that (to avoid
having to code re-entrancy into everything on the browser side) PPB_*
functions aren't allowed to call into PPP_* functions. So the previous call
sequence of Initialize->ProvidePictureBuffers->CreateTextures would deadlock
(requiring plugins to pre-create any textures they were going to get asked for
in the future).
Also a bit of minor cleanup & added TODOs for future cleanup, and changed
GetConfigs()' signature to match the PPAPI version of itself (return bool
instead of void).
BUG=none
TEST=OVDATest passes on kaen.
Review URL: http://codereview.chromium.org/7147006
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@89251 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Windows.
Added a three second timeout for not receiving an OnData callback in AudioInputController and a one second timeout for not receiving any callback in PCMWaveInAudioInputStream. Also added a unit test fixture for it.
BUG=79936
TEST=Unplug microphone while using speech input, browser should no longer hang.
Review URL: http://codereview.chromium.org/7129057
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@89197 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
Don't use base::SysInfo::OperatingSystemVersionNumbers, because it calls
Gestalt, which has a few bad properties. Introduce new functions that perform
specific version checks.
BUG=85972
TEST=base_unittests MacUtilTest.IsOSEllipsis
Review URL: http://codereview.chromium.org/7144007
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@89028 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
/Users/thakis/src/chrome-git/src/base/md5.cc:250:24: error: The sizeof expression in 'memset' has type 'struct Context *', the same type that the first argument has. The sizeof expression should probably have type 'struct Context' instead. [-Werror]
memset(ctx, 0, sizeof(ctx)); /* In case it's sensitive */
^
1 error generated.
/Users/thakis/src/chrome-git/src/media/video/capture/fake_video_capture_device.cc:75:32: error: The sizeof expression in 'memset' has type 'unsigned char *', the same type that the first argument has. The sizeof expression should probably have type 'unsigned char' instead. [-Werror]
memset(fake_frame_.get(), 0, sizeof(fake_frame_.get()));
^
1 error generated.
The warning is up for review here: http://lists.cs.uiuc.edu/pipermail/cfe-commits/Week-of-Mon-20110613/042719.html
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/7003140
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@88855 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=None
TEST=None
CID=16744
Review URL: http://codereview.chromium.org/7044110
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@88786 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=None
TEST=None
CID=17000
Review URL: http://codereview.chromium.org/6995134
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@88785 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Fills in implementation for AssignGLESBuffers where it was missing. Also
updates OmxVideoDecodeAccelerator to reflect the changes.
BUG=NONE
TEST=NONE
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=88698
Review URL: http://codereview.chromium.org/6965010
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@88733 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Fills in implementation for AssignGLESBuffers where it was missing. Also
updates OmxVideoDecodeAccelerator to reflect the changes.
BUG=NONE
TEST=NONE
Review URL: http://codereview.chromium.org/6965010
TBR=vrk@google.com
Review URL: http://codereview.chromium.org/7044115
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@88702 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
Fills in implementation for AssignGLESBuffers where it was missing. Also
updates OmxVideoDecodeAccelerator to reflect the changes.
BUG=NONE
TEST=NONE
Review URL: http://codereview.chromium.org/6965010
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@88698 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
but now
client and audio threads are the same, so we don't need locks/ref counts/etc.
http://codereview.chromium.org/7117001
BUG=62588
Review URL: http://codereview.chromium.org/7117001
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@88369 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
extradata buffer.
BUG=82438
Review URL: http://codereview.chromium.org/7137002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@88354 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
BUG=85100
TEST=HTML5 videos with stereo channel layout on Mac with unconfigured speakers can be heard.
Review URL: http://codereview.chromium.org/7134001
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@88339 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
- Rip out support for HW decode to sysmem buffers from OVDA (which doesn't work
and for which we don't have a need (yet)).
- Deliver on the promise of Picture to provide a bitstream buffer id from which
the picture was decoded by propagating the bitstream buffer id from input
buffers (BitstreamBuffer) to output buffers (Picture) through
OMX_BUFFERHEADERTYPE's nTimeStamp field (HACK!). Now checking for this in the
test.
- Free output buffers uniformly: both dismiss from OMX & from client at the same
time, during OnStateChangeExecutingToIdle, instead of freeing from OMX during
OnStateChangeExecutingToIdle and from client during OnPortCommandFlush.
- Clean up OVDA implementation:
- Replaced unnecessary OMX_AllocateBuffer with OMX_UseBuffer.
- Unify two parallel vector<>s (one of which is of pairs) into a single
map<id, struct> to ease readability and to increase robustness (should be
harder now to have these data items fall out of sync with each other).
- Removed unnecessary output_buffer_count_ field.
- Removed unused field output_buffer_size_.
- Replaced use of explicitly-managed omx_buff_ids_ map with implicit use of
pAppPrivate field on input buffer headers.
BUG=none
TEST=run the test per instructions in http://codereview.chromium.org/7112019
Review URL: http://codereview.chromium.org/6993051
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@87996 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
These haven't been used in quite some time and have been replaced by the newer VideoDecoderAccelerator set of classes.
BUG=none
TEST=the world still compiles
Review URL: http://codereview.chromium.org/6993016
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@87841 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
Follow up CL to http://codereview.chromium.org/6990065/
BUG=75458
TEST=Open file out29367209.webm from Bug#75458 and video playback
should be more than one frame.
Review URL: http://codereview.chromium.org/7058020
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@87839 0039d316-1c4b-4281-b951-d872f2087c98
|