summaryrefslogtreecommitdiffstats
path: root/remoting/jingle_glue
Commit message (Collapse)AuthorAgeFilesLines
* Roll libjingle r313.ronghuawu@chromium.org2013-05-072-5/+5
| | | | | | | | | | | Reviewed and tried here: https://chromiumcodereview.appspot.com/15021002/ TBR=sergeyu@chromium.org Review URL: https://codereview.chromium.org/14781007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@198614 0039d316-1c4b-4281-b951-d872f2087c98
* remoting: Use base::MessageLoop.xhwang@chromium.org2013-04-302-3/+3
| | | | | | | | | BUG=236029 R=alexeypa@chromium.org Review URL: https://chromiumcodereview.appspot.com/14314026 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@197247 0039d316-1c4b-4281-b951-d872f2087c98
* remoting: Remove use of ALLOW_THIS_IN_INITIALIZER_LIST.scherkus@chromium.org2013-04-271-1/+1
| | | | | | | | | | It's no longer providing value as the MSVC warning is disabled during compilation. Refer to bug for details. BUG=234765 Review URL: https://chromiumcodereview.appspot.com/14522010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@196888 0039d316-1c4b-4281-b951-d872f2087c98
* Rewrite std::string("") to std::string(), Linux edition.dcheng@chromium.org2013-04-092-13/+14
| | | | | | | | | | | | | | | | | | | This patch was generated by running the empty_string clang tool across the Chromium Linux compilation database. Implicitly or explicitly constructing std::string() with a "" argument is inefficient as the caller needs to emit extra instructions to pass an argument, and the constructor needlessly copies a byte into internal storage. Rewriting these instances to simply call the default constructor appears to save ~14-18 kilobytes on an optimized release build. BUG=none Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=193020 Review URL: https://codereview.chromium.org/13145003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@193040 0039d316-1c4b-4281-b951-d872f2087c98
* Revert "Rewrite std::string("") to std::string(), Linux edition."dcheng@chromium.org2013-04-092-14/+13
| | | | | | | | | | | | | | This reverts commit e59558b78e8c6a1b0bd916a724724b638c3c91b6. Revert "Fix build after r193020." This reverts commit 558a35897f6b3ffbcaefde927c1f150b815d140a. Revert "Really fix build after r193020." This reverts commit e3748a79b523a8d365d4a33ef986eebb4186fa78. git-svn-id: svn://svn.chromium.org/chrome/trunk/src@193030 0039d316-1c4b-4281-b951-d872f2087c98
* Rewrite std::string("") to std::string(), Linux edition.dcheng@chromium.org2013-04-092-13/+14
| | | | | | | | | | | | | | | | | This patch was generated by running the empty_string clang tool across the Chromium Linux compilation database. Implicitly or explicitly constructing std::string() with a "" argument is inefficient as the caller needs to emit extra instructions to pass an argument, and the constructor needlessly copies a byte into internal storage. Rewriting these instances to simply call the default constructor appears to save ~14-18 kilobytes on an optimized release build. BUG=none Review URL: https://codereview.chromium.org/13145003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@193020 0039d316-1c4b-4281-b951-d872f2087c98
* Linux/ChromeOS Chromium style checker cleanup, remoting/ edition.rsleevi@chromium.org2013-02-061-10/+10
| | | | | | | | | | | Automated clean up of style checker errors that were missed due to the plugin not being executed on implementation files. BUG=115047 Review URL: https://chromiumcodereview.appspot.com/12207034 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@181075 0039d316-1c4b-4281-b951-d872f2087c98
* Make host/url references configurable.rmsousa@chromium.org2013-02-052-12/+28
| | | | | | | | | | | | | | | | | | In C++, they're all passed as optional command line parameters (like google_apis/gaia already allows us to do with the gaia oauth urls). So one can start the host with a monstrosity such as: out/Debug/remoting_me2me_host out/Debug/remoting_start_host --gaia-host=accounts.google.com --oauth-user-info-url=https://www.googleapis.com/oauth2/v1/userinfo --chromoting-bot-jid=remoting@bot.talk.google.com --chromoting-base-url=https://www.googleapis.com/chromoting/v1 --talk-server-host=talk.google.com --talk-server-port=5222 -[-disable-talk-server-tls] --oauth2-token-url=https://accounts.google.com/o/oauth2/token In Javascript and IT2Me, unfortunately we don't have a command line, so instead we put all these settings in plugin_settings.js, and the javascript passes them all into the it2me plugin as appropriate. They can then be changed by manually hacking plugin_settings.js and manifest.json in the build output, changing the remoting.settings object at runtime (although any new hosts you configure that are not in the manifest's CSP won't work), or, preferrably, by passing them via environment variables to the build, and letting build-webapp.py do its magic - for example, like with this: GOOGLE_CLIENT_ID_REMOTING=<clientid> GOOGLE_CLIENT_SECRET_REMOTING=<secret> OAUTH2_ACCOUNTS_HOST=https://accounts.google.com OAUTH2_API_HOST=https://www.googleapis.com CHROMOTING_BOT_JID=remoting@bot.talk.google.com CHROMOTING_API_HOST=https://www.googleapis.com TALK_SERVER_HOST=talk.google.com TALK_SERVER_PORT=5222 TALK_SERVER_USE_TLS=true TALK_GADGET_HOST_SUFFIX=talkgadget.google.com OAUTH2_REDIRECT_HOST_PREFIX=https://chromoting-oauth. TALK_GADGET_HOST_PREFIX=https://chromoting-client. ninja -C out/Debug remoting_webapp I've tested with me2me (host and client) and it2me (share and access), and verified both that the defaults work as before (without command line parameters), and that I can substitute test server hostnames in all of those parameters. BUG= Review URL: https://chromiumcodereview.appspot.com/11346040 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@180622 0039d316-1c4b-4281-b951-d872f2087c98
* remoting: Update calls froms RunAllPending() to RunUntilIdle().tfarina@chromium.org2012-11-011-3/+3
| | | | | | | | | | | BUG=131220 TEST=remoting_unittests R=sergeyu@chromium.org,wez@chromium.org Review URL: https://chromiumcodereview.appspot.com/11364031 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@165528 0039d316-1c4b-4281-b951-d872f2087c98
* Delete some unused code found by -Wunused-functionthakis@chromium.org2012-10-011-8/+0
| | | | | | | | BUG=none,100309 Review URL: https://chromiumcodereview.appspot.com/10977073 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@159592 0039d316-1c4b-4281-b951-d872f2087c98
* Create stub jingle library for iOS.qsr@chromium.org2012-08-231-1/+2
| | | | | | | | | | | | iOS doesn't use libjingle, but needs a stub jingle library for sync to compile. BUG=143886 R=akalin@chromium.org Review URL: https://chromiumcodereview.appspot.com/10829424 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@152975 0039d316-1c4b-4281-b951-d872f2087c98
* Clean-up inline members of nested classes (remoting/)hans@chromium.org2012-08-091-2/+2
| | | | | | | | | | | | | | | | Due to a bug, the Clang-plugin style checker failed to warn about inline constructors, destructors, non-empty virtual methods, etc. for nested classes. The plugin has been fixed, and this patch is part of a clean-up of all the code that now causes the plugin to issue errors. BUG=139346 Review URL: https://chromiumcodereview.appspot.com/10855076 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@150913 0039d316-1c4b-4281-b951-d872f2087c98
* Fix destruction order in XmppSignalStrategy.sergeyu@chromium.org2012-08-021-0/+9
| | | | | | | | | | | | | This fixes a crash that was introduced in crrev.com/148803. Specifically when XMPP authentication fails. It may or may not fix the crash in the linked bug. BUG=140096 Review URL: https://chromiumcodereview.appspot.com/10850008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149662 0039d316-1c4b-4281-b951-d872f2087c98
* Don't use MessageLoop and MessageLoopProxy in remoting/jingle_gluesergeyu@chromium.org2012-07-272-7/+9
| | | | | | | | | | | MessageLoopProxy::current() may not work properly on plugin threads unless the plugin is compiled into the browser. BUG=134216 Review URL: https://chromiumcodereview.appspot.com/10830017 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@148811 0039d316-1c4b-4281-b951-d872f2087c98
* Always use chromium threads for IO in remoting hostsergeyu@chromium.org2012-07-2711-1597/+40
| | | | | | | | | | BUG=137140 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=148418 Review URL: https://chromiumcodereview.appspot.com/10808094 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@148803 0039d316-1c4b-4281-b951-d872f2087c98
* Ingore ERR_ADDRESS_INVALID error returned from sendto().sergeyu@chromium.org2012-07-271-2/+8
| | | | | | | | | | | | This is a speculative fix for the failure in crbug.com/139136. It's not clear why we get WSAEADDRNOTAVAIL from sendto(), but it should be safe to ignore that error. BUG=137140 Review URL: https://chromiumcodereview.appspot.com/10827064 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@148784 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 148418 - Always use chromium threads for IO in remoting hostsergeyu@chromium.org2012-07-2611-40/+1597
| | | | | | | | | | | | BUG=137140 Review URL: https://chromiumcodereview.appspot.com/10808094 TBR=sergeyu@chromium.org Review URL: https://chromiumcodereview.appspot.com/10829040 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@148578 0039d316-1c4b-4281-b951-d872f2087c98
* Always use chromium threads for IO in remoting hostsergeyu@chromium.org2012-07-2511-1597/+40
| | | | | | | | | BUG=137140 Review URL: https://chromiumcodereview.appspot.com/10808094 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@148418 0039d316-1c4b-4281-b951-d872f2087c98
* Implement ChromiumSocketFactory.sergeyu@chromium.org2012-07-233-0/+489
| | | | | | | | | | | | The new PacketSocketFactory will be used by chromoting host in order to be able to use chromium UDP sockets instead of libjingle sockets BUG=137140 Review URL: https://chromiumcodereview.appspot.com/10783028 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@147934 0039d316-1c4b-4281-b951-d872f2087c98
* Change SpdySession::GetSSLInfo to get the SSLInfo from the underlying socketrch@chromium.org2012-07-192-0/+12
| | | | | | | | | | | | | | | | even if the session is not "secure". This required refactoring StreamSocket to add WasNpnNegotiated() and GetSSLInfo() methods. This allows for a change to SpdySession::GetSSLInfo to accurately return the correct SSLInfo in the case of SPDY Proxy sessions. BUG=134690 TEST=\*DoNotUseSpdySessionIfCertDoesNotMatch\* Review URL: https://chromiumcodereview.appspot.com/10690122 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@147479 0039d316-1c4b-4281-b951-d872f2087c98
* Enable HSTS for XMPP connection used by Chromotingsergeyu@chromium.org2012-07-162-2/+6
| | | | | | | | | | | Chromoting host uses XMPP connection encrypted with SSL to connect to talk.google.com. Previously it was accepting any certificate signed by a known CA. Updating SSL adapter code to enable HSTS code, so that only limited set of CA's is accepted for the XMPP connection. BUG=132884 Review URL: https://chromiumcodereview.appspot.com/10787009 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@146876 0039d316-1c4b-4281-b951-d872f2087c98
* Remove SSLHostInfo.agl@chromium.org2012-06-271-1/+1
| | | | | | | BUG=105208 TEST=none git-svn-id: svn://svn.chromium.org/chrome/trunk/src@144468 0039d316-1c4b-4281-b951-d872f2087c98
* Fix SSLSocketAdapter to handle asynchronous writes appropriately.sergeyu@chromium.org2012-06-132-63/+142
| | | | | | | | | | | | Also fixed some other bugs, particularly asynchronous reads were not handled properly in some cases. BUG=129658 Review URL: https://chromiumcodereview.appspot.com/10543069 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@141946 0039d316-1c4b-4281-b951-d872f2087c98
* Update libjingle to 153.ronghuawu@chromium.org2012-06-132-3/+3
| | | | | | | | Remove the overrides for thread and messagequeue. Review URL: https://chromiumcodereview.appspot.com/10534122 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@141908 0039d316-1c4b-4281-b951-d872f2087c98
* fixed issue 128383 - replace GetPeerAddress(AddressList* address) with ↵zhaoqin@chromium.org2012-06-082-5/+3
| | | | | | | | | | | | | GetPeerAddress(IPEndPoint* address) R=szym@chromium.org BUG=128383 TEST=try bot Review URL: https://chromiumcodereview.appspot.com/10491007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@141125 0039d316-1c4b-4281-b951-d872f2087c98
* Changes needed to roll libjingle r141 to chrome.sergeyu@chromium.org2012-05-221-2/+2
| | | | | | | | | 1. Setting ice username and password for P2PTransportChannel. 2. Updating XMPP Auth cookie with Auth Token Review URL: https://chromiumcodereview.appspot.com/10382003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@138334 0039d316-1c4b-4281-b951-d872f2087c98
* Reland 137824 - was reverted by mistakesergeyu@chromium.org2012-05-221-18/+0
| | | | | | | | | | | | | | | | | | | | | | | Properly handle accounts that don't have GMail account. 1. Me2MeHostAuthenticatorFactory now verifies that the bare JID of the remote client matches bare JID of the host. Previously it was comparing it with the user's email, which may be different from JID. 2. GaiaOAuthClient now fetches user's email. 3. SignalingConnector verifies that user's email matches the expected value stored in xmpp_login. If it doesn't, then the auth token was generated for a different account and the host treats it as an authentication error. BUG=128102 Review URL: https://chromiumcodereview.appspot.com/10332187 TBR=dharani@chromium.org Review URL: https://chromiumcodereview.appspot.com/10332285 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@138224 0039d316-1c4b-4281-b951-d872f2087c98
* This broke M20 build.dharani@chromium.org2012-05-221-0/+18
| | | | | | | | | | | | | | | | | | | | | | | Revert 137824 - Properly handle accounts that don't have GMail account. 1. Me2MeHostAuthenticatorFactory now verifies that the bare JID of the remote client matches bare JID of the host. Previously it was comparing it with the user's email, which may be different from JID. 2. GaiaOAuthClient now fetches user's email. 3. SignalingConnector verifies that user's email matches the expected value stored in xmpp_login. If it doesn't, then the auth token was generated for a different account and the host treats it as an authentication error. BUG=128102 Review URL: https://chromiumcodereview.appspot.com/10332187 TBR=sergeyu@chromium.org Review URL: https://chromiumcodereview.appspot.com/10388226 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@138204 0039d316-1c4b-4281-b951-d872f2087c98
* Properly handle accounts that don't have GMail account.sergeyu@chromium.org2012-05-181-18/+0
| | | | | | | | | | | | | | | | | | 1. Me2MeHostAuthenticatorFactory now verifies that the bare JID of the remote client matches bare JID of the host. Previously it was comparing it with the user's email, which may be different from JID. 2. GaiaOAuthClient now fetches user's email. 3. SignalingConnector verifies that user's email matches the expected value stored in xmpp_login. If it doesn't, then the auth token was generated for a different account and the host treats it as an authentication error. BUG=128102 Review URL: https://chromiumcodereview.appspot.com/10332187 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@137824 0039d316-1c4b-4281-b951-d872f2087c98
* Verify that xmpp_login specified in the config matches auth token.sergeyu@chromium.org2012-05-128-4/+75
| | | | | | | | | | | | | Previously the host would ignore the case when the oauth token doesn't match the xmpp_login specified in the config. That would lead to situations when host is connected with one account, but verifies incoming connections with a different account. Fix this by verifying that the JID that we receive from XMPP server matches the value in the config. Review URL: https://chromiumcodereview.appspot.com/10378110 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@136745 0039d316-1c4b-4281-b951-d872f2087c98
* Reimplements net::AddressList without struct addrinfo.szym@chromium.org2012-05-071-16/+7
| | | | | | | | | | | | | | | | net::AddressList extends std::vector<std::IPEndPoint> by canonical name. (Canonical name is planned to be removed as well.) Removes dependency on sys_addrinfo.h throughout the codebase. Introduces net::SockaddrStorage for convenience. BUG=125696 TEST=green waterfall Review URL: http://codereview.chromium.org/10309002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@135731 0039d316-1c4b-4281-b951-d872f2087c98
* Roll libjingle 132:135sergeyu@chromium.org2012-04-261-0/+2
| | | | | | Review URL: https://chromiumcodereview.appspot.com/10209008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@134065 0039d316-1c4b-4281-b951-d872f2087c98
* Handle IQ responses asynchronously.sergeyu@chromium.org2012-04-204-5/+28
| | | | | | | | | | | | | | Previously IqRequest was calling response callback synchronously. The callback is allowed to delete SignalStrategy object, but it may be unsafe to delete XmppSignalStrategy when handling an incoming message. Fixed IqRequest to invoke response callback asynchronously, so that the callback is called with a clean stack. BUG=124430 Review URL: https://chromiumcodereview.appspot.com/10161010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@133302 0039d316-1c4b-4281-b951-d872f2087c98
* Send keep-alive messages for XMPP connections.sergeyu@chromium.org2012-04-202-0/+20
| | | | | | | | BUG=124295 Review URL: https://chromiumcodereview.appspot.com/10134013 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@133275 0039d316-1c4b-4281-b951-d872f2087c98
* Use NSS for SSL encryption of XMPP connection on Windows.sergeyu@chromium.org2012-04-201-5/+1
| | | | | | Review URL: http://codereview.chromium.org/10169012 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@133247 0039d316-1c4b-4281-b951-d872f2087c98
* Objects that derive from RefCounted should not have public dtorsrsleevi@chromium.org2012-04-051-0/+2
| | | | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/10009002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@131026 0039d316-1c4b-4281-b951-d872f2087c98
* Add a new GetNegotiatedProtocol method to StreamSocket and implement in all ↵rch@chromium.org2012-04-032-1/+7
| | | | | | | | | | subclasses. BUG=120890 Review URL: https://chromiumcodereview.appspot.com/9958028 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@130388 0039d316-1c4b-4281-b951-d872f2087c98
* Fix some grammar in comments, error messages and documentation.gavinp@chromium.org2012-04-031-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Thanks to all my reviewers, you are legion. I hope I didn't waste too much of your time. BUG=None Review URL: http://codereview.chromium.org/9854039 Review URL: http://codereview.chromium.org/9854043 Review URL: http://codereview.chromium.org/9863058 Review URL: http://codereview.chromium.org/9863059 Review URL: http://codereview.chromium.org/9887005 Review URL: http://codereview.chromium.org/9890002 Review URL: http://codereview.chromium.org/9891002 Review URL: http://codereview.chromium.org/9895003 Review URL: http://codereview.chromium.org/9896002 Review URL: http://codereview.chromium.org/9896003 Review URL: http://codereview.chromium.org/9897002 Review URL: http://codereview.chromium.org/9897003 Review URL: http://codereview.chromium.org/9903004 Review URL: http://codereview.chromium.org/9904003 Review URL: http://codereview.chromium.org/9904002 Review URL: http://codereview.chromium.org/9904004 Review URL: http://codereview.chromium.org/9906002 Review URL: http://codereview.chromium.org/9906001 Review URL: http://codereview.chromium.org/9906003 Review URL: http://codereview.chromium.org/9909001 Review URL: http://codereview.chromium.org/9909002 Review URL: http://codereview.chromium.org/9909003 Review URL: http://codereview.chromium.org/9909004 Review URL: http://codereview.chromium.org/9910001 Review URL: http://codereview.chromium.org/9910002 Review URL: http://codereview.chromium.org/9910010 Review URL: http://codereview.chromium.org/9911001 Review URL: http://codereview.chromium.org/9912001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@130359 0039d316-1c4b-4281-b951-d872f2087c98
* Make CertVerifier a pure virtual interface.rsleevi@chromium.org2012-03-211-2/+2
| | | | | | | | | | | | | The existing CertVerifier implementation has been renamed to MultiThreadedCertVerifier, consistent with ProxyResolver naming. This is patch 1 of N for http://crbug.com/114343 BUG=114343 TEST=Compiles and existing unittests pass. Review URL: https://chromiumcodereview.appspot.com/9476035 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@128090 0039d316-1c4b-4281-b951-d872f2087c98
* Roll libjingle to r124. Removed overrides for the ws2 fixes.ronghuawu@chromium.org2012-03-151-1/+1
| | | | | | | | | | Roll webrtc to 1888. BUG=115702,117252 TEST=apprtc.appspot.com on windows and linux Review URL: https://chromiumcodereview.appspot.com/9600066 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@126793 0039d316-1c4b-4281-b951-d872f2087c98
* Don't crash in ~HostNPScriptObject() when the object wasn't initialized.sergeyu@chromium.org2012-03-062-5/+7
| | | | | | | | BUG=112656 Review URL: https://chromiumcodereview.appspot.com/9582031 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@125079 0039d316-1c4b-4281-b951-d872f2087c98
* Convert uses of int ms to TimeDelta in jingle and remoting.tedvessenes@gmail.com2012-03-052-3/+3
| | | | | | | | | | R=sergeyu@chromium.org BUG=108171 Review URL: http://codereview.chromium.org/9572038 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@124915 0039d316-1c4b-4281-b951-d872f2087c98
* Include the headers from the libjingle overrides.ronghuawu@chromium.org2012-03-031-2/+2
| | | | | | | | | BUG=116567 TEST=apprtc.appspot.com Review URL: http://codereview.chromium.org/9581015 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@124834 0039d316-1c4b-4281-b951-d872f2087c98
* Fix check_deps broken in 124769sergeyu@chromium.org2012-03-021-1/+1
| | | | | | | | | BUG=115501 TBR=yzshen@chromium.org Review URL: https://chromiumcodereview.appspot.com/9562051 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@124577 0039d316-1c4b-4281-b951-d872f2087c98
* Include override the new override headers in libjingle.sergeyu@chromium.org2012-03-021-2/+2
| | | | | | | | | | BUG=115501 TBR=ronghuawu@chromium.org TEST=Remoting host should work Review URL: https://chromiumcodereview.appspot.com/9564045 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@124569 0039d316-1c4b-4281-b951-d872f2087c98
* Fix JingleSession to properly set initiator field in session.sergeyu@chromium.org2012-02-292-5/+14
| | | | | | | | | | Previously session-initiate messages generated by JingleSession had an empty initiator attribute. Also added tests that verify that initiator field is set. Review URL: https://chromiumcodereview.appspot.com/9477010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@124081 0039d316-1c4b-4281-b951-d872f2087c98
* Implement timeouts for IQ requests.sergeyu@chromium.org2012-02-265-37/+159
| | | | | | | | | | | | Now the IqRequest class supports setting timeouts for each request, and JingleSession uses it to disconnect if no response is receive within 10 secons from a request. BUG=107925 Review URL: http://codereview.chromium.org/9452038 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@123680 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 123635 - Implement timeouts for IQ requests.jamesr@chromium.org2012-02-255-154/+37
| | | | | | | | | | | | | | | | Now the IqRequest class supports setting timeouts for each request, and JingleSession uses it to disconnect if no response is receive within 10 secons from a request. BUG=107925 Review URL: http://codereview.chromium.org/9452038 TBR=sergeyu@chromium.org Review URL: https://chromiumcodereview.appspot.com/9447087 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@123638 0039d316-1c4b-4281-b951-d872f2087c98
* Implement timeouts for IQ requests.sergeyu@chromium.org2012-02-255-37/+154
| | | | | | | | | | | | | Now the IqRequest class supports setting timeouts for each request, and JingleSession uses it to disconnect if no response is receive within 10 secons from a request. BUG=107925 Review URL: http://codereview.chromium.org/9452038 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@123635 0039d316-1c4b-4281-b951-d872f2087c98
* Use scoped_ptr to pass ownership in SignalStrategy.sergeyu@chromium.org2012-02-2312-70/+77
| | | | | | | | | | Updated SignalStrategy and some other code where we pass ownership of XML objects. Review URL: http://codereview.chromium.org/9453001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@123222 0039d316-1c4b-4281-b951-d872f2087c98
generated by cgit v1.1 at 2025-02-16 07:22:46 +0000