Histogram the scheme of an origin on the 1st navigation to it.

components/navigation_metrics/navigation_metrics.cc

 // Copyright 2013 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 "components/navigation_metrics/navigation_metrics.h"
 
 #include "base/macros.h"
 #include "base/metrics/histogram.h"
 #include "url/gurl.h"
 
@@ skipping 29 matching lines @@
   "max",
 };
 
 static_assert(arraysize(kSchemeNames) == SCHEME_MAX + 1,
               "kSchemeNames should have SCHEME_MAX + 1 elements");
 
 }  // namespace
 
 namespace navigation_metrics {
 
-void RecordMainFrameNavigation(const GURL& url, bool is_in_page) {
+void RecordMainFrameNavigation(const GURL& url,
+                               bool is_in_page,
+                               bool is_off_the_record,
+                               bool have_already_seen_origin) {
   Scheme scheme = SCHEME_UNKNOWN;
   for (int i = 1; i < SCHEME_MAX; ++i) {
     if (url.SchemeIs(kSchemeNames[i])) {
       scheme = static_cast<Scheme>(i);
       break;
     }
   }
-  UMA_HISTOGRAM_ENUMERATION("Navigation.MainFrameScheme", scheme, SCHEME_MAX);
-  if (!is_in_page) {
-    UMA_HISTOGRAM_ENUMERATION(
-        "Navigation.MainFrameSchemeDifferentPage", scheme, SCHEME_MAX);
+
+  if (is_off_the_record) {
+    if (!have_already_seen_origin) {
+      UMA_HISTOGRAM_ENUMERATION("Navigation.SchemePerUniqueOriginOTR", scheme,
+                                SCHEME_MAX);
+    }
+  } else {

[felt, 2016/04/07 02:28:04]

nit: could you simplify this by returning early?

[palmer, 2016/04/07 23:45:43]

Simplified in a different way.

+    if (!have_already_seen_origin) {
+      UMA_HISTOGRAM_ENUMERATION("Navigation.SchemePerUniqueOrigin", scheme,
+                                SCHEME_MAX);
+    }
+    UMA_HISTOGRAM_ENUMERATION("Navigation.MainFrameScheme", scheme, SCHEME_MAX);

[felt, 2016/04/07 02:28:04]

do we still want to calculate MFS like normal for OTR pages?

[palmer, 2016/04/07 23:45:43]

Oh, right. Yep!

+    if (!is_in_page) {
+      UMA_HISTOGRAM_ENUMERATION("Navigation.MainFrameSchemeDifferentPage",
+                                scheme, SCHEME_MAX);
+    }
   }
 }
 
 }  // namespace navigation_metrics