summaryrefslogtreecommitdiffstats
path: root/remoting/codec
Commit message (Collapse)AuthorAgeFilesLines
* LOG(INFO) tidying in remoting/scottmg@chromium.org2013-11-252-8/+8
| | | | | | | | | | | Some deletions, some to VLOG, some to ERROR. R=sergeyu@chromium.org, wez@chromium.org BUG=322805 Review URL: https://codereview.chromium.org/82923009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@237092 0039d316-1c4b-4281-b951-d872f2087c98
* Add VP9 decode support to the remoting client.wez@chromium.org2013-10-224-3/+28
| | | | | | | | BUG=260879 Review URL: https://codereview.chromium.org/26921005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@230003 0039d316-1c4b-4281-b951-d872f2087c98
* Move VideoDecoderVp8 to VideoDecoderVpx base that will support VP8/9.wez@chromium.org2013-10-198-292/+335
| | | | | | | | BUG=260879 Review URL: https://codereview.chromium.org/26888009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@229624 0039d316-1c4b-4281-b951-d872f2087c98
* Clean up a few more unused globals.thakis@chromium.org2013-09-252-8/+0
| | | | | | | | | | | Found by clang's new -Wunused-const-variable. BUG=290204 R=akalin@chromium.org, avi@chromium.org, isherman@chromium.org, jamesr@chromium.org, joi@chromium.org, sergeyu@chromium.org, sky@chromium.org, thestig@chromium.org, xhwang@chromium.org Review URL: https://codereview.chromium.org/24649002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@225273 0039d316-1c4b-4281-b951-d872f2087c98
* Remove dependency on Skia from chromoting client.sergeyu@chromium.org2013-09-258-180/+198
| | | | | | | | | | | | | | Now DesktopRegion, DesktopRect and DesktopSize are used instead of corresponding skia types. R=alexeypa@chromium.org TBR=reed@google.com (for _moved_ skia dependency) Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=224101 Review URL: https://codereview.chromium.org/23440046 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@225265 0039d316-1c4b-4281-b951-d872f2087c98
* Cleanup VideoDecoder interface.lambroslambrou@chromium.org2013-09-236-78/+34
| | | | | | | | This removes some un-needed methods and the DecodeResult enumeration. Review URL: https://chromiumcodereview.appspot.com/23702056 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@224762 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 224101 "Remove dependency on Skia from chromoting client."sergeyu@chromium.org2013-09-198-200/+184
| | | | | | | | | | | | | | | | | > Remove dependency on Skia from chromoting client. > > Now DesktopRegion, DesktopRect and DesktopSize are used instead of > corresponding skia types. > > TBR=reed@google.com (for _moved_ skia dependency) > > Review URL: https://chromiumcodereview.appspot.com/23440046 TBR=sergeyu@chromium.org Review URL: https://codereview.chromium.org/24217003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@224205 0039d316-1c4b-4281-b951-d872f2087c98
* Remove dependency on Skia from chromoting client.sergeyu@chromium.org2013-09-198-184/+200
| | | | | | | | | | | Now DesktopRegion, DesktopRect and DesktopSize are used instead of corresponding skia types. TBR=reed@google.com (for _moved_ skia dependency) Review URL: https://chromiumcodereview.appspot.com/23440046 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@224101 0039d316-1c4b-4281-b951-d872f2087c98
* Simplify VideoEncoder interface.sergeyu@chromium.org2013-09-1311-474/+139
| | | | | | | | | | | This should also avoid assert in the linked bug. BUG=284775 R=wez@chromium.org Review URL: https://codereview.chromium.org/23477059 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@223152 0039d316-1c4b-4281-b951-d872f2087c98
* Remove obsolete audio codecs and channel configurations.sergeyu@chromium.org2013-09-125-338/+0
| | | | | | | | | | | | We always use Opus for audio. Removed Speex audio codec. Also removed non-multiplexed channel configurations for control, event and audio channels. R=wez@chromium.org Review URL: https://codereview.chromium.org/24041005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@222855 0039d316-1c4b-4281-b951-d872f2087c98
* Fix uninitialized field in constructor.etienneb@chromium.org2013-08-211-1/+2
| | | | | | | | | | | | This issue was found by a linter. R=rmsousa@chromium.org BUG= Review URL: https://chromiumcodereview.appspot.com/22909025 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@218787 0039d316-1c4b-4281-b951-d872f2087c98
* Use a direct include of time headers in ppapi/, printing/, remoting/.avi@chromium.org2013-06-284-4/+4
| | | | | | | | | | BUG=254986 TEST=none TBR=ben@chromium.org Review URL: https://codereview.chromium.org/18052008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@209151 0039d316-1c4b-4281-b951-d872f2087c98
* Added the desktop shape fields to VideoPacket.alexeypa@chromium.org2013-06-215-0/+112
| | | | | | | | If the host passes the desktop shape region in a VideoPacket the client will use it to draw transparent regions. The client also informs the web-app of the desktop shape changes, so it can set the input passthrough region correctly. Review URL: https://chromiumcodereview.appspot.com/17511004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@207789 0039d316-1c4b-4281-b951-d872f2087c98
* Rewrite scoped_ptr<T>(NULL) to use the default ctor in remoting/.dcheng@chromium.org2013-06-143-7/+4
| | | | | | | | | | | This is the result of running the rewrite_scoped_ptr_ctor_null tool across all files built on Linux in the remoting/ directory. BUG=173286 Review URL: https://chromiumcodereview.appspot.com/16964002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@206247 0039d316-1c4b-4281-b951-d872f2087c98
* Remove screen capturers from media/video/capture/screen.sergeyu@chromium.org2013-06-071-1/+0
| | | | | | | | | | | | Screen capturers have been moved to webrtc. This change switches chromoting and getUserMedia() to the new copy of the capturers in webrtc and removes old copy from media. TBR=jschuh@chromium.org Review URL: https://chromiumcodereview.appspot.com/15692018 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@204966 0039d316-1c4b-4281-b951-d872f2087c98
* Roll webrtc 4143:4179sergeyu@chromium.org2013-06-051-12/+8
| | | | | | | | R=alexeypa@chromium.org, ronghuawu@chromium.org Review URL: https://codereview.chromium.org/16383002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@204088 0039d316-1c4b-4281-b951-d872f2087c98
* Use webrtc::DesktopCapturer for screen capturer implementation.sergeyu@chromium.org2013-05-1610-231/+237
| | | | | | | | | | | | | Screen capturers are being moved from media/video/capture/screen to third_party/webrtc. This CL is an intermediate step in that process. Depends on https://webrtc-codereview.appspot.com/1322007/ TBR=brettw@chromium.org (third_party/webrtc dependency) Review URL: https://chromiumcodereview.appspot.com/13983010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@200504 0039d316-1c4b-4281-b951-d872f2087c98
* Reduce jitter from uneven SincResampler buffer size requests.dalecurtis@google.com2013-05-081-3/+6
| | | | | | | | | | | | | | Ensures all buffer requests are for the same size. Reduces jitter by allowing clients with specific buffer size requirements to avoid overreading. BUG=none TEST=media_unittests. R=crogers@google.com, henrika@chromium.org, sergeyu@chromium.org Review URL: https://codereview.chromium.org/14189035 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@199032 0039d316-1c4b-4281-b951-d872f2087c98
* Use libyuv for non-scaling RGB<->YUV conversions.wez@chromium.org2013-04-191-1/+1
| | | | | | | | | | | | This switches Chromoting hosts to use libyuv for RGB->YUV and clients to use libyuv for non-scaling rendering. TEST=remoting_unittests BUG=145561 Review URL: https://chromiumcodereview.appspot.com/13474013 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@195268 0039d316-1c4b-4281-b951-d872f2087c98
* Rewrite scoped_array<T> to scoped_ptr<T[]> in remoting/, Linux edition.dcheng@chromium.org2013-04-085-12/+12
| | | | | | | | | | This changelist was automatically generated using a clang tool. BUG=171111 Review URL: https://codereview.chromium.org/13642007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@192873 0039d316-1c4b-4281-b951-d872f2087c98
* Update calls to scoped_array<T>::reset(NULL) to use the no-arg version.dcheng@chromium.org2013-04-041-1/+1
| | | | | | | | | | | scoped_ptr<T[]>, which will replace scoped_array<T>, does not support passing NULL as an explicit argument to reset(). BUG=171118 Review URL: https://codereview.chromium.org/13650005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@192407 0039d316-1c4b-4281-b951-d872f2087c98
* Add debug message to help investigate Opus tests flakiness.sergeyu@chromium.org2013-03-021-0/+1
| | | | | | | | BUG=179563 Review URL: https://codereview.chromium.org/12390046 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@185657 0039d316-1c4b-4281-b951-d872f2087c98
* Move screen capturers from remoting/capturer to media/video/capturer/screensergeyu@chromium.org2013-01-289-39/+43
| | | | | | | | | | | | | | | | Screen capturers will be used in content, and so they have to be moved to avoid dependency on remoting/capturer in content. Beside moving the files this CL also renames classes as follows: remoting::VideoFrameCapturer -> media::ScreenCapturer remoting::VideoFrame -> media::ScreenCaptureFrame remoting::CaptureData -> media::ScreenCaptureData BUG=134249 Review URL: https://codereview.chromium.org/12047101 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@179218 0039d316-1c4b-4281-b951-d872f2087c98
* Roll speex, flac, libvpx DEPS to get shim header changes.phajdan.jr@chromium.org2013-01-074-4/+8
| | | | | | | | | | | R=dalecurtis TBR=darin,garykac BUG=165264 Review URL: https://codereview.chromium.org/11791006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@175348 0039d316-1c4b-4281-b951-d872f2087c98
* Simplify remoting::CaptureDatasergeyu@chromium.org2012-12-224-53/+21
| | | | | | | | | | | | | 1. Previously CaptureData had pixel_format() field but we only use that class for RGB. Removed that field. 2. Removed DataPlanes because it's doesn't make sense for RGB32. Instead CaptureData class now stored data pointer and stride values. TBR=tsepez@chromium.org Review URL: https://chromiumcodereview.appspot.com/11637031 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@174519 0039d316-1c4b-4281-b951-d872f2087c98
* Linux: use generated shim headers for opus.phajdan.jr@chromium.org2012-12-192-2/+2
| | | | | | | | BUG=165264 Review URL: https://codereview.chromium.org/11635009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@173978 0039d316-1c4b-4281-b951-d872f2087c98
* Move screen capturers to remoting/capturer.sergeyu@chromium.org2012-12-185-4/+5
| | | | | | | | | | | | Separating screen capturer code from the rest of remoting code so that it can be reused for screen capturing in WebRTC. Also added struct MouseCursorShape to avoid protobuf dependency. BUG=134249 Review URL: https://codereview.chromium.org/11470028 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@173789 0039d316-1c4b-4281-b951-d872f2087c98
* Capturing and encoding time are now calculated by the thread doing the ↵alexeypa@chromium.org2012-11-284-3/+12
| | | | | | | | operation. This gives more accurate result in the case when multiple frames are scheduled for capturing/encoding simultaneously. The side effect is that the thread scheduling delays are not accounted for any more. Review URL: https://chromiumcodereview.appspot.com/11280068 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@170045 0039d316-1c4b-4281-b951-d872f2087c98
* Collapse AudioRendererMixer and OnMoreDataResampler into AudioTransform.dalecurtis@chromium.org2012-11-212-2/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently we have roughly equivalent functionality in two places, and the CloudView project will add a third. As such there's a need for a single super class which can handle mixing, resampling, and general conversion from one set of AudioParameters to another. This change introduces the AudioTransform object which collapses the key functionality from AudioRendererMixer and OnMoreDataResampler into a single AudioTransform class which can do everything and is oblivious to the peculiars of RenderCallback vs AudioSourceCallback. It also introduces output_frames_ready() methods to the AudioPullFifo and MultiChannelResampler classes so that buffer delay can be measured accurately without resorting to input vs output byte counting. Due to the bulk of AudioRendererMixer's functionality moving into the new AudioTransform, it made sense to move some decisions into the AudioRendererMixerInput class as well. On my Z600, benchmarking 50000 iterations: Convert() w/ FIFO took 7030.11ms. Convert() w/o FIFO took 5218.83ms. BUG=none TEST=AudioTransform* unittests. TBR=sergeyu Review URL: https://chromiumcodereview.appspot.com/11410012 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@168976 0039d316-1c4b-4281-b951-d872f2087c98
* Limit number of frames per audio packet in Opus and Speex decoders.sergeyu@chromium.org2012-11-072-4/+23
| | | | | | | | | | | | | Previously Opus and Speex decoders didn't validate number of frames in the incoming packets, so there was a potential buffer overflow when there are enough frames in a packet to cause integer overflow. BUG=159553,159229 Review URL: https://chromiumcodereview.appspot.com/11368112 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@166368 0039d316-1c4b-4281-b951-d872f2087c98
* Improve codec tests to test a broader variety of dimensions.wez@chromium.org2012-10-231-10/+16
| | | | | | | | | BUG=147522 Review URL: https://chromiumcodereview.appspot.com/11195062 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@163679 0039d316-1c4b-4281-b951-d872f2087c98
* Add opus audio codec support in remotingsergeyu@chromium.org2012-10-237-0/+669
| | | | | | | | BUG=154714 Review URL: https://codereview.chromium.org/11189047 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@163675 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 163650 - Add opus audio codec support in remotingsergeyu@chromium.org2012-10-237-669/+0
| | | | | | | | | | | | BUG=154714 Review URL: https://chromiumcodereview.appspot.com/11189047 TBR=sergeyu@chromium.org Review URL: https://codereview.chromium.org/11252002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@163661 0039d316-1c4b-4281-b951-d872f2087c98
* Add opus audio codec support in remotingsergeyu@chromium.org2012-10-237-0/+669
| | | | | | | | | BUG=154714 Review URL: https://chromiumcodereview.appspot.com/11189047 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@163650 0039d316-1c4b-4281-b951-d872f2087c98
* Remove ZLib codec support from chromoting host and client.sergeyu@chromium.org2012-10-188-265/+154
| | | | | | | | | We were not using ZLIB codec anyway, so there is no reason to keep it around. Review URL: https://chromiumcodereview.appspot.com/11195029 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@162635 0039d316-1c4b-4281-b951-d872f2087c98
* Linux: fix build with system speex (2/2)phajdan.jr@chromium.org2012-10-042-3/+0
| | | | | | | | | BUG=145507 TEST=none Review URL: https://chromiumcodereview.appspot.com/11029021 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@160255 0039d316-1c4b-4281-b951-d872f2087c98
* Swap CHECKs for DCHECKs in VideoEncoderVP8's active map loop.wez@chromium.org2012-10-021-5/+5
| | | | | | Review URL: https://chromiumcodereview.appspot.com/11021005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@159637 0039d316-1c4b-4281-b951-d872f2087c98
* Limit audio buffer size in the audio player used by the chromoting client.sergeyu@chromium.org2012-09-131-1/+3
| | | | | | Review URL: https://chromiumcodereview.appspot.com/10914210 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@156465 0039d316-1c4b-4281-b951-d872f2087c98
* Fix typo in remoting::VideoEncoderVp8.wez@chromium.org2012-09-071-1/+1
| | | | | | | | | BUG=146840 Review URL: https://chromiumcodereview.appspot.com/10914124 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@155369 0039d316-1c4b-4281-b951-d872f2087c98
* Cleaned up codec_test.cckxing@chromium.org2012-08-251-7/+6
| | | | | | | | | BUG=92086 Review URL: https://chromiumcodereview.appspot.com/10871068 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@153361 0039d316-1c4b-4281-b951-d872f2087c98
* Renamed Encoder -> VideoEncoder.kxing@chromium.org2012-08-2411-123/+136
| | | | | | Review URL: https://chromiumcodereview.appspot.com/10870071 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@153261 0039d316-1c4b-4281-b951-d872f2087c98
* Renamed Decoder -> VideoDecoder.kxing@chromium.org2012-08-249-85/+90
| | | | | | Review URL: https://chromiumcodereview.appspot.com/10873047 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@153140 0039d316-1c4b-4281-b951-d872f2087c98
* Moved the video encoders/decoders to the codec directory.kxing@chromium.org2012-08-2316-0/+2084
| | | | | | Review URL: https://chromiumcodereview.appspot.com/10877014 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@152912 0039d316-1c4b-4281-b951-d872f2087c98
* Speex encoding/decoding.kxing@chromium.org2012-08-216-1/+331
| | | | | | Review URL: https://chromiumcodereview.appspot.com/10831246 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@152637 0039d316-1c4b-4281-b951-d872f2087c98
* Added more error checking for audio packets.kxing@chromium.org2012-08-213-1/+8
| | | | | | | | | BUG= Review URL: https://chromiumcodereview.appspot.com/10823420 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@152578 0039d316-1c4b-4281-b951-d872f2087c98
* Changed AudioPacket data to a repeated field.kxing@chromium.org2012-08-162-1/+8
| | | | | | | | | BUG= Review URL: https://chromiumcodereview.appspot.com/10827324 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@151825 0039d316-1c4b-4281-b951-d872f2087c98
* Piping for audio decoding.kxing@chromium.org2012-08-045-0/+111
| | | | | | Review URL: https://chromiumcodereview.appspot.com/10843031 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149990 0039d316-1c4b-4281-b951-d872f2087c98
* Piping for audio encoding.kxing@chromium.org2012-08-014-0/+77
Review URL: https://chromiumcodereview.appspot.com/10836017 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149441 0039d316-1c4b-4281-b951-d872f2087c98