From 193b0b89b4b49f8e8eb985b8f1a16e3c8506ee66 Mon Sep 17 00:00:00 2001 From: "lzheng@chromium.org" Date: Sat, 26 Jun 2010 03:57:57 +0000 Subject: Add field trial stats for alternate_protocol. The histogram we collected are: http stats when alternate protocol is available but npn was not negotiated ( which means not usng spdy), and when alternate is available AND spdy is used. Noticable changes: 1. In http_network_transaction.cc, changed the logic that always parse response from server for alternate protocol and remember that in HttpAlternateProtocols strucuture. We need to remember this to collect stats for servers with alternate protocol support but used http for. 2. In spdy_stream.cc, get rid of the response copy from spdy_stream. This copy overwrites some early status in response set in http_network_transaction. TEST=http_network_transaction_unittest.cc BUG=46689 Review URL: http://codereview.chromium.org/2808010 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@50927 0039d316-1c4b-4281-b951-d872f2087c98 --- chrome/common/render_messages.h | 4 ++++ 1 file changed, 4 insertions(+) (limited to 'chrome/common') diff --git a/chrome/common/render_messages.h b/chrome/common/render_messages.h index 061a236..fc54b29 100644 --- a/chrome/common/render_messages.h +++ b/chrome/common/render_messages.h @@ -1366,6 +1366,7 @@ struct ParamTraits { WriteParam(m, p.appcache_manifest_url); WriteParam(m, p.was_fetched_via_spdy); WriteParam(m, p.was_npn_negotiated); + WriteParam(m, p.was_alternate_protocol_available); WriteParam(m, p.was_fetched_via_proxy); } static bool Read(const Message* m, void** iter, param_type* r) { @@ -1381,6 +1382,7 @@ struct ParamTraits { ReadParam(m, iter, &r->appcache_manifest_url) && ReadParam(m, iter, &r->was_fetched_via_spdy) && ReadParam(m, iter, &r->was_npn_negotiated) && + ReadParam(m, iter, &r->was_alternate_protocol_available) && ReadParam(m, iter, &r->was_fetched_via_proxy); } static void Log(const param_type& p, std::wstring* l) { @@ -1407,6 +1409,8 @@ struct ParamTraits { l->append(L", "); LogParam(p.was_npn_negotiated, l); l->append(L", "); + LogParam(p.was_alternate_protocol_available, l); + l->append(L", "); LogParam(p.was_fetched_via_proxy, l); l->append(L")"); } -- cgit v1.1