Remove histograms NewTabPage.IconsReal/IconsColor/IconsGray

components/ntp_tiles/metrics.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 "components/ntp_tiles/metrics.h"
 
 #include <string>
 
 #include "base/metrics/histogram.h"
 #include "base/metrics/histogram_macros.h"
@@ skipping 64 matching lines @@
   }
   return nullptr;
 }
 
 }  // namespace
 
 void RecordPageImpression(const std::vector<TileImpression>& tiles,
                           rappor::RapporService* rappor_service) {
   UMA_HISTOGRAM_SPARSE_SLOWLY("NewTabPage.NumberOfTiles", tiles.size());
 
-  int counts_per_type[NUM_RECORDED_TILE_TYPES] = {0};
-  bool have_tile_types = false;
   for (int index = 0; index < static_cast<int>(tiles.size()); index++) {
     NTPTileSource source = tiles[index].source;
     MostVisitedTileType tile_type = tiles[index].type;
     const GURL& url = tiles[index].url;
 
     UMA_HISTOGRAM_ENUMERATION("NewTabPage.SuggestionsImpression", index,
                               kMaxNumTiles);
 
     std::string source_name = GetSourceHistogramName(source);
     std::string impression_histogram = base::StringPrintf(
         "NewTabPage.SuggestionsImpression.%s", source_name.c_str());
     LogHistogramEvent(impression_histogram, index, kMaxNumTiles);
 
     if (tile_type >= NUM_RECORDED_TILE_TYPES) {
       continue;
     }
 
-    have_tile_types = true;
-    ++counts_per_type[tile_type];
-
     UMA_HISTOGRAM_ENUMERATION("NewTabPage.TileType", tile_type,
                               NUM_RECORDED_TILE_TYPES);
 
     std::string tile_type_histogram =
         base::StringPrintf("NewTabPage.TileType.%s", source_name.c_str());
     LogHistogramEvent(tile_type_histogram, tile_type, NUM_RECORDED_TILE_TYPES);
 
     const char* icon_type_suffix = GetIconTypeSuffix(tile_type);
     if (icon_type_suffix) {
       rappor::SampleDomainAndRegistryFromGURL(
           rappor_service,
           base::StringPrintf("NTP.SuggestionsImpressions.%s", icon_type_suffix),
           url);
 
       std::string icon_impression_histogram = base::StringPrintf(
           "NewTabPage.SuggestionsImpression.%s", icon_type_suffix);
       LogHistogramEvent(icon_impression_histogram, index, kMaxNumTiles);
     }
   }
-
-  if (have_tile_types) {
-    UMA_HISTOGRAM_SPARSE_SLOWLY("NewTabPage.IconsReal",
-                                counts_per_type[ICON_REAL]);
-    UMA_HISTOGRAM_SPARSE_SLOWLY("NewTabPage.IconsColor",
-                                counts_per_type[ICON_COLOR]);
-    UMA_HISTOGRAM_SPARSE_SLOWLY("NewTabPage.IconsGray",
-                                counts_per_type[ICON_DEFAULT]);
-  }
 }
 
 void RecordTileClick(int index,
                      NTPTileSource source,
                      MostVisitedTileType tile_type) {
   UMA_HISTOGRAM_ENUMERATION("NewTabPage.MostVisited", index, kMaxNumTiles);
 
   std::string histogram = base::StringPrintf(
       "NewTabPage.MostVisited.%s", GetSourceHistogramName(source).c_str());
   LogHistogramEvent(histogram, index, kMaxNumTiles);
@@ skipping 11 matching lines @@
 
     std::string histogram =
         base::StringPrintf("NewTabPage.TileTypeClicked.%s",
                            GetSourceHistogramName(source).c_str());
     LogHistogramEvent(histogram, tile_type, NUM_RECORDED_TILE_TYPES);
   }
 }
 
 }  // namespace metrics
 }  // namespace ntp_tiles