summaryrefslogtreecommitdiffstats
path: root/jingle/notifier
Commit message (Collapse)AuthorAgeFilesLines
* 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
* 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
* 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-2710-506/+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-163-0/+1720
| | | | | | | | | | | 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
* Move chrome/common/net/notifier to jingle/notifier so that it can be used by ↵ajwong@chromium.org2010-06-3056-0/+5906
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-19 21:21:07 +0000