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

Unified Diff: chrome/renderer/page_load_histograms.cc

Issue 6804032: Add TLS-SRP (RFC 5054) support Base URL: http://git.chromium.org/git/chromium.git@trunk
Patch Set: use system srp and mpi libs, not local copies Created 9 years, 8 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: chrome/renderer/page_load_histograms.cc
diff --git a/chrome/renderer/page_load_histograms.cc b/chrome/renderer/page_load_histograms.cc
index 48ef7e5f333a5dde7282ea04f6ebef0ebe3df869..2223fab37142a543a634466ff45a7fd0bc13eb8a 100644
--- a/chrome/renderer/page_load_histograms.cc
+++ b/chrome/renderer/page_load_histograms.cc
@@ -35,6 +35,8 @@ static URLPattern::SchemeMasks GetSupportedSchemeType(const GURL& url) {
return URLPattern::SCHEME_HTTP;
else if (url.SchemeIs("https"))
return URLPattern::SCHEME_HTTPS;
+ else if (url.SchemeIs("httpsv"))
+ return URLPattern::SCHEME_HTTPSV;
return static_cast<URLPattern::SchemeMasks>(0);
}
@@ -596,7 +598,8 @@ void PageLoadHistograms::Dump(WebFrame* frame) {
// Histograms to determine if SPDY has an impact for https traffic.
// TODO(mbelshe): After we've seen the difference between BeginToFinish
// and StartToFinish, consider removing one or the other.
- if (scheme_type == URLPattern::SCHEME_HTTPS &&
+ if ((scheme_type == URLPattern::SCHEME_HTTPS ||
+ scheme_type == URLPattern::SCHEME_HTTPSV) &&
navigation_state->was_npn_negotiated()) {
UMA_HISTOGRAM_ENUMERATION(
base::FieldTrial::MakeName("PLT.Abandoned", "SpdyImpact"),
@@ -714,7 +717,8 @@ void PageLoadHistograms::Dump(WebFrame* frame) {
// Record page load time and abandonment rates for proxy cases.
if (navigation_state->was_fetched_via_proxy()) {
- if (scheme_type == URLPattern::SCHEME_HTTPS) {
+ if (scheme_type == URLPattern::SCHEME_HTTPS ||
+ scheme_type == URLPattern::SCHEME_HTTPSV) {
PLT_HISTOGRAM("PLT.StartToFinish.Proxy.https", start_to_finish_all_loads);
UMA_HISTOGRAM_ENUMERATION("PLT.Abandoned.Proxy.https",
abandoned_page ? 1 : 0, 2);
@@ -725,7 +729,8 @@ void PageLoadHistograms::Dump(WebFrame* frame) {
abandoned_page ? 1 : 0, 2);
}
} else {
- if (scheme_type == URLPattern::SCHEME_HTTPS) {
+ if (scheme_type == URLPattern::SCHEME_HTTPS ||
+ scheme_type == URLPattern::SCHEME_HTTPSV) {
PLT_HISTOGRAM("PLT.StartToFinish.NoProxy.https",
start_to_finish_all_loads);
UMA_HISTOGRAM_ENUMERATION("PLT.Abandoned.NoProxy.https",

Powered by Google App Engine
This is Rietveld 408576698