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

Unified Diff: net/spdy/spdy_session.cc

Issue 9696045: Expose a spdy session's pooled aliases in the net-internals page. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 9 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.cc
diff --git a/net/spdy/spdy_session.cc b/net/spdy/spdy_session.cc
index e9c7cd80996e71899832243ae478bd5bee216a18..4d187429a85915356f6263233a52b9436e23adbe 100644
--- a/net/spdy/spdy_session.cc
+++ b/net/spdy/spdy_session.cc
@@ -1132,6 +1132,12 @@ Value* SpdySession::GetInfoAsValue() const {
dict->SetInteger("source_id", net_log_.source().id);
dict->SetString("host_port_pair", host_port_proxy_pair_.first.ToString());
+ ListValue* alias_list = new ListValue();
+ for (std::set<HostPortProxyPair>::const_iterator it = pooled_aliases_.begin();
+ it != pooled_aliases_.end(); it++) {
+ alias_list->Append(Value::CreateStringValue(it->first.ToString()));
+ }
+ dict->Set("aliases", alias_list);
eroman 2012/03/13 20:08:42 I suggest only setting the aliases property if the
Ryan Hamilton 2012/03/13 21:55:37 Done. (I had thought we would prefer the property
dict->SetString("proxy", host_port_proxy_pair_.second.ToURI());
dict->SetInteger("active_streams", active_streams_.size());

Powered by Google App Engine
This is Rietveld 408576698