Small: Add store specific histograms also.

components/safe_browsing_db/v4_store.cc

 // Copyright 2016 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 "base/base64.h"
 #include "base/bind.h"
 #include "base/files/file_util.h"
 #include "base/memory/ptr_util.h"
 #include "base/metrics/histogram_macros.h"
 #include "base/metrics/sparse_histogram.h"
@@ skipping 15 matching lines @@
 // Part 1: Represent the overall operation being performed.
 const char kProcessFullUpdate[] = "SafeBrowsing.V4ProcessFullUpdate";
 const char kProcessPartialUpdate[] = "SafeBrowsing.V4ProcessPartialUpdate";
 const char kReadFromDisk[] = "SafeBrowsing.V4ReadFromDisk";
 // Part 2: Represent the sub-operation being performed as part of the larger
 // operation from part 1.
 const char kApplyUpdate[] = ".ApplyUpdate";
 const char kDecodeAdditions[] = ".DecodeAdditions";
 const char kDecodeRemovals[] = ".DecodeRemovals";
 const char kMergeUpdate[] = ".MergeUpdate";
-// Part 3: Represent the unit of value being measured and logged.
+// Part 3 (optional): Represent the name of the list for which the metric is
+// being logged. For instance, ".UrlSoceng".
+// Part 4: Represent the unit of value being measured and logged.
 const char kResult[] = ".Result";
 const char kTime[] = ".Time";
 // UMA metric names for this file are generated by appending one value each,
-// in order, from parts 1, 2, and 3.
+// in order, from parts [1, 2, and 4], or [1, 2, 3, and 4]. For example:
+// SafeBrowsing.V4ProcessPartialUpdate.ApplyUpdate.Result, or
+// SafeBrowsing.V4ProcessPartialUpdate.ApplyUpdate.UrlSoceng.Result
 
 const uint32_t kFileMagic = 0x600D71FE;
 const uint32_t kFileVersion = 9;
 
 std::string GetUmaSuffixForStore(const base::FilePath& file_path) {
   return base::StringPrintf(
       ".%" PRIsFP, file_path.BaseName().RemoveExtension().value().c_str());
 }
 
 void RecordTimeWithAndWithoutSuffix(const std::string& metric,
@@ skipping 753 matching lines @@
                << "; expected: " << expected_checksum_b64
                << "; store: " << *this;
 #endif
       return false;
     }
   }
   return true;
 }
 
 }  // namespace safe_browsing