summaryrefslogtreecommitdiffstats
path: root/remoting
Commit message (Collapse)AuthorAgeFilesLines
* Simplified frame rate control in the chromoting host.sergeyu@chromium.org2010-12-0917-249/+217
| | | | | | | | | | | | | | | | Insted of keeping semi-fixed frame rate, now capturing rate is controlled by how fast we can send data to the client. Capturing of frame n is started only after frame n-2 is sent (while n-1 is being encoded). This guarantees that we don't clog the video channel buffers, and that we start capturing only if we know that the frame will not need to wait for too long in the buffer. TEST=None BUG=None Review URL: http://codereview.chromium.org/5634002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68688 0039d316-1c4b-4281-b951-d872f2087c98
* Remove ChromotingClientMessagegarykac@chromium.org2010-12-087-27/+21
| | | | | | | | | BUG=None TEST=Remoting unittests Review URL: http://codereview.chromium.org/5559011 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68627 0039d316-1c4b-4281-b951-d872f2087c98
* Add the "virtual" keyword on method overrides that are missing it.erg@google.com2010-12-083-4/+4
| | | | | | | | | BUG=none TEST=compiles Review URL: http://codereview.chromium.org/5648004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68606 0039d316-1c4b-4281-b951-d872f2087c98
* Move more code from headers to implementation.erg@google.com2010-12-074-20/+27
| | | | | | | | | BUG=none TEST=compiles Review URL: http://codereview.chromium.org/5624002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68534 0039d316-1c4b-4281-b951-d872f2087c98
* Reenable encoder/decoder tests.sergeyu@chromium.org2010-12-077-146/+81
| | | | | | | | | BUG=57351 TEST=Unittests Review URL: http://codereview.chromium.org/5635003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68504 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 68434 - Reenable encoder/decoder tests.scherkus@chromium.org2010-12-077-75/+146
| | | | | | | | | | | | BUG=57351 TEST=Unittests Review URL: http://codereview.chromium.org/5635003 TBR=sergeyu@chromium.org Review URL: http://codereview.chromium.org/5581007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68437 0039d316-1c4b-4281-b951-d872f2087c98
* Reenable encoder/decoder tests.sergeyu@chromium.org2010-12-077-146/+75
| | | | | | | | | BUG=57351 TEST=Unittests Review URL: http://codereview.chromium.org/5635003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68434 0039d316-1c4b-4281-b951-d872f2087c98
* Removed stream_writer.[cc|h]sergeyu@chromium.org2010-12-076-111/+0
| | | | | | | | | | | StreamWriter classes are not used anymore. Removing them. BUG=None TEST=None Review URL: http://codereview.chromium.org/5575005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68433 0039d316-1c4b-4281-b951-d872f2087c98
* Rename SessionManager to ScreenRecorderhclam@chromium.org2010-12-066-103/+96
| | | | | | | | | BUG=None TEST=None Review URL: http://codereview.chromium.org/5118002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68396 0039d316-1c4b-4281-b951-d872f2087c98
* Refactor ZLib and Verbatim encoders.sergeyu@chromium.org2010-12-0616-273/+205
| | | | | | | | | | | Added EncoderRowBased that implements encoding for both ZLib and Verbatim. BUG=None TEST=None Review URL: http://codereview.chromium.org/5382008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68394 0039d316-1c4b-4281-b951-d872f2087c98
* Fix RGB->YUV conversion: input is BGR instead of RGB.sergeyu@chromium.org2010-12-061-6/+6
| | | | | | | | | TEST=Colors are not screwed up BUG=None Review URL: http://codereview.chromium.org/5510003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68388 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 68384: Landed with incorrect commit message.sergeyu@chromium.org2010-12-061-6/+6
| | | | | | | | | | | | | | | | Refactor ZLib and Verbatim encoders. Added EncoderRowBased that implements encoding for both ZLib and Verbatim. BUG=None TEST=None Review URL: http://codereview.chromium.org/5382008 TBR=sergeyu@chromium.org Review URL: http://codereview.chromium.org/5586006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68385 0039d316-1c4b-4281-b951-d872f2087c98
* Refactor ZLib and Verbatim encoders.sergeyu@chromium.org2010-12-061-6/+6
| | | | | | | | | | | Added EncoderRowBased that implements encoding for both ZLib and Verbatim. BUG=None TEST=None Review URL: http://codereview.chromium.org/5382008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68384 0039d316-1c4b-4281-b951-d872f2087c98
* Fix crash when shutting down xmpp connection.sergeyu@chromium.org2010-12-031-0/+3
| | | | | | | | | BUG=None TEST=Client doesn't crash when shuttind down connection. Review URL: http://codereview.chromium.org/5604002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68113 0039d316-1c4b-4281-b951-d872f2087c98
* Move move classes to the remoting::protocol namespace. Minor cleanups.sergeyu@chromium.org2010-12-0226-64/+88
| | | | | | | | | BUG=None TEST=compiles, unittests Review URL: http://codereview.chromium.org/5068001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@68056 0039d316-1c4b-4281-b951-d872f2087c98
* Reverse rows in VP8 decoder.sergeyu@chromium.org2010-12-022-5/+14
| | | | | | | | | TEST=None BUG=None Review URL: http://codereview.chromium.org/5310011 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@67918 0039d316-1c4b-4281-b951-d872f2087c98
* Fix threading problem caught by DCHECKs in ChromotingHosthclam@chromium.org2010-11-302-10/+31
| | | | | | | | | | | Post tasks to the right message loop. BUG=None TEST=None Review URL: http://codereview.chromium.org/5096009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@67769 0039d316-1c4b-4281-b951-d872f2087c98
* Properly parse multiple channel configs.sergeyu@chromium.org2010-11-301-3/+3
| | | | | | | | | BUG=None TEST=None Review URL: http://codereview.chromium.org/5319009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@67766 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 67662 - FBTF: Remove unneeded headers from base/ (part 10)thestig@chromium.org2010-11-308-11/+5
| | | | | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/5139006 TBR=thestig@chromium.org Review URL: http://codereview.chromium.org/5270010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@67674 0039d316-1c4b-4281-b951-d872f2087c98
* FBTF: Remove unneeded headers from base/ (part 10)thestig@chromium.org2010-11-308-5/+11
| | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/5139006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@67662 0039d316-1c4b-4281-b951-d872f2087c98
* Fix 2 crashers in remoting client.sergeyu@chromium.org2010-11-302-16/+25
| | | | | | | | | | | | | There are two issues: 1) VideoReader::Create() was returning null for VP8 over TCP. 2) InputHandler was trying to send events before InputStub is created. BUG=None TEST=remoting client works Review URL: http://codereview.chromium.org/5327002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@67638 0039d316-1c4b-4281-b951-d872f2087c98
* Hook up client events to new protocol stubs.sergeyu@chromium.org2010-11-2920-325/+234
| | | | | | | | | | | | Patch by garykac@chromium.com. Original review: http://codereview.chromium.org/5062001 BUG=none TEST=none Review URL: http://codereview.chromium.org/5345007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@67594 0039d316-1c4b-4281-b951-d872f2087c98
* Begin adding mutual authentication into the SessionManager::connect() call.ajwong@chromium.org2010-11-2419-67/+239
| | | | | | | | | | | | This CL mainly changes APIs and stubs out functionality needed to actually create the correct auth token stanzas. BUG=None TEST=compiles. Review URL: http://codereview.chromium.org/4941001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@67316 0039d316-1c4b-4281-b951-d872f2087c98
* Build fix for Clang. const type qualifier on return type has no effect.hans@chromium.org2010-11-241-1/+1
| | | | | | | | | | BUG=none TEST=build goes green TBR=thakis Review URL: http://codereview.chromium.org/5254004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@67243 0039d316-1c4b-4281-b951-d872f2087c98
* Implemented RtcpWriter for sending RTCP Receiver Reports.sergeyu@chromium.org2010-11-2412-17/+298
| | | | | | | | | BUG=None TEST=Unittests Review URL: http://codereview.chromium.org/5122008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@67215 0039d316-1c4b-4281-b951-d872f2087c98
* Refactoring in RTP reader code: sequence number wrapping logic moved tosergeyu@chromium.org2010-11-249-35/+114
| | | | | | | | | | | RtpReader BUG=None TEST=Unittests Review URL: http://codereview.chromium.org/5110008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@67212 0039d316-1c4b-4281-b951-d872f2087c98
* Use VP8 over PseudoTCP by default.sergeyu@chromium.org2010-11-2317-122/+204
| | | | | | | | | | | Added option in simple_host to choose video codec. Removed ugly ifdefs from chromoting_host.cc. BUG=None TEST=Unittests Review URL: http://codereview.chromium.org/5298001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@67164 0039d316-1c4b-4281-b951-d872f2087c98
* Fix ThreadChecker to use Locks and not use scoped_ptr.willchan@chromium.org2010-11-211-0/+2
| | | | | | | | | | | | | | | It needs to synchronize its checks, since in order to assert correctly, it needs to make sure the thread id is synchronized on all threads. It doesn't need scoped_ptr. It was trying to use NULL to catch invalid thread ids. 0 is already assumed to be invalid (see base::Thread's use). Eliminating scoped_ptr fixes a valgrind/heapcheck issue where they don't follow LazyInstance objects' member pointers. So they think the ThreadChecker's member variable is leaked, even though the global object still has a pointer to it. Removing the scoped_ptr.h caused a bunch of other lame files to fail to compile. I had to fix those places. #include what you use please :( TBR=levin (I want to green the memory bots) BUG=none TEST=none Review URL: http://codereview.chromium.org/5180006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@66915 0039d316-1c4b-4281-b951-d872f2087c98
* Use VideoStub interface on the host side.sergeyu@chromium.org2010-11-1916-85/+89
| | | | | | | | | BUG=None TEST=None Review URL: http://codereview.chromium.org/5232004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@66841 0039d316-1c4b-4281-b951-d872f2087c98
* Fix up mac capturerdmaclach@chromium.org2010-11-192-1/+2
| | | | | | | | | | | Make it so that it sets the pixel type correctly, and initializes the video buffers properly. BUG=NONE TEST=BUILD Review URL: http://codereview.chromium.org/5159004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@66834 0039d316-1c4b-4281-b951-d872f2087c98
* Fix GDI capturer initialization.sergeyu@chromium.org2010-11-191-0/+1
| | | | | | | | | BUG=None TEST=GDI capturer works. Review URL: http://codereview.chromium.org/5173007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@66802 0039d316-1c4b-4281-b951-d872f2087c98
* Remove capture thread in Chromoting hosthclam@chromium.org2010-11-185-25/+17
| | | | | | | | | | Move operations from capture thread to main thread. Also move operations in SessionManager on main thread to network thread. BUG=None TEST=None Review URL: http://codereview.chromium.org/5123002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@66696 0039d316-1c4b-4281-b951-d872f2087c98
* Use CompoundBuffer in MessageDecoder.sergeyu@chromium.org2010-11-185-79/+106
| | | | | | | | | TEST=Unittests BUG=None Review URL: http://codereview.chromium.org/5067001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@66574 0039d316-1c4b-4281-b951-d872f2087c98
* Unittests for RTP packetizer and depacketizer.sergeyu@chromium.org2010-11-178-37/+690
| | | | | | | | | BUG=None TEST=Unittests Review URL: http://codereview.chromium.org/4946001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@66536 0039d316-1c4b-4281-b951-d872f2087c98
* Move creation of capturer, input stub into ChromotingHosthclam@chromium.org2010-11-173-49/+57
| | | | | | | | | BUG=None TEST=None Review URL: http://codereview.chromium.org/5065001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@66390 0039d316-1c4b-4281-b951-d872f2087c98
* Introduce a fake HostStub since there's no need to handle it nowhclam@chromium.org2010-11-175-4/+60
| | | | | | | | | | | Also changed ChormotingHost to use it. BUG=None TEST=None Review URL: http://codereview.chromium.org/4975003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@66389 0039d316-1c4b-4281-b951-d872f2087c98
* Inject MessageLoop into Capturerhclam@chromium.org2010-11-1715-23/+37
| | | | | | | | | | | Simple patch to juse inject the message loop. BUG=None TEST=None Review URL: http://codereview.chromium.org/4971003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@66387 0039d316-1c4b-4281-b951-d872f2087c98
* Implement input stub in the host side for chromotinghclam@chromium.org2010-11-1624-246/+292
| | | | | | | | | | | Implement InputStub for the host. BUG=None TEST=None Review URL: http://codereview.chromium.org/4726003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@66314 0039d316-1c4b-4281-b951-d872f2087c98
* Fixed checkdeps error.sergeyu@chromium.org2010-11-161-0/+1
| | | | | | | | BUG=None TEST=checkdeps is happy TBR=ajwong git-svn-id: svn://svn.chromium.org/chrome/trunk/src@66215 0039d316-1c4b-4281-b951-d872f2087c98
* Packetizer/Depacketizer for RTP.sergeyu@chromium.org2010-11-169-70/+437
| | | | | | | | | BUG=None TEST=None Review URL: http://codereview.chromium.org/4925001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@66213 0039d316-1c4b-4281-b951-d872f2087c98
* Added CompoundBuffer that will be used to store data in the encoding/decodingsergeyu@chromium.org2010-11-1621-294/+666
| | | | | | | | | | | pipeline. BUG=None TEST=Unittests Review URL: http://codereview.chromium.org/4779001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@66209 0039d316-1c4b-4281-b951-d872f2087c98
* Reset decompressor after each frame in DecoderRowBased::DecodePacket().sergeyu@chromium.org2010-11-161-6/+11
| | | | | | | | | BUG=None TEST=None Review URL: http://codereview.chromium.org/4953001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@66201 0039d316-1c4b-4281-b951-d872f2087c98
* Correctly handle SSL Client Authentication requests when connectingrch@chromium.org2010-11-121-1/+2
| | | | | | | | | | | to an HTTPS/SPDY proxy. Modify SSLClientSocket classes to correctly set the host_and_port field of the cert_request_info. Modify HttpNetworkTransaction to use this field when populating the SSL client auth cache. BUG=59292 TEST=HttpProxyClientSocketPoolTest.SslClientAuth Review URL: http://codereview.chromium.org/4339001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@65976 0039d316-1c4b-4281-b951-d872f2087c98
* Handle error in BufferedStreamWriter and friendshclam@chromium.org2010-11-112-2/+29
| | | | | | | | | | | Sets the closed flag to true when an error is encountered in the socket. BUG=None TEST=None Review URL: http://codereview.chromium.org/4705002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@65859 0039d316-1c4b-4281-b951-d872f2087c98
* Reapply of r65413. Add linux executor support for Chromoting host.ajwong@chromium.org2010-11-114-4/+405
| | | | | | | | | | | Original CL: http://codereview.chromium.org/4058002 See the Original CL and revision for a full description. All bots have been reverified to have the XTest dependency added. BUG=None TEST=None git-svn-id: svn://svn.chromium.org/chrome/trunk/src@65852 0039d316-1c4b-4281-b951-d872f2087c98
* Fix Windows Chromoting host by setting pixel format.garykac@google.com2010-11-111-0/+1
| | | | | | | | | BUG=none TEST=build/test windows connection Review URL: http://codereview.chromium.org/4774001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@65752 0039d316-1c4b-4281-b951-d872f2087c98
* Revert "Reapply of r65413. XTest dependencies added to _chroots_ on linux bots."ajwong@chromium.org2010-11-104-405/+4
| | | | | | TBR=jcivelli git-svn-id: svn://svn.chromium.org/chrome/trunk/src@65712 0039d316-1c4b-4281-b951-d872f2087c98
* Reapply of r65413. XTest dependencies added to _chroots_ on linux bots.ajwong@chromium.org2010-11-104-4/+405
| | | | | | | | | | | Original CL: http://codereview.chromium.org/4058002 BUG=None TEST=Was already passing try bots, etc. Now we're checking waterfall buildbot configs. Review URL: http://codereview.chromium.org/4772001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@65709 0039d316-1c4b-4281-b951-d872f2087c98
* Add VideoPacket struct for video packets. Refactor Decode interface to use it.sergeyu@chromium.org2010-11-0942-363/+332
| | | | | | | | | | Various cleanups. BUG=None TEST=Unittests. Review URL: http://codereview.chromium.org/4476003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@65590 0039d316-1c4b-4281-b951-d872f2087c98
* Fix build breakhclam@chromium.org2010-11-092-3/+1
| | | | | | | | | | | | Fix gyp merge issue. TBR=ajwong BUG=None TEST=None Review URL: http://codereview.chromium.org/4704004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@65474 0039d316-1c4b-4281-b951-d872f2087c98