Improve memory estimate of SpdySessionPool in net/ MemoryDumpProvider.

net/spdy/spdy_session_key.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "net/spdy/spdy_session_key.h"
 
 #include <tuple>
 
 #include "base/logging.h"
+#include "base/trace_event/memory_usage_estimator.h"
+#include "net/base/host_port_pair.h"
 
 namespace net {
 
 SpdySessionKey::SpdySessionKey() : privacy_mode_(PRIVACY_MODE_DISABLED) {
 }
 
 SpdySessionKey::SpdySessionKey(const HostPortPair& host_port_pair,
                                const ProxyServer& proxy_server,
                                PrivacyMode privacy_mode)
     : host_port_proxy_pair_(host_port_pair, proxy_server),
@@ skipping 22 matching lines @@
          std::tie(other.privacy_mode_, other.host_port_proxy_pair_.first,
                   other.host_port_proxy_pair_.second);
 }
 
 bool SpdySessionKey::Equals(const SpdySessionKey& other) const {
   return privacy_mode_ == other.privacy_mode_ &&
       host_port_proxy_pair_.first.Equals(other.host_port_proxy_pair_.first) &&
       host_port_proxy_pair_.second == other.host_port_proxy_pair_.second;
 }
 
+size_t SpdySessionKey::EstimateMemoryUsage() const {
+  return HostPortPair::EstimateMemoryUsage(host_port_proxy_pair_.first) +

[DmitrySkiba, 2017/02/02 18:16:27]

Why EMU() is not an instance method in HostPortPair? It it was you could write
just EMU(host_port_proxy_pair_) because we have generic EMU for std::pair.

[xunjieli, 2017/02/03 22:25:10]

Done.

+         base::trace_event::EstimateMemoryUsage(host_port_proxy_pair_.second);
+}
+
 }  // namespace net