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

Unified Diff: chrome/renderer/page_load_histograms.cc

Issue 10985083: Upstreaming SpdyProxy-related switches, OTR logic, and histograms (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Removed non-change from pref_names.h Created 8 years, 3 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 85b45a096a98a9ec2d4bdbea4e5ced16dda23f83..960c00f6707eb6c051d8639742eb3b95fb6503a9 100644
--- a/chrome/renderer/page_load_histograms.cc
+++ b/chrome/renderer/page_load_histograms.cc
@@ -22,6 +22,11 @@
#include "third_party/WebKit/Source/WebKit/chromium/public/platform/WebURLResponse.h"
#include "third_party/WebKit/Source/WebKit/chromium/public/WebView.h"
+#if defined(OS_ANDROID)
+#include "base/command_line.h"
+#include "chrome/common/chrome_switches.h"
+#endif
+
using WebKit::WebDataSource;
using WebKit::WebFrame;
using WebKit::WebPerformance;
@@ -351,6 +356,17 @@ void PageLoadHistograms::Dump(WebFrame* frame) {
}
}
+#if defined(OS_ANDROID)
+ if (document_state->was_fetched_via_proxy() &&
+ document_state->was_fetched_via_spdy() &&
+ CommandLine::ForCurrentProcess()->HasSwitch(switches::kSpdyProxyOrigin)) {
+ UMA_HISTOGRAM_ENUMERATION(
+ "PLT.Abandoned_SpdyProxy", abandoned_page ? 1 : 0, 2);
+ PLT_HISTOGRAM("PLT.BeginToFinishDoc_SpdyProxy", begin_to_finish_doc);
+ PLT_HISTOGRAM("PLT.BeginToFinish_SpdyProxy", begin_to_finish_all_loads);
+ }
+#endif
+
// Histograms to determine if prefetch & prerender has an impact on PLT.
static const bool prefetching_fieldtrial =
base::FieldTrialList::TrialExists("Prefetch");

Powered by Google App Engine
This is Rietveld 408576698