Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(98)

Unified Diff: chrome/browser/ui/webui/ntp/ntp_user_data_logger.h

Issue 2544883006: Cleanup: Replace NTPLoggingTileSource by ntp_tiles::NTPTileSource (Closed)
Patch Set: tests Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/webui/ntp/ntp_user_data_logger.h
diff --git a/chrome/browser/ui/webui/ntp/ntp_user_data_logger.h b/chrome/browser/ui/webui/ntp/ntp_user_data_logger.h
index 17c28611321ef2a715b1696e5856429ab89f92be..59e967c25d91cab6b943cac45323427e37fa0e05 100644
--- a/chrome/browser/ui/webui/ntp/ntp_user_data_logger.h
+++ b/chrome/browser/ui/webui/ntp/ntp_user_data_logger.h
@@ -13,6 +13,7 @@
#include "base/macros.h"
#include "base/time/time.h"
#include "chrome/common/search/ntp_logging_events.h"
+#include "components/ntp_tiles/ntp_tile_source.h"
#include "content/public/browser/web_contents_observer.h"
#include "content/public/browser/web_contents_user_data.h"
@@ -39,10 +40,12 @@ class NTPUserDataLogger
void LogEvent(NTPLoggingEventType event, base::TimeDelta time);
// Logs an impression on one of the NTP tiles by a given source.
- void LogMostVisitedImpression(int position, NTPLoggingTileSource tile_source);
+ void LogMostVisitedImpression(int position,
+ ntp_tiles::NTPTileSource tile_source);
// Logs a navigation on one of the NTP tiles by a given source.
- void LogMostVisitedNavigation(int position, NTPLoggingTileSource tile_source);
+ void LogMostVisitedNavigation(int position,
+ ntp_tiles::NTPTileSource tile_source);
protected:
explicit NTPUserDataLogger(content::WebContents* contents);
@@ -80,7 +83,7 @@ class NTPUserDataLogger
// Stores the tile source for each impression. Entries are only valid if the
// corresponding entry in |impression_was_logged_| is true.
- std::vector<NTPLoggingTileSource> impression_tile_source_;
+ std::vector<ntp_tiles::NTPTileSource> impression_tile_source_;
// Whether we have already emitted NTP stats for this web contents.
bool has_emitted_;
« no previous file with comments | « chrome/browser/ui/search/search_tab_helper_unittest.cc ('k') | chrome/browser/ui/webui/ntp/ntp_user_data_logger.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698