Add filesystem scheme to navigation histograms.

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"
 
 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
 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_MAX,
 };
 
 const char* const kSchemeNames[] = {
-  "unknown",
-  url::kHttpScheme,
-  url::kHttpsScheme,
-  url::kFileScheme,
-  url::kFtpScheme,
-  url::kDataScheme,
-  url::kJavaScriptScheme,
-  url::kAboutScheme,
-  "chrome",
-  url::kBlobScheme,
-  "max",
+    "unknown",
+    url::kHttpScheme,
+    url::kHttpsScheme,
+    url::kFileScheme,
+    url::kFtpScheme,
+    url::kDataScheme,
+    url::kJavaScriptScheme,
+    url::kAboutScheme,
+    "chrome",
+    url::kBlobScheme,
+    url::kFileSystemScheme,
+    "max",

[Ilya Sherman, 2016/11/04 18:34:30]

nit: By the way, I don't think the indentation adjustment here is correct.  Is
this a result of "git cl format", or a manual edit?  (If it's git cl format,
then it might be worth filing a git cl format bug, but mostly you can ignore my
comment.)

[meacer, 2016/11/04 19:11:46]

I used git cl format. Should the correct indentation be 2 spaces? It looks like
there is existing 4 space examples in the codebase too:
https://cs.chromium.org/chromium/src/net/dns/mdns_client_unittest.cc?q=k%5Ba-...

[Ilya Sherman, 2016/11/05 00:29:00]

Yeah, I would have expected 2 spaces.  But, if this is git cl format output,
it's probably not worth worrying about too much.  Thanks!

 };
 
 static_assert(arraysize(kSchemeNames) == SCHEME_MAX + 1,
               "kSchemeNames should have SCHEME_MAX + 1 elements");
 
 }  // namespace
 
 namespace navigation_metrics {
 
 void RecordMainFrameNavigation(const GURL& url,
@@ skipping 19 matching lines @@
   }
 
   UMA_HISTOGRAM_ENUMERATION("Navigation.MainFrameScheme", scheme, SCHEME_MAX);
   if (!is_in_page) {
     UMA_HISTOGRAM_ENUMERATION("Navigation.MainFrameSchemeDifferentPage", scheme,
                               SCHEME_MAX);
   }
 }
 
 }  // namespace navigation_metrics