Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1008)

Unified Diff: net/http/http_stream_factory_impl_job.cc

Issue 2623383005: Log states of JobController in Net MemoryDumpProvider (Closed)
Patch Set: Separate out preconnect vs non-preconnect Created 3 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: net/http/http_stream_factory_impl_job.cc
diff --git a/net/http/http_stream_factory_impl_job.cc b/net/http/http_stream_factory_impl_job.cc
index 2cd3bcbe0b6c7cf73b0129896405537a9a92cfcb..f72c40ab9b5f1e751b038f98e0f19d00d43198e3 100644
--- a/net/http/http_stream_factory_impl_job.cc
+++ b/net/http/http_stream_factory_impl_job.cc
@@ -46,6 +46,7 @@
#include "net/socket/socks_client_socket_pool.h"
#include "net/socket/ssl_client_socket.h"
#include "net/socket/ssl_client_socket_pool.h"
+#include "net/socket/stream_socket.h"
#include "net/spdy/bidirectional_stream_spdy_impl.h"
#include "net/spdy/spdy_http_stream.h"
#include "net/spdy/spdy_protocol.h"
@@ -334,6 +335,13 @@ bool HttpStreamFactoryImpl::Job::using_spdy() const {
return using_spdy_;
}
+size_t HttpStreamFactoryImpl::Job::EstimateMemoryUsage() const {
+ StreamSocket::SocketMemoryStats stats;
+ if (connection_)
+ connection_->DumpMemoryStats(&stats);
+ return stats.total_size;
ssid 2017/01/18 23:51:52 So, this does not include stream_, websocket_strea
xunjieli 2017/01/19 14:50:42 The lifetime of a Job is very short. Once connecti
ssid 2017/01/20 20:20:23 Okay this is good for now. If we see a lot of diff
+}
+
const SSLConfig& HttpStreamFactoryImpl::Job::server_ssl_config() const {
return server_ssl_config_;
}

Powered by Google App Engine
This is Rietveld 408576698