summaryrefslogtreecommitdiffstats
path: root/jingle
Commit message (Collapse)AuthorAgeFilesLines
* Added support for the new Google push notifications.sanjeevr@chromium.org2010-09-2511-8/+562
| | | | | | | | | BUG=None TEST=Unit-tests, Cloud print proxy. Review URL: http://codereview.chromium.org/3493008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@60565 0039d316-1c4b-4281-b951-d872f2087c98
* Removed TalkMediator SSL initialization.akalin@chromium.org2010-09-163-68/+4
| | | | | | | | | | | Since we're now using chrome sockets, we don't need this code. BUG=54146 TEST=Unit tests Review URL: http://codereview.chromium.org/3411012 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@59741 0039d316-1c4b-4281-b951-d872f2087c98
* Fixed notifier_unit_tests failures.akalin@chromium.org2010-09-161-1/+13
| | | | | | | | | | | | These were introduced by r59629, but weren't caught since notifier_unit_tests isn't on the waterfall (bug 55799). BUG=None TEST=ran notifier_unit_tests manually Review URL: http://codereview.chromium.org/3416005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@59739 0039d316-1c4b-4281-b951-d872f2087c98
* Refactored handling of XmppClient.akalin@chromium.org2010-09-1625-859/+828
| | | | | | | | | | | | Added new classes WeakXmppClient and XmppConnection, which let me simplify MediatorThreadImpl et al. BUG=55302 TEST=new unit tests, manually Review URL: http://codereview.chromium.org/3290025 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@59629 0039d316-1c4b-4281-b951-d872f2087c98
* Change HostResolver::RequestInfo to take a HostPortPair rather than naked ↵eroman@chromium.org2010-09-151-2/+1
| | | | | | | | host string + port. Review URL: http://codereview.chromium.org/3420001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@59577 0039d316-1c4b-4281-b951-d872f2087c98
* Fixed test broken in Release build.akalin@chromium.org2010-09-131-1/+5
| | | | | | | | | BUG=None TEST=ChromeAsyncSocket.WrongRead now passes Review URL: http://codereview.chromium.org/3323026 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@59261 0039d316-1c4b-4281-b951-d872f2087c98
* Removed use of XmppSocketAdapter by sync.akalin@chromium.org2010-09-1019-1558/+26
| | | | | | | | | | | | | | | | Moved XmppSocketAdapter and friends to remoting directory. Removed some dead code in jingle/. This is a relanding of 59012, which was mistakenly reverted at 59104. BUG=54146,55116 TEST=none TBR=willchan Review URL: http://codereview.chromium.org/3300030 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@59134 0039d316-1c4b-4281-b951-d872f2087c98
* Revert r59012 which started using Chrome sockets for sync.willchan@chromium.org2010-09-1019-26/+1558
| | | | | | | | | | | This caused us to write to ChromeNetLog from the sync thread. ChromeNetLog is not thread safe, so this causes problems. BUG=55116,54146 TEST=Start up chrome. Does not repeatedly hit DCHECKs on ChromeNetLog::AddEntry(). Review URL: http://codereview.chromium.org/3358028 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@59104 0039d316-1c4b-4281-b951-d872f2087c98
* Removed use of XmppSocketAdapter by sync.akalin@chromium.org2010-09-0919-1558/+26
| | | | | | | | | | | | | Moved XmppSocketAdapter and friends to remoting directory. Removed some dead code in jingle/. BUG=54146 TEST=none Review URL: http://codereview.chromium.org/3298021 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@59012 0039d316-1c4b-4281-b951-d872f2087c98
* Adding the sync-notification-host command line parameter to the integration ↵rsimha@chromium.org2010-09-087-27/+204
| | | | | | | | | | | | | | | | | | tests. The sync integration tests currently access the network to communicate with the notification server. We need to move to a model where all server accesses are stubbed out by local servers. This is the first step towards implementing a local notification server. BUG=53933,53931 TEST=sync_integration_tests First time committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=58768 Review URL: http://codereview.chromium.org/3326007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@58860 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 58768 - Adding the sync-notification-host command line parameter to ↵rsimha@chromium.org2010-09-077-204/+27
| | | | | | | | | | | | | | | | | | | the integration tests. The sync integration tests currently access the network to communicate with the notification server. We need to move to a model where all server accesses are stubbed out by local servers. This is the first step towards implementing a local notification server. BUG=53933,53931 TEST=sync_integration_tests Review URL: http://codereview.chromium.org/3326007 TBR=rsimha@chromium.org Review URL: http://codereview.chromium.org/3318016 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@58769 0039d316-1c4b-4281-b951-d872f2087c98
* Adding the sync-notification-host command line parameter to the integration ↵rsimha@chromium.org2010-09-077-27/+204
| | | | | | | | | | | | | | | | tests. The sync integration tests currently access the network to communicate with the notification server. We need to move to a model where all server accesses are stubbed out by local servers. This is the first step towards implementing a local notification server. BUG=53933,53931 TEST=sync_integration_tests Review URL: http://codereview.chromium.org/3326007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@58768 0039d316-1c4b-4281-b951-d872f2087c98
* Fix ClientSocketHandle reuse_type(). Correctly track socket use.willchan@chromium.org2010-09-025-2/+23
| | | | | | | | | | | | | In particular, we used to consider that a socket had been used whenever it got returned to the ClientSocketPool. But, with preconnect, that is no longer true. Luckily, we now have UseHistory in the transport sockets. So, I create a WasEverUsed() method in ClientSocket, plumb this into all sockets, and use that in ClientSocketPoolBaseHelper instead of tracking whether or not the socket had been returned to the client or not. This ultimately will have two implications. We will record the correct values in Net.HttpSocketType histograms and we will use the correct timeout for preconnect sockets in ClientSocketPoolBaseHelper::CleanupIdleSockets(). BUG=none TEST=none Review URL: http://codereview.chromium.org/3353004 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@58363 0039d316-1c4b-4281-b951-d872f2087c98
* Fixed crash involving using NetAddress::Adopt() incorrectlyakalin@chromium.org2010-09-021-17/+13
| | | | | | | | | | | | | | Replaced use of NetAddress::Adopt() in ChromeAsyncSocket with the single-IP constructor. Clarified comment for NetAddress::Adopt(). BUG=54108 TEST=Windows sync integration tests Review URL: http://codereview.chromium.org/3353003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@58273 0039d316-1c4b-4281-b951-d872f2087c98
* FBTF: Move more ctors/dtors from headers to implementation.erg@google.com2010-09-012-0/+8
| | | | | | | | | | | | | | | | | - Adding/Moving dtors of objects that have CancelableRequestConsumers shaves three megs off browser.a. - Adding/Moving dtors of objects that have ScopedRunnableMethodFactories only shaved 100k off browser.a/renderer.a. - Adding/Moving dtors of objects that used some form of base::*Timer<> was negligible; there were only a few classes that had a Timer but had a ctor/dtor in the header after the previous cleanups. BUG=none TEST=compiles Review URL: http://codereview.chromium.org/3278006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@58192 0039d316-1c4b-4281-b951-d872f2087c98
* Style fix.akalin@chromium.org2010-09-011-12/+0
| | | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/3310001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@58144 0039d316-1c4b-4281-b951-d872f2087c98
* Added XmppClientSocketFactory for use with ChromeAsyncSocketakalin@chromium.org2010-09-017-91/+179
| | | | | | | | | | | | | XmppClientSocketFactory serves up regular TCP sockets or fake SSL TCP sockets. This way, ChromeAsyncSocket has feature parity with XmppSocketAdapter. BUG=45612 TEST=existing unittests Review URL: http://codereview.chromium.org/3299001 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@58121 0039d316-1c4b-4281-b951-d872f2087c98
* Added FakeSSLClentSocket for connecting to Google Talk on the SSL portakalin@chromium.org2010-09-014-0/+793
| | | | | | | | | | | | | | FakeSSLClientSocket is a wrapper around a ClientSocket that simulates an SSL handshake, which is needed to connect to Google Talk servers on the SSL port. This was adapted from the libjingle socket implementation. BUG=45612 TEST=new unit tests Review URL: http://codereview.chromium.org/3231006 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@58110 0039d316-1c4b-4281-b951-d872f2087c98
* FBTF: Remove unneeded headers from base/ (part 7)thestig@chromium.org2010-08-275-5/+1
| | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/3176026 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@57737 0039d316-1c4b-4281-b951-d872f2087c98
* Gather preconnection use vs waste statisticsjar@chromium.org2010-08-252-0/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | I now gather statistics only in the transport socket classes TcpClientSocket*. All other socket classes forward significant data (that they are used in a speculation) to their transport class, where this data is stored, and then dumped when the TCP socket is closed. This CL also repaired a slight miscount in bytes read and written on Windows, as error codes were mistakenly accumulated as byte counts. This CL repaired a significant undercounting in linux/mac via StatsCounter (the logging counter appears correct). Libjingle support is minimal (NOTREACHED), but I don't know that there are subresources that will be speculatively preconnected from that class (and it is less obvious how the class uses underlying sockets, if at all). BUG=42694 r=willchan Review URL: http://codereview.chromium.org/3163033 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@57377 0039d316-1c4b-4281-b951-d872f2087c98
* In about:net-internals, each SOCKET created by a CONNECT_JOB holds a ↵mmenke@chromium.org2010-08-251-2/+3
| | | | | | | | | | | reference to the CONNECT_JOB that spawned it. BUG=47082 TEST=manual Review URL: http://codereview.chromium.org/3164005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@57339 0039d316-1c4b-4281-b951-d872f2087c98
* Adds HostResolveImpl Requests and Jobs to log.mmenke@chromium.org2010-08-241-1/+2
| | | | | | | | | | | | | | | Differences over the reverted version (http://codereview.chromium.org/3080034/show - reverted http://codereview.chromium.org/3137022/show ): * Jobs never log anything in their destructor, as the HostResolverImpl randomly outlives the NetLog, at least in some unit tests. * Removed the extra log entries for when a DNS lookup starts/completes. Instead, the job's event is ended in OnLookupComplete(). * Slight modification of the CanceledAsynchronousLookup unit test, as the Job is now closed before posting any events to the Request. For actual lookups, this behavior means the Job's duration more accurately reflects the time the DNS lookup takes itself. BUG=46844 TEST=Look at the net-internals screen. Review URL: http://codereview.chromium.org/3119027 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@57189 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 56539 - Added HostResolveImpl Requests and Jobs to log.mmenke@chromium.org2010-08-181-2/+1
| | | | | | | | | | | | | | | | | | ConnectJobs point to the requests, Requests point back to ConnectJobs and to the DNS lookup they were attached to, if any. Also CONNECT_JOBs are now identified by their host/port on the Requests list. BUG= 46844 TEST= Look at the net-internals screen. Review URL: http://codereview.chromium.org/3080034 TBR=eroman@chromium.org Review URL: http://codereview.chromium.org/3137022 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56556 0039d316-1c4b-4281-b951-d872f2087c98
* Added HostResolveImpl Requests and Jobs to log.mmenke@chromium.org2010-08-181-1/+2
| | | | | | | | | | | | | | | ConnectJobs point to the requests, Requests point back to ConnectJobs and to the DNS lookup they were attached to, if any. Also CONNECT_JOBs are now identified by their host/port on the Requests list. BUG= 46844 TEST= Look at the net-internals screen. Review URL: http://codereview.chromium.org/3080034 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56539 0039d316-1c4b-4281-b951-d872f2087c98
* GTTF: some more FRIEND_TEST_ALL_PREFIXESphajdan.jr@chromium.org2010-08-162-4/+4
| | | | | | | | | TEST=none BUG=44549 Review URL: http://codereview.chromium.org/3161021 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56253 0039d316-1c4b-4281-b951-d872f2087c98
* Re-land just the project file changes from r55416. This makes projects that usebrettw@chromium.org2010-08-151-1/+1
| | | | | | | | | base test files dependent on test_support_base. TEST=none BUG=none git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56172 0039d316-1c4b-4281-b951-d872f2087c98
* Use SSL/TCP port for notifications in sync tests.rsimha@chromium.org2010-08-1012-9/+47
| | | | | | | | | | | | | The sync integration tests start off by using the XMPP port (5222) to connect to the sync notification server. Since this port is blocked on the chromium builders, the tests wait for upto a minute before they failover to the SSL/TCP port (443), by which time the tests time out and fail. This checkin adds a switch that, when enabled, causes the tests to try the SSL/TCP port before the XMPP port. BUG=50595 TEST=sync_integration_tests Review URL: http://codereview.chromium.org/3050036 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@55599 0039d316-1c4b-4281-b951-d872f2087c98
* Revert 55400 - Cleanup in base. This moves the implementation (and a bunch ↵nsylvain@chromium.org2010-08-091-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | of header file dependencies) from the multiprocess test and the test_suite headers to .cc files. Moves multiprocess_test to the test directory, and all of this stuff to the existing base_test_support project. I also used the base namespace. Previously other projects included this functionality just by #include because it was all inline, so I had to add dependencies on base_test_support in a few places. Moves and renames the command line switch this was using to base_switches. Move the base switch for process type to chrome switches. TEST=none BUG=none Review URL: http://codereview.chromium.org/3026055 TBR=brettw@chromium.org Review URL: http://codereview.chromium.org/3035062 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@55416 0039d316-1c4b-4281-b951-d872f2087c98
* Cleanup in base. This moves the implementation (and a bunch of header filebrettw@chromium.org2010-08-091-1/+1
| | | | | | | | | | | | | | | | | | | dependencies) from the multiprocess test and the test_suite headers to .cc files. Moves multiprocess_test to the test directory, and all of this stuff to the existing base_test_support project. I also used the base namespace. Previously other projects included this functionality just by #include because it was all inline, so I had to add dependencies on base_test_support in a few places. Moves and renames the command line switch this was using to base_switches. Move the base switch for process type to chrome switches. TEST=none BUG=none Review URL: http://codereview.chromium.org/3026055 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@55400 0039d316-1c4b-4281-b951-d872f2087c98
* Skip unresolved hosts for sync notifications instead of having libjingle ↵akalin@chromium.org2010-08-061-0/+7
| | | | | | | | | | | | | resolve them. This is a speculative fix for the crash in the bug below. BUG=chromeos:5145 TEST=manual Review URL: http://codereview.chromium.org/2868113 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@55264 0039d316-1c4b-4281-b951-d872f2087c98
* Move the number conversions from string_util to a new file.brettw@chromium.org2010-07-301-2/+2
| | | | | | | | | | | | | Use the base namespace in the new file. Update callers. I removed all wstring variants and also the string->number ones that ignore the return value. That encourages people to write code and forget about error handling. TEST=included unit tests BUG=none Review URL: http://codereview.chromium.org/3056029 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@54355 0039d316-1c4b-4281-b951-d872f2087c98
* Rewrote handing of auto-reconnection and network changes for sync notifier.akalin@chromium.org2010-07-2711-508/+97
| | | | | | | | | BUG=47883 TEST=manual (turning network connection off and on) Review URL: http://codereview.chromium.org/2809056 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@53839 0039d316-1c4b-4281-b951-d872f2087c98
* GTTF: more FRIEND_TEST_ALL_PREFIXESphajdan.jr@chromium.org2010-07-271-5/+6
| | | | | | | | | TEST=compile BUG=44549 Review URL: http://codereview.chromium.org/3064007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@53793 0039d316-1c4b-4281-b951-d872f2087c98
* Net: Get rid of ProxyServer::host_and_port() and friends.thestig@chromium.org2010-07-222-6/+6
| | | | | | | | BUG=none TEST=none Review URL: http://codereview.chromium.org/3032017 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@53403 0039d316-1c4b-4281-b951-d872f2087c98
* Fixed invalid reads when XmppClient is destroyed.akalin@chromium.org2010-07-201-2/+5
| | | | | | | | | BUG=49274 TEST=manual testing with code to trigger bug repro Review URL: http://codereview.chromium.org/3034016 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@53014 0039d316-1c4b-4281-b951-d872f2087c98
* Add a command line flag to change the default number of parallel DNS ↵eroman@chromium.org2010-07-171-1/+2
| | | | | | | | | | | | | requests issued by chrome. --host-resolver-parallelism=XXX This is a useful debugging tool, since for some users too many parallel DNS requests results in spurious DNS failures from their resolver. BUG=44489 Review URL: http://codereview.chromium.org/3019007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@52802 0039d316-1c4b-4281-b951-d872f2087c98
* Added switch to use ChromeAsyncSocket for sync.akalin@chromium.org2010-07-167-32/+84
| | | | | | | | | BUG=45612 TEST=manual Review URL: http://codereview.chromium.org/2819049 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@52737 0039d316-1c4b-4281-b951-d872f2087c98
* Moved ChromeAsyncSocket to jingle/notifier/base.akalin@chromium.org2010-07-164-0/+1725
| | | | | | | | | | | Fixed some signed/unsigned warnings. BUG=45612 TEST=existing unittests Review URL: http://codereview.chromium.org/2857039 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@52638 0039d316-1c4b-4281-b951-d872f2087c98
* Fixed bug where sync notifications don't work for non-gmail accounts.akalin@chromium.org2010-07-141-1/+34
| | | | | | | | | BUG=48988 TEST=manual Review URL: http://codereview.chromium.org/2981010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@52261 0039d316-1c4b-4281-b951-d872f2087c98
* Fixed events handling in XmppSocketAdapter. Removed unnecessary warnings.sergeyu@chromium.org2010-07-012-118/+92
| | | | | | | | | | | The problem was that SSLSocketAdapter::OnReadEvent() was calling AsyncSocketAdapter::OnReadEvent() only when there are no pending read requests, so the reader wasn't always receiving notification when new data is received on the socket, especially when data received is bigger than the pending read request. BUG=none TEST=xmpp connection succeeds 100% of the time, particularly on mac. Review URL: http://codereview.chromium.org/2834024 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@51385 0039d316-1c4b-4281-b951-d872f2087c98
* TBR: akalin@chromium.orgajwong@chromium.org2010-06-301-5/+0
| | | | git-svn-id: svn://svn.chromium.org/chrome/trunk/src@51217 0039d316-1c4b-4281-b951-d872f2087c98
* Move chrome/common/net/notifier to jingle/notifier so that it can be used by ↵ajwong@chromium.org2010-06-3058-0/+6048
remoting/remoting.gyp. BUG=none TEST=compiles Review URL: http://codereview.chromium.org/2885005 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@51216 0039d316-1c4b-4281-b951-d872f2087c98
generated by cgit v1.1 at 2025-02-01 09:14:05 +0000