Add missing schemes to RecordMainFrameNavigation

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_macros.h"
 #include "url/gurl.h"
 
 namespace {
 
-// This enum is used in building the histogram. So, this is append only,
-// any new scheme should be added at the end, before SCHEME_MAX
+// These values are written to logs. New enum values can be added, but existing
+// enums must never be renumbered or deleted and reused. Any new scheme should
+// be added at the end, before SCHEME_MAX.
 enum Scheme {
-  SCHEME_UNKNOWN,
-  SCHEME_HTTP,
-  SCHEME_HTTPS,
-  SCHEME_FILE,
-  SCHEME_FTP,
-  SCHEME_DATA,
-  SCHEME_JAVASCRIPT,
-  SCHEME_ABOUT,
-  SCHEME_CHROME,
-  SCHEME_BLOB,
-  SCHEME_FILESYSTEM,
+  SCHEME_UNKNOWN = 0,
+  SCHEME_HTTP = 1,
+  SCHEME_HTTPS = 2,
+  SCHEME_FILE = 3,
+  SCHEME_FTP = 4,
+  SCHEME_DATA = 5,
+  SCHEME_JAVASCRIPT = 6,
+  SCHEME_ABOUT = 7,
+  SCHEME_CHROME = 8,
+  SCHEME_BLOB = 9,
+  SCHEME_FILESYSTEM = 10,
+  SCHEME_CHROMENATIVE = 11,

[Mark P, 2017/06/28 16:25:19]

optional nit: SCHEME_CHROMENATIVE is hard to parse for me.  Consider
SCHEME_CHROME_NATIVE
analogous comment below

[elawrence, 2017/06/28 17:03:40]

Fixed, thanks.

+  SCHEME_CHROMESEARCH = 12,
   SCHEME_MAX,
 };
 
 const char* const kSchemeNames[] = {
     "unknown",
     url::kHttpScheme,
     url::kHttpsScheme,
     url::kFileScheme,
     url::kFtpScheme,
     url::kDataScheme,
     url::kJavaScriptScheme,
     url::kAboutScheme,
     "chrome",
     url::kBlobScheme,
     url::kFileSystemScheme,
+    "chrome-native",
+    "chrome-search",
     "max",
 };
 
 static_assert(arraysize(kSchemeNames) == SCHEME_MAX + 1,
               "kSchemeNames should have SCHEME_MAX + 1 elements");
 
 }  // namespace
 
 namespace navigation_metrics {
 
@@ skipping 18 matching lines @@
     UMA_HISTOGRAM_ENUMERATION("Navigation.MainFrameSchemeOTR", scheme,
                               SCHEME_MAX);
     if (!is_same_document) {
       UMA_HISTOGRAM_ENUMERATION("Navigation.MainFrameSchemeDifferentPageOTR",
                                 scheme, SCHEME_MAX);
     }
   }
 }
 
 }  // namespace navigation_metrics