Index: chrome/browser/ui/webui/ntp/ntp_user_data_logger.cc |
diff --git a/chrome/browser/ui/webui/ntp/ntp_user_data_logger.cc b/chrome/browser/ui/webui/ntp/ntp_user_data_logger.cc |
index 3ca9c8412b0fbd488a6e9191c96044298d022384..b7928842d2cca9556b1f6f5f81a952558463f0b3 100644 |
--- a/chrome/browser/ui/webui/ntp/ntp_user_data_logger.cc |
+++ b/chrome/browser/ui/webui/ntp/ntp_user_data_logger.cc |
@@ -98,19 +98,21 @@ void NTPUserDataLogger::LogEvent(NTPLoggingEventType event, |
void NTPUserDataLogger::LogMostVisitedImpression( |
int position, |
- ntp_tiles::TileSource tile_source) { |
+ ntp_tiles::TileSource tile_source, |
+ ntp_tiles::TileVisualType tile_type) { |
if ((position >= kNumMostVisited) || impression_was_logged_[position]) { |
return; |
} |
impression_was_logged_[position] = true; |
impression_tile_source_[position] = tile_source; |
+ impression_tile_type_[position] = tile_type; |
} |
void NTPUserDataLogger::LogMostVisitedNavigation( |
int position, |
- ntp_tiles::TileSource tile_source) { |
- ntp_tiles::metrics::RecordTileClick(position, tile_source, |
- ntp_tiles::TileVisualType::THUMBNAIL); |
+ ntp_tiles::TileSource tile_source, |
+ ntp_tiles::TileVisualType tile_type) { |
+ ntp_tiles::metrics::RecordTileClick(position, tile_source, tile_type); |
// Records the action. This will be available as a time-stamped stream |
// server-side and can be used to compute time-to-long-dwell. |
@@ -120,6 +122,7 @@ void NTPUserDataLogger::LogMostVisitedNavigation( |
NTPUserDataLogger::NTPUserDataLogger(content::WebContents* contents) |
: content::WebContentsObserver(contents), |
impression_tile_source_(kNumMostVisited), |
+ impression_tile_type_(kNumMostVisited), |
has_emitted_(false), |
during_startup_(!AfterStartupTaskUtils::IsBrowserStartupComplete()) { |
// We record metrics about session data here because when this class typically |
@@ -168,8 +171,8 @@ void NTPUserDataLogger::EmitNtpStatistics(base::TimeDelta load_time) { |
} |
// No URL passed since we're not interested in favicon-related Rappor |
// metrics. |
- tiles.emplace_back(impression_tile_source_[i], |
- ntp_tiles::TileVisualType::THUMBNAIL, GURL()); |
+ tiles.emplace_back(impression_tile_source_[i], impression_tile_type_[i], |
+ GURL()); |
} |
// Not interested in Rappor metrics. |