Introduce ServiceWorker.MainFramePageLoad.CoreTransition and RedirectChainLength UMAs.

content/browser/service_worker/service_worker_metrics.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/browser/service_worker/service_worker_metrics.h"
 
 #include <limits>
 #include <string>
 
 #include "base/metrics/histogram_macros.h"
@@ skipping 303 matching lines @@
   UMA_HISTOGRAM_SPARSE_SLOWLY("ServiceWorker.Storage.PurgeResourceResult",
                               std::abs(net_error));
 }
 
 void ServiceWorkerMetrics::RecordDeleteAndStartOverResult(
     DeleteAndStartOverResult result) {
   UMA_HISTOGRAM_ENUMERATION("ServiceWorker.Storage.DeleteAndStartOverResult",
                             result, NUM_DELETE_AND_START_OVER_RESULT_TYPES);
 }
 
-void ServiceWorkerMetrics::CountControlledPageLoad(Site site,
-                                                   const GURL& url,
-                                                   bool is_main_frame_load) {
+void ServiceWorkerMetrics::CountControlledPageLoad(
+    Site site,
+    const GURL& url,
+    bool is_main_frame_load,
+    ui::PageTransition page_transition,
+    size_t redirect_chain_length) {
   DCHECK_NE(site, Site::OTHER);
   UMA_HISTOGRAM_ENUMERATION("ServiceWorker.PageLoad", static_cast<int>(site),
                             static_cast<int>(Site::NUM_TYPES));
   if (is_main_frame_load) {
     UMA_HISTOGRAM_ENUMERATION("ServiceWorker.MainFramePageLoad",
                               static_cast<int>(site),
                               static_cast<int>(Site::NUM_TYPES));
   }
-
   if (ShouldExcludeSiteFromHistogram(site))
     return;
+
+  if (is_main_frame_load) {
+    UMA_HISTOGRAM_ENUMERATION(
+        "ServiceWorker.MainFramePageLoad.CoreTransition",
+        static_cast<int>(ui::PageTransitionStripQualifier(page_transition)),
+        static_cast<int>(ui::PAGE_TRANSITION_LAST_CORE));

[falken, 2016/12/06 01:25:14]

should be LAST_CORE + 1?

[horo, 2016/12/06 01:44:51]

Done.

+    // The max number of http redirects is 20. So the max number of the chain
+    // length is 21.

[falken, 2016/12/06 01:25:14]

Could we make kMaxRedirects public and refer to it here instead of a literal 21?

[horo, 2016/12/06 01:44:51]

Done.

+    UMA_HISTOGRAM_EXACT_LINEAR(
+        "ServiceWorker.MainFramePageLoad.RedirectChainLength",
+        redirect_chain_length, 21);
+  }
   BrowserThread::PostTask(BrowserThread::UI, FROM_HERE,
                           base::Bind(&RecordURLMetricOnUI, url));
 }
 
 void ServiceWorkerMetrics::RecordStartWorkerStatus(
     ServiceWorkerStatusCode status,
     EventType purpose,
     bool is_installed) {
   if (!is_installed) {
     UMA_HISTOGRAM_ENUMERATION("ServiceWorker.StartNewWorker.Status", status,
@@ skipping 392 matching lines @@
   } else if (failure_count == 2) {
     UMA_HISTOGRAM_ENUMERATION("ServiceWorker.StartWorker.AfterFailureStreak_2",
                               status, SERVICE_WORKER_ERROR_MAX_VALUE);
   } else if (failure_count == 3) {
     UMA_HISTOGRAM_ENUMERATION("ServiceWorker.StartWorker.AfterFailureStreak_3",
                               status, SERVICE_WORKER_ERROR_MAX_VALUE);
   }
 }
 
 }  // namespace content