diff options
| author | eroman@chromium.org <eroman@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-06-03 00:49:17 +0000 |
|---|---|---|
| committer | eroman@chromium.org <eroman@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-06-03 00:49:17 +0000 |
| commit | 06650c582f9cfcf3869c490f3dc1ca224d6c9963 (patch) | |
| tree | b6585569ec02f0bec32aa65b6e36b3cbf8a8533f /net/spdy | |
| parent | cb364d78210e5d64f3c899bbfa87dadebac7d6f7 (diff) | |
| download | chromium_src-06650c582f9cfcf3869c490f3dc1ca224d6c9963.zip chromium_src-06650c582f9cfcf3869c490f3dc1ca224d6c9963.tar.gz chromium_src-06650c582f9cfcf3869c490f3dc1ca224d6c9963.tar.bz2 | |
Rework the logging for sockets/connectjobs.
In particular, make it work better when using backup jobs / late binding (the display was very confused before because of how these asynchronous events would nest).
Also changed the paradigm for how PassiveLogCollector preserves these async associations -- this fixes how it replays the events to net-internals. (Before we would collapse the event streams into the SOURCE_URL_REQUEST which lost some hiearchy.. now I keep the separate streams).
Some of the particular changes to the event streams:
* ConnectJobs now create their own source stream internally.
* Sockets are now bounded by +SOCKET_ALIVE / -SOCKET_ALIVE events (removed the one-off SOCKET_DONE event).
* The socket log streams contains +SOCKET_IN_USE / -SOCKET_IN_USE event blocks to show which URLRequest was controlling it at various points in time (this makes it much easier to understand which read/writes belonged to a particular network transaction when a socket gets re-used).
* ConnectJobs are bounded by +SOCKET_POOL_CONNECT_JOB / - SOCKET_POOL_CONNECT_JOB events.
* ConnectJobs log the net error they failed with.
* Removed the SOCKET_BACKUP_TIMER_EXTENDED event.
Review URL: http://codereview.chromium.org/2363003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@48797 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'net/spdy')
| -rw-r--r-- | net/spdy/spdy_network_transaction_unittest.cc | 3 | ||||
| -rw-r--r-- | net/spdy/spdy_session_unittest.cc | 1 | ||||
| -rw-r--r-- | net/spdy/spdy_stream_unittest.cc | 3 |
3 files changed, 5 insertions, 2 deletions
diff --git a/net/spdy/spdy_network_transaction_unittest.cc b/net/spdy/spdy_network_transaction_unittest.cc index add273b..e8278a5 100644 --- a/net/spdy/spdy_network_transaction_unittest.cc +++ b/net/spdy/spdy_network_transaction_unittest.cc @@ -87,7 +87,8 @@ HttpNetworkSession* CreateSession(SessionDependencies* session_deps) { &session_deps->socket_factory, session_deps->ssl_config_service, session_deps->spdy_session_pool, - session_deps->http_auth_handler_factory.get()); + session_deps->http_auth_handler_factory.get(), + NULL); } // Chop a frame into an array of MockWrites. diff --git a/net/spdy/spdy_session_unittest.cc b/net/spdy/spdy_session_unittest.cc index ab7b12c..d022232 100644 --- a/net/spdy/spdy_session_unittest.cc +++ b/net/spdy/spdy_session_unittest.cc @@ -55,6 +55,7 @@ HttpNetworkSession* CreateSession(SessionDependencies* session_deps) { &session_deps->socket_factory, session_deps->ssl_config_service, session_deps->spdy_session_pool, + NULL, NULL); } diff --git a/net/spdy/spdy_stream_unittest.cc b/net/spdy/spdy_stream_unittest.cc index 3da9da73..84db904 100644 --- a/net/spdy/spdy_stream_unittest.cc +++ b/net/spdy/spdy_stream_unittest.cc @@ -78,7 +78,8 @@ HttpNetworkSession* CreateSession(SessionDependencies* session_deps) { &session_deps->socket_factory, session_deps->ssl_config_service, session_deps->spdy_session_pool, - session_deps->http_auth_handler_factory.get()); + session_deps->http_auth_handler_factory.get(), + NULL); } class SpdyStreamTest : public testing::Test { |