Add new memory histograms which fix inprecise bucketing.

chrome/browser/metrics/metrics_memory_details.cc

 // Copyright 2015 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 "chrome/browser/metrics/metrics_memory_details.h"
 
 #include <vector>
 
 #include "base/location.h"
 #include "base/metrics/histogram_macros.h"
@@ skipping 177 matching lines @@
   UMA_HISTOGRAM_COUNTS_100("Memory.PluginProcessCount", plugin_count);
   UMA_HISTOGRAM_COUNTS_100("Memory.PepperPluginProcessCount",
                            pepper_plugin_count);
   UMA_HISTOGRAM_COUNTS_100("Memory.PepperPluginBrokerProcessCount",
                            pepper_plugin_broker_count);
   UMA_HISTOGRAM_COUNTS_100("Memory.RendererProcessCount", renderer_count);
   UMA_HISTOGRAM_COUNTS_100("Memory.WorkerProcessCount", worker_count);
   // TODO(viettrungluu): Do we want separate counts for the other
   // (platform-specific) process types?
 
-  int total_sample = static_cast<int>(aggregate_memory / 1000);
-  UMA_HISTOGRAM_MEMORY_MB("Memory.Total", total_sample);
+  // TODO(rkaplow): Remove once we've verified Memory.Total2 is ok.
+  int total_sample_old = static_cast<int>(aggregate_memory / 1000);
+  UMA_HISTOGRAM_MEMORY_MB("Memory.Total", total_sample_old);
+  int total_sample = static_cast<int>(aggregate_memory / 1024);
+  UMA_HISTOGRAM_MEMORY_LARGE_MB("Memory.Total2", total_sample);
 
 #if defined(OS_CHROMEOS)
   UpdateSwapHistograms();
 #endif
 }
 
 #if defined(OS_CHROMEOS)
 void MetricsMemoryDetails::UpdateSwapHistograms() {
   UMA_HISTOGRAM_BOOLEAN("Memory.Swap.HaveSwapped", swap_info().num_writes > 0);
   if (swap_info().num_writes == 0)
@@ skipping 55 matching lines @@
         continue;
       case PROCESS_TYPE_NACL_BROKER:
         UMA_HISTOGRAM_MEMORY_KB("Memory.Swap.NativeClientBroker", sample);
         continue;
       default:
         NOTREACHED();
         continue;
     }
   }
 
-  int total_sample = static_cast<int>(aggregate_memory / 1000);
-  UMA_HISTOGRAM_MEMORY_MB("Memory.Swap.Total", total_sample);
+  // TODO(rkaplow): Remove once we've verified Memory.Swap.Total2 is ok.
+  int total_sample_old = static_cast<int>(aggregate_memory / 1000);
+  UMA_HISTOGRAM_MEMORY_MB("Memory.Swap.Total", total_sample_old);
+  int total_sample = static_cast<int>(aggregate_memory / 1024);
+  UMA_HISTOGRAM_MEMORY_LARGE_MB("Memory.Swap.Total2", total_sample);
 
   UMA_HISTOGRAM_CUSTOM_COUNTS("Memory.Swap.CompressedDataSize",
                               swap_info().compr_data_size / (1024 * 1024), 1,
                               4096, 50);
   UMA_HISTOGRAM_CUSTOM_COUNTS("Memory.Swap.OriginalDataSize",
                               swap_info().orig_data_size / (1024 * 1024), 1,
                               4096, 50);
   UMA_HISTOGRAM_CUSTOM_COUNTS("Memory.Swap.MemUsedTotal",
                               swap_info().mem_used_total / (1024 * 1024), 1,
                               4096, 50);
   UMA_HISTOGRAM_CUSTOM_COUNTS("Memory.Swap.NumReads", swap_info().num_reads, 1,
                               100000000, 100);
   UMA_HISTOGRAM_CUSTOM_COUNTS("Memory.Swap.NumWrites", swap_info().num_writes,
                               1, 100000000, 100);
 
   if (swap_info().orig_data_size > 0 && swap_info().compr_data_size > 0) {
     UMA_HISTOGRAM_CUSTOM_COUNTS(
         "Memory.Swap.CompressionRatio",
         swap_info().orig_data_size / swap_info().compr_data_size, 1, 20, 20);
   }
 }
 #endif  // defined(OS_CHROMEOS)