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

Unified Diff: chrome/browser/ui/webui/ntp_tiles_internals_ui.cc

Issue 2790463003: ntp_tiles: Cleanup enum names (Closed)
Patch Set: add back histogram enums Created 3 years, 8 months 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
« no previous file with comments | « chrome/browser/ui/webui/ntp/ntp_user_data_logger_unittest.cc ('k') | chrome/common/instant.mojom » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/webui/ntp_tiles_internals_ui.cc
diff --git a/chrome/browser/ui/webui/ntp_tiles_internals_ui.cc b/chrome/browser/ui/webui/ntp_tiles_internals_ui.cc
index d7dbab52acb40278650c12d9d3ba59c825dd20f6..6cb3f876407fb0d7ae636c08d7d6ffc9d6bf6677 100644
--- a/chrome/browser/ui/webui/ntp_tiles_internals_ui.cc
+++ b/chrome/browser/ui/webui/ntp_tiles_internals_ui.cc
@@ -42,7 +42,7 @@ class ChromeNTPTilesInternalsMessageHandlerClient
// ntp_tiles::NTPTilesInternalsMessageHandlerClient
bool SupportsNTPTiles() override;
- bool DoesSourceExist(ntp_tiles::NTPTileSource source) override;
+ bool DoesSourceExist(ntp_tiles::TileSource source) override;
std::unique_ptr<ntp_tiles::MostVisitedSites> MakeMostVisitedSites() override;
PrefService* GetPrefs() override;
void RegisterMessageCallback(
@@ -67,13 +67,13 @@ bool ChromeNTPTilesInternalsMessageHandlerClient::SupportsNTPTiles() {
}
bool ChromeNTPTilesInternalsMessageHandlerClient::DoesSourceExist(
- ntp_tiles::NTPTileSource source) {
+ ntp_tiles::TileSource source) {
switch (source) {
- case ntp_tiles::NTPTileSource::TOP_SITES:
- case ntp_tiles::NTPTileSource::SUGGESTIONS_SERVICE:
- case ntp_tiles::NTPTileSource::WHITELIST:
+ case ntp_tiles::TileSource::TOP_SITES:
+ case ntp_tiles::TileSource::SUGGESTIONS_SERVICE:
+ case ntp_tiles::TileSource::WHITELIST:
return true;
- case ntp_tiles::NTPTileSource::POPULAR:
+ case ntp_tiles::TileSource::POPULAR:
#if defined(OS_ANDROID)
return true;
#else
« no previous file with comments | « chrome/browser/ui/webui/ntp/ntp_user_data_logger_unittest.cc ('k') | chrome/common/instant.mojom » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698