summaryrefslogtreecommitdiffstats
path: root/remoting/protocol
Commit message (Collapse)AuthorAgeFilesLines
* Fix stuck keys problem with Chromoting when host disconnects while keys aregarykac@chromium.org2011-08-301-1/+5
| | | | | | | | | | | down. BUG=84544 TEST=manual Review URL: http://codereview.chromium.org/7799014 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@98825 0039d316-1c4b-4281-b951-d872f2087c98
* Trampoline MessageReader::OnMessageDone() to correct thread.lambroslambrou@chromium.org2011-08-245-7/+92
| | | | | | | | | | | | | | | | | | This ensures that all socket operations occur on one thread, even when MessageReader::message_received_callback_ punts its Task parameter onto another thread. Fixes a Host plugin crash on Linux during a remote desktop session. In this case, the problem was caused by EventExecutorLinux::Inject{Key|Mouse}Event() doing the injection and the "done" task on a different thread. BUG=None TEST=Unit-test: MessageReaderTest.UseSocketOnCorrectThread Review URL: http://codereview.chromium.org/7695010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@97974 0039d316-1c4b-4281-b951-d872f2087c98
* Remove DisableOCSP().sergeyu@chromium.org2011-08-231-0/+5
| | | | | | | | | | | Instead of calling DisableOCSP() all SSL socket used in sandbox should be created with reb_checking_enabled flag set to false. BUG=93273 TEST=None Review URL: http://codereview.chromium.org/7685040 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@97918 0039d316-1c4b-4281-b951-d872f2087c98
* Use plugin main thread for network code on the client sidesergeyu@chromium.org2011-08-223-15/+11
| | | | | | | | | BUG=None TEST=Everything still works. Review URL: http://codereview.chromium.org/7655060 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@97706 0039d316-1c4b-4281-b951-d872f2087c98
* Parsers and formatters for jingle messages.sergeyu@chromium.org2011-08-199-14/+780
| | | | | | | | | | BUG=51198 TEST=Unittests Review URL: http://codereview.chromium.org/7677042 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@97535 0039d316-1c4b-4281-b951-d872f2087c98
* Add MessageLoopProxy::currentnduca@chromium.org2011-08-155-8/+8
| | | | | | Review URL: http://codereview.chromium.org/7583053 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@96819 0039d316-1c4b-4281-b951-d872f2087c98
* Use MessageLoopProxy for network message loop.sergeyu@chromium.org2011-08-1241-115/+197
| | | | | | | | | | | | | When we switch to P2P Pepper API we will need to run networking code on the main plugin thread. Switching to MessageLoopProxy for network thread, so that it's easier to switch network thread in the future. BUG=None TEST=None Review URL: http://codereview.chromium.org/7633009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@96651 0039d316-1c4b-4281-b951-d872f2087c98
* Remove key exchange code from JingleSession.sergeyu@chromium.org2011-08-116-112/+6
| | | | | | | | | BUG=None TEST=None Review URL: http://codereview.chromium.org/7616017 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@96451 0039d316-1c4b-4281-b951-d872f2087c98
* Simplify channel creation callbacks in remoting::Session interface.sergeyu@chromium.org2011-08-1115-64/+46
| | | | | | | | | | | | | Remove name parameters from StreamChannelCallback and DatagramChannelCallback. This argument is redundant: callback currying can be used instead where neccessary. BUG=None TEST=compiles. Review URL: http://codereview.chromium.org/7605018 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@96310 0039d316-1c4b-4281-b951-d872f2087c98
* Remove video_channel() from Session interfacesergeyu@chromium.org2011-08-1124-230/+594
| | | | | | | | | | | BUG=None TEST=Unittests. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=96089 Review URL: http://codereview.chromium.org/7508044 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@96301 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 96089 - Remove video_channel() from Session interfacesergeyu@chromium.org2011-08-1024-481/+177
| | | | | | | | | | | | BUG=None TEST=Unittests. Review URL: http://codereview.chromium.org/7508044 TBR=sergeyu@chromium.org Review URL: http://codereview.chromium.org/7604021 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@96106 0039d316-1c4b-4281-b951-d872f2087c98
* Remove video_channel() from Session interfacesergeyu@chromium.org2011-08-0924-177/+481
| | | | | | | | | BUG=None TEST=Unittests. Review URL: http://codereview.chromium.org/7508044 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@96089 0039d316-1c4b-4281-b951-d872f2087c98
* Resolve addresses of STUN/Relay servers in JingleInfoRequest.sergeyu@chromium.org2011-08-094-2/+19
| | | | | | | | | BUG=83242 TEST=NAT traversal works Review URL: http://codereview.chromium.org/7598004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@96060 0039d316-1c4b-4281-b951-d872f2087c98
* Use HMAC SHA-256, since SHA-1 won't work inside the Windows sandbox.wez@chromium.org2011-08-093-8/+8
| | | | | | | | | BUG=91878 TEST=Can connect remoting Client on Windows, with sandbox active, to a Host. Review URL: http://codereview.chromium.org/7528015 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@95946 0039d316-1c4b-4281-b951-d872f2087c98
* Set send buffer size for the raw socket used by chromotinghclam@chromium.org2011-08-041-0/+10
| | | | | | | | | | | | | | | On Windows the send buffer size of UDP socket has a much smaller value than the PseudoTcp's send buffer size. This causes PseudoTcp to flood the UDP socket and results in a EWOULDBLOCK, however the code doesn't handle this properly but simply treat the packet as lost. This happens very often and causes Windows host to be very janky. BUG=91495 TEST=None Review URL: http://codereview.chromium.org/7508015 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@95473 0039d316-1c4b-4281-b951-d872f2087c98
* Rename PepperP2PChannel to PepperTransportSocketAdaptor. Implement ↵sergeyu@chromium.org2011-08-034-246/+368
| | | | | | | | | | | net::SocketStream interface. BUG=51198 TEST=None Review URL: http://codereview.chromium.org/7549027 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@95325 0039d316-1c4b-4281-b951-d872f2087c98
* Disable RTP support in remoting protocol code.sergeyu@chromium.org2011-08-034-32/+10
| | | | | | | | | | | Also disabled Zip encoder/decoder. TEST=Compiles,connects BUG=91538 Review URL: http://codereview.chromium.org/7563010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@95286 0039d316-1c4b-4281-b951-d872f2087c98
* Respect NAT traversal policy in the Chromoting client plugin.ajwong@chromium.org2011-08-031-3/+3
| | | | | | | | | BUG=90213 TEST=manual Review URL: http://codereview.chromium.org/7453066 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@95185 0039d316-1c4b-4281-b951-d872f2087c98
* Add missing virtual qualifiers to appease clang.wez@chromium.org2011-08-021-4/+4
| | | | | | | | | BUG= TEST= Review URL: http://codereview.chromium.org/7553004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@95039 0039d316-1c4b-4281-b951-d872f2087c98
* Separate channel authentication code from JingleStreamConnecter.sergeyu@chromium.org2011-08-024-167/+337
| | | | | | | | | | | | The new ChannelAuthenticator will also be used for pepper-based protocol implementation. BUG=51198 TEST=Unittests Review URL: http://codereview.chromium.org/7537036 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@95030 0039d316-1c4b-4281-b951-d872f2087c98
* Increase PseudoTcp receive and send window sizeshclam@chromium.org2011-07-301-0/+7
| | | | | | | | | | | | Increase the sizes of these two values so that we can cope with high latency network. BUG=91011 TEST=None Review URL: http://codereview.chromium.org/7539006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@94850 0039d316-1c4b-4281-b951-d872f2087c98
* Add WARN_UNUSED_RESULT to crypto/hmac.hrsleevi@chromium.org2011-07-302-20/+23
| | | | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/7522014 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@94826 0039d316-1c4b-4281-b951-d872f2087c98
* Fix a flake in ConnectBadChannelAuth.wez@chromium.org2011-07-291-6/+12
| | | | | | | | | BUG=88130,90961 TEST=remoting_unittests don't fail under ThreadSanitizer. Review URL: http://codereview.chromium.org/7532036 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@94778 0039d316-1c4b-4281-b951-d872f2087c98
* Re-enable use of TLS False Start on stream channels.wez@chromium.org2011-07-291-5/+0
| | | | | | | | | | | Requires r94105 to work. BUG= TEST=remoting_unittests Review URL: http://codereview.chromium.org/7493071 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@94751 0039d316-1c4b-4281-b951-d872f2087c98
* Add an authentication step to stream channel setup in JingleStreamConnector.wez@chromium.org2011-07-2911-62/+327
| | | | | | | | | | | | | Fix JingleStreamConnector and JingleDatagramConnector tear-down semantics in case of failure. BUG=88130,90624 TEST= Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=94247 Review URL: http://codereview.chromium.org/7501007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@94610 0039d316-1c4b-4281-b951-d872f2087c98
* Reenable JingleSessionTest.TestSpeed.sergeyu@chromium.org2011-07-281-1/+1
| | | | | | | | | BUG=None TEST=None Review URL: http://codereview.chromium.org/7300014 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@94540 0039d316-1c4b-4281-b951-d872f2087c98
* Asynchronous initialization in SessionManager.sergeyu@chromium.org2011-07-287-84/+112
| | | | | | | | | BUG=90213 TEST=NAT traversal works reliably. Review URL: http://codereview.chromium.org/7461087 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@94491 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 94247 - Add an authentication step to stream channel setup in ↵wez@chromium.org2011-07-2711-327/+62
| | | | | | | | | | | | | | | | JingleStreamConnector. Fix JingleStreamConnector and JingleDatagramConnector tear-down semantics in case of failure. BUG=88130,90624 TEST= Review URL: http://codereview.chromium.org/7501007 TBR=wez@chromium.org Review URL: http://codereview.chromium.org/7497026 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@94249 0039d316-1c4b-4281-b951-d872f2087c98
* Add an authentication step to stream channel setup in JingleStreamConnector.wez@chromium.org2011-07-2711-62/+327
| | | | | | | | | | | Fix JingleStreamConnector and JingleDatagramConnector tear-down semantics in case of failure. BUG=88130,90624 TEST= Review URL: http://codereview.chromium.org/7501007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@94247 0039d316-1c4b-4281-b951-d872f2087c98
* Cleanups and bugfixes for JingleInfoRequest.sergeyu@chromium.org2011-07-262-20/+13
| | | | | | | | | BUG=None TEST=JinlgeInfoRequest can fetch jingle info on both host and client Review URL: http://codereview.chromium.org/7495041 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@94149 0039d316-1c4b-4281-b951-d872f2087c98
* Expose nat_traversal flag to host and client plugins.sergeyu@chromium.org2011-07-2610-42/+45
| | | | | | | | | BUG=90213 TEST=None Review URL: http://codereview.chromium.org/7471054 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@94119 0039d316-1c4b-4281-b951-d872f2087c98
* Remove explicit keyword from multi-argument (w/o default values) constructorsdilmah@chromium.org2011-07-261-4/+4
| | | | | | | | | BUG=None TEST=None Review URL: http://codereview.chromium.org/7477008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@94115 0039d316-1c4b-4281-b951-d872f2087c98
* Fix instability in SSL client/server socketssergeyu@chromium.org2011-07-211-6/+3
| | | | | | | | | | | | | | | | | | When writing to a socket, net::StreamSocket::Write() may return immediately, but it may not write the whole buffer passed to it. In this case SSL sockets were not trying to call Write() again to send rest of the data, until client tries to call Write() next time. If client doesn't call Write the remaining data is never sent to the transport socket. remoting_unittests was flaky due to this bug. BUG=88726 TEST=updated net_unittest to catch the bug, remoting_unittests is not flaky Review URL: http://codereview.chromium.org/7399025 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@93445 0039d316-1c4b-4281-b951-d872f2087c98
* Added an OriginBoundCertStore field to the SSLClientSocketNSS class andrkn@chromium.org2011-07-211-1/+3
| | | | | | | | | | | | did the plumbing to pass this field through the layers. In addition, this CL groups several fields together into a single struct |SSLClientSocket::Context|. BUG=None TEST=None Review URL: http://codereview.chromium.org/7315009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@93416 0039d316-1c4b-4281-b951-d872f2087c98
* Message validation in message dispatchers.sergeyu@chromium.org2011-07-219-50/+26
| | | | | | | | | BUG=70335 TEST=None Review URL: http://codereview.chromium.org/7466004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@93296 0039d316-1c4b-4281-b951-d872f2087c98
* Don't use X509Certificate in SSLConfig.sergeyu@chromium.org2011-07-201-3/+2
| | | | | | | | | | | | | X509Certificate class depends in OS-dependant APIs and hense cannot be created inside of sandbox. This change allows specifying allow_bed_certs when running inside of sandbox. BUG=80587 TEST=Unittests Review URL: http://codereview.chromium.org/7401003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@93153 0039d316-1c4b-4281-b951-d872f2087c98
* some cleanup for base/stl_utildilmah@chromium.org2011-07-193-3/+3
| | | | | | | | | | | | | removed unused/irrelevant functions removed irrelevant comments Moved stl_util-inl.h => stl_util.h BUG=None TEST=None Review URL: http://codereview.chromium.org/7342047 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@93110 0039d316-1c4b-4281-b951-d872f2087c98
* Don't use X509Certificate in chromoting code.sergeyu@chromium.org2011-07-1614-85/+59
| | | | | | | | | | | | X509Certificate currently is not usable in sandbox on Mac. Don't use it in chromoting code. BUG=80587 TEST=Unittests Review URL: http://codereview.chromium.org/7401005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@92772 0039d316-1c4b-4281-b951-d872f2087c98
* Move channel initialization logic to JingleChannel class.sergeyu@chromium.org2011-07-1511-344/+667
| | | | | | | | | | | | | Added new CreateChannel() method in Session interface. The new method can be used to create arbitrary channels. BUG=None TEST=Unittests Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=92641 Review URL: http://codereview.chromium.org/7326024 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@92712 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 92641 because it doesn't compile - Move channel initialization logic ↵sadrul@chromium.org2011-07-1511-667/+344
| | | | | | | | | | | | | | | | to JingleChannel class. Added new CreateChannel() method in Session interface. The new method can be used to create arbitrary channels. BUG=None TEST=Unittests Review URL: http://codereview.chromium.org/7326024 TBR=sergeyu@chromium.org Review URL: http://codereview.chromium.org/7385002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@92642 0039d316-1c4b-4281-b951-d872f2087c98
* Move channel initialization logic to JingleChannel class.sergeyu@chromium.org2011-07-1511-344/+667
| | | | | | | | | | | Added new CreateChannel() method in Session interface. The new method can be used to create arbitrary channels. BUG=None TEST=Unittests Review URL: http://codereview.chromium.org/7326024 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@92641 0039d316-1c4b-4281-b951-d872f2087c98
* Rename CommandLine::GetArgs(), update callers.msw@chromium.org2011-07-131-1/+1
| | | | | | | | | 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
* Disable TCP connections for chromotinghclam@chromium.org2011-07-131-2/+8
| | | | | | | | | | | | | | | Chromoting always runs PseudoTcp on top of jingle connections. Sometimes this jingle connection is a TCP connection and it destroys performance completely. Until we can bypass PseudoTcp when TCP connection is established and understand the connection establishing algorithm we only UDP for now, which is known to work best. BUG=None TEST=None Review URL: http://codereview.chromium.org/7344002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@92407 0039d316-1c4b-4281-b951-d872f2087c98
* Roll new libjingle with asynchronous NetworkManager.sergeyu@chromium.org2011-07-131-1/+1
| | | | | | | | | BUG=84243 TEST=Unittests Review URL: http://codereview.chromium.org/7338003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@92384 0039d316-1c4b-4281-b951-d872f2087c98
* Remove SocketWrapper, which should no longer be needed since we're ↵wez@chromium.org2011-07-084-152/+20
| | | | | | | | | | | | explicitly telling writers not to touch channels in CloseInternal(). BUG= TEST=Unit-tests should pass. End-to-end remoting sessions should not make calls to deleted StreamSocket instances for channels during shutdown. Review URL: http://codereview.chromium.org/7289032 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@91874 0039d316-1c4b-4281-b951-d872f2087c98
* Mark failing channel tests flaky, for investigation ASAP.wez@chromium.org2011-07-081-3/+6
| | | | | | | | | | BUG=88726 TEST=Test<foo>Channel tests do not cause the tree to close. Review URL: http://codereview.chromium.org/7324015 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@91802 0039d316-1c4b-4281-b951-d872f2087c98
* Minor cleanups in JingleSession and JingleSessionManager.sergeyu@chromium.org2011-07-078-71/+66
| | | | | | | | | BUG=None TEST=None Review URL: http://codereview.chromium.org/7312013 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@91751 0039d316-1c4b-4281-b951-d872f2087c98
* Fix Valgrind errors in remoting_unittests.sergeyu@chromium.org2011-07-074-20/+22
| | | | | | | | | | | | | - FakeSession was used after it is destroyed. - JingleThreadMessageLoop::Quit() was calling Thread::Stop() before all tasks are finished. - JingleSessionManager::AcceptConnection() would destroy JingleSession when called by that JingleSession. BUG=None TEST=Valgrind is happy. Review URL: http://codereview.chromium.org/7227017 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@91652 0039d316-1c4b-4281-b951-d872f2087c98
* Comment another failing DCHECK.sergeyu@chromium.org2011-07-071-3/+4
| | | | | | | | | BUG=88600 TEST=Chromoting Host works. Review URL: http://codereview.chromium.org/7275039 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@91648 0039d316-1c4b-4281-b951-d872f2087c98
* Comment DCHECK that fails on the host side.sergeyu@chromium.org2011-07-061-1/+4
| | | | | | | | | BUG=88600 TEST=Host doesn't crash Review URL: http://codereview.chromium.org/7319001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@91629 0039d316-1c4b-4281-b951-d872f2087c98
generated by cgit v1.1 at 2025-03-30 00:55:17 +0000