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

Unified Diff: net/spdy/spdy_session.h

Issue 2623803002: Avoid creating MemoryAllocatorDump for individual sockets (Closed)
Patch Set: self review 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/spdy/spdy_session.h
diff --git a/net/spdy/spdy_session.h b/net/spdy/spdy_session.h
index 61f2a9d14840448dbd3dddeec8b8e570d4b07f85..ca6c5e9e0f2ed8c0a9380f03632f7d90ecbd8ed7 100644
--- a/net/spdy/spdy_session.h
+++ b/net/spdy/spdy_session.h
@@ -47,12 +47,6 @@
#include "url/gurl.h"
#include "url/scheme_host_port.h"
-namespace base {
-namespace trace_event {
-class ProcessMemoryDump;
-}
-}
-
namespace net {
namespace test {
@@ -560,10 +554,10 @@ class NET_EXPORT SpdySession : public BufferedSpdyFramerVisitorInterface,
// HigherLayeredPool implementation:
bool CloseOneIdleConnection() override;
- // Dumps memory allocation stats. |parent_dump_absolute_name| is the name
- // used by the parent MemoryAllocatorDump in the memory dump hierarchy.
- void DumpMemoryStats(base::trace_event::ProcessMemoryDump* pmd,
- const std::string& parent_dump_absolute_name) const;
+ // Dumps memory allocation stats to |stats|. Set |*is_session_active| to
ssid 2017/01/10 23:48:13 Set/Sets. consistent with Dumps
xunjieli 2017/01/11 20:05:07 Done.
+ // indicate whether session is active.
+ void DumpMemoryStats(StreamSocket::SocketMemoryStats* stats,
+ bool* is_session_active) const;
private:
friend class test::SpdyStreamTest;

Powered by Google App Engine
This is Rietveld 408576698