Desktop NTP metrics: Use ntp_tiles::metrics:: functions

chrome/browser/ui/webui/ntp/ntp_user_data_logger.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 "chrome/browser/ui/webui/ntp/ntp_user_data_logger.h"
 
 #include <algorithm>
 #include <string>
 
 #include "base/metrics/histogram_macros.h"
 #include "base/strings/stringprintf.h"
 #include "chrome/browser/after_startup_task_utils.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/search/search.h"
 #include "chrome/browser/sync/profile_sync_service_factory.h"
 #include "chrome/common/search/search_urls.h"
 #include "chrome/common/url_constants.h"
 #include "components/browser_sync/profile_sync_service.h"
+#include "components/ntp_tiles/metrics.h"
 #include "components/sync_sessions/sessions_sync_manager.h"
 #include "components/sync_sessions/sync_sessions_metrics.h"
 #include "content/public/browser/navigation_details.h"
 #include "content/public/browser/navigation_entry.h"
 #include "content/public/browser/user_metrics.h"
 #include "content/public/browser/web_contents.h"
 
 namespace {
 
-// Name of the histogram keeping track of suggestion impressions.
-const char kMostVisitedImpressionHistogramName[] =
-    "NewTabPage.SuggestionsImpression";
-
-// Format string to generate the name for the histogram keeping track of
-// suggestion impressions.
-const char kMostVisitedImpressionHistogramWithProvider[] =
-    "NewTabPage.SuggestionsImpression.%s";
-
-// Name of the histogram keeping track of suggestion navigations.
-const char kMostVisitedNavigationHistogramName[] =
-    "NewTabPage.MostVisited";
-
-// Format string to generate the name for the histogram keeping track of
-// suggestion navigations.
-const char kMostVisitedNavigationHistogramWithProvider[] =
-    "NewTabPage.MostVisited.%s";
-
-std::string GetSourceName(NTPLoggingTileSource tile_source) {
-  switch (tile_source) {
-    case NTPLoggingTileSource::CLIENT:
-      return "client";
-    case NTPLoggingTileSource::SERVER:
-      return "server";
-  }
-  NOTREACHED();
-  return std::string();
-}
-
 void RecordSyncSessionMetrics(content::WebContents* contents) {
   if (!contents)
     return;
   browser_sync::ProfileSyncService* sync =
       ProfileSyncServiceFactory::GetForProfile(
           Profile::FromBrowserContext(contents->GetBrowserContext()));
   if (!sync)
     return;
   sync_sessions::SessionsSyncManager* sessions =
       static_cast<sync_sessions::SessionsSyncManager*>(
           sync->GetSessionsSyncableService());
   sync_sessions::SyncSessionsMetrics::RecordYoungestForeignTabAgeOnNTP(
       sessions);
 }
 
+ntp_tiles::NTPTileSource ConvertTileSource(NTPLoggingTileSource tile_source) {
+  switch (tile_source) {
+    case NTPLoggingTileSource::CLIENT:
+      return ntp_tiles::NTPTileSource::TOP_SITES;
+    case NTPLoggingTileSource::SERVER:
+      return ntp_tiles::NTPTileSource::SUGGESTIONS_SERVICE;
+  }
+  NOTREACHED();
+  return ntp_tiles::NTPTileSource::TOP_SITES;
+}
+
 }  // namespace
 
 DEFINE_WEB_CONTENTS_USER_DATA_KEY(NTPUserDataLogger);
 
 
 // Log a time event for a given |histogram| at a given |value|. This
 // routine exists because regular histogram macros are cached thus can't be used
 // if the name of the histogram will change at a given call site.
 void LogLoadTimeHistogram(const std::string& histogram, base::TimeDelta value) {
   base::HistogramBase* counter = base::Histogram::FactoryTimeGet(
@@ skipping 53 matching lines @@
 
   switch (tile_source) {
     case NTPLoggingTileSource::CLIENT:
       has_client_side_suggestions_ = true;
       break;
     case NTPLoggingTileSource::SERVER:
       has_server_side_suggestions_ = true;
       break;
   }
 
-  UMA_HISTOGRAM_ENUMERATION(kMostVisitedImpressionHistogramName, position,
-                            kNumMostVisited);
-
-  // Cannot rely on UMA histograms macro because the name of the histogram is
-  // generated dynamically.
-  base::HistogramBase* counter = base::LinearHistogram::FactoryGet(
-      base::StringPrintf(kMostVisitedImpressionHistogramWithProvider,
-                         GetSourceName(tile_source).c_str()),
-      1,
-      kNumMostVisited,
-      kNumMostVisited + 1,
-      base::Histogram::kUmaTargetedHistogramFlag);
-  counter->Add(position);
+  ntp_tiles::metrics::RecordTileImpression(position,
+                                           ConvertTileSource(tile_source));
 }
 
 void NTPUserDataLogger::LogMostVisitedNavigation(
     int position, NTPLoggingTileSource tile_source) {
-  UMA_HISTOGRAM_ENUMERATION(kMostVisitedNavigationHistogramName, position,
-                            kNumMostVisited);
-
-  // Cannot rely on UMA histograms macro because the name of the histogram is
-  // generated dynamically.
-  base::HistogramBase* counter = base::LinearHistogram::FactoryGet(
-      base::StringPrintf(kMostVisitedNavigationHistogramWithProvider,
-                         GetSourceName(tile_source).c_str()),
-      1,
-      kNumMostVisited,
-      kNumMostVisited + 1,
-      base::Histogram::kUmaTargetedHistogramFlag);
-  counter->Add(position);
+  ntp_tiles::metrics::RecordTileClick(position, ConvertTileSource(tile_source),
+                                      ntp_tiles::metrics::THUMBNAIL);
 
   // Records the action. This will be available as a time-stamped stream
   // server-side and can be used to compute time-to-long-dwell.
   content::RecordAction(base::UserMetricsAction("MostVisited_Clicked"));
 }
 
 NTPUserDataLogger::NTPUserDataLogger(content::WebContents* contents)
     : content::WebContentsObserver(contents),
       has_server_side_suggestions_(false),
       has_client_side_suggestions_(false),
@@ skipping 27 matching lines @@
 
 void NTPUserDataLogger::EmitNtpStatistics(base::TimeDelta load_time) {
   // We only send statistics once per page.
   if (has_emitted_)
     return;
 
   size_t number_of_tiles = impression_was_logged_.count();
   DVLOG(1) << "Emitting NTP load time: " << load_time << ", "
            << "number of tiles: " << number_of_tiles;
 
+  ntp_tiles::metrics::RecordPageImpression(number_of_tiles);
+
   LogLoadTimeHistogram("NewTabPage.LoadTime", load_time);
 
   // Split between ML and MV.
   std::string type = has_server_side_suggestions_ ?
       "MostLikely" : "MostVisited";
   LogLoadTimeHistogram("NewTabPage.LoadTime." + type, load_time);
+
   // Split between Web and Local.
   std::string source = ntp_url_.SchemeIsHTTPOrHTTPS() ? "Web" : "LocalNTP";
   LogLoadTimeHistogram("NewTabPage.LoadTime." + source, load_time);
 
   // Split between Startup and non-startup.
   std::string status = during_startup_ ? "Startup" : "NewTab";
   LogLoadTimeHistogram("NewTabPage.LoadTime." + status, load_time);
 
-  UMA_HISTOGRAM_CUSTOM_COUNTS(
-      "NewTabPage.NumberOfTiles", number_of_tiles, 1, kNumMostVisited,
-      kNumMostVisited + 1);
   has_emitted_ = true;
   during_startup_ = false;
 }