diff options
author | lzheng@chromium.org <lzheng@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-05-21 06:56:53 +0000 |
---|---|---|
committer | lzheng@chromium.org <lzheng@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-05-21 06:56:53 +0000 |
commit | 65041fa2a7d79cbd289d6bfa05a55bfb32ec4fa3 (patch) | |
tree | 5ba89200e8bdc109b64b9327fad2bc563bc03406 /chrome/common | |
parent | 5ea89fbf0d7647669fc91dda7fe944d8f2ef1f1c (diff) | |
download | chromium_src-65041fa2a7d79cbd289d6bfa05a55bfb32ec4fa3.zip chromium_src-65041fa2a7d79cbd289d6bfa05a55bfb32ec4fa3.tar.gz chromium_src-65041fa2a7d79cbd289d6bfa05a55bfb32ec4fa3.tar.bz2 |
This change enables FieldTrial for SPDY. When --use-spdy=npn is used, field test won't be enabled.
However, when that flag is missing, A/B test is added to browser_main.cc. Trial A: use npn and spdy. B: use npn but no spdy. C: do nothing. A and B are set to zero for now
The histograms we collect are:
1. Page begin to finish time when spdy is enabled/disabled across all sites;
2. Page begin to finish time when spdy is used on sites that support spdy and when spdy is intentionally ignored.
BUG=43997
TEST=Don't use --use-spdy=npn, manually change _npn_nospdy and _npn_withspdy percentile and go to spdy supported sites.
Review URL: http://codereview.chromium.org/2036012
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@47896 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/common')
-rw-r--r-- | chrome/common/render_messages.h | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/chrome/common/render_messages.h b/chrome/common/render_messages.h index e1f8791..ab0dcc1 100644 --- a/chrome/common/render_messages.h +++ b/chrome/common/render_messages.h @@ -1327,6 +1327,7 @@ struct ParamTraits<webkit_glue::ResourceLoaderBridge::ResponseInfo> { WriteParam(m, p.appcache_id); WriteParam(m, p.appcache_manifest_url); WriteParam(m, p.was_fetched_via_spdy); + WriteParam(m, p.was_npn_negotiated); } static bool Read(const Message* m, void** iter, param_type* r) { return @@ -1339,7 +1340,8 @@ struct ParamTraits<webkit_glue::ResourceLoaderBridge::ResponseInfo> { ReadParam(m, iter, &r->content_length) && ReadParam(m, iter, &r->appcache_id) && ReadParam(m, iter, &r->appcache_manifest_url) && - ReadParam(m, iter, &r->was_fetched_via_spdy); + ReadParam(m, iter, &r->was_fetched_via_spdy) && + ReadParam(m, iter, &r->was_npn_negotiated); } static void Log(const param_type& p, std::wstring* l) { l->append(L"("); @@ -1362,6 +1364,8 @@ struct ParamTraits<webkit_glue::ResourceLoaderBridge::ResponseInfo> { LogParam(p.appcache_manifest_url, l); l->append(L", "); LogParam(p.was_fetched_via_spdy, l); + l->append(L","); + LogParam(p.was_npn_negotiated, l); l->append(L")"); } }; |