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

Unified Diff: chrome/browser/ntp_tiles/chrome_popular_sites_factory.h

Issue 2572533002: Add ChromeMostVisitedSitesFactory (Closed)
Patch Set: Use factory for webui 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/ntp_tiles/chrome_popular_sites_factory.h
diff --git a/chrome/browser/android/ntp/popular_sites.h b/chrome/browser/ntp_tiles/chrome_popular_sites_factory.h
similarity index 51%
rename from chrome/browser/android/ntp/popular_sites.h
rename to chrome/browser/ntp_tiles/chrome_popular_sites_factory.h
index 07d37306ead33594c6c5ba696c302dedfe71e162..97c355fcd5218a742dd5f0b066ad98320530f710 100644
--- a/chrome/browser/android/ntp/popular_sites.h
+++ b/chrome/browser/ntp_tiles/chrome_popular_sites_factory.h
@@ -2,28 +2,23 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-#ifndef CHROME_BROWSER_ANDROID_NTP_POPULAR_SITES_H_
-#define CHROME_BROWSER_ANDROID_NTP_POPULAR_SITES_H_
+#ifndef CHROME_BROWSER_NTP_TILES_CHROME_POPULAR_SITES_FACTORY_H_
+#define CHROME_BROWSER_NTP_TILES_CHROME_POPULAR_SITES_FACTORY_H_
#include <memory>
-#include "base/files/file_path.h"
-#include "base/macros.h"
-
class Profile;
namespace ntp_tiles {
class PopularSites;
} // namespace ntp_tiles
-// TODO(sfiera): move to chrome_popular_sites.h
-class ChromePopularSites {
+class ChromePopularSitesFactory {
public:
static std::unique_ptr<ntp_tiles::PopularSites> NewForProfile(
Profile* profile);
- private:
- DISALLOW_IMPLICIT_CONSTRUCTORS(ChromePopularSites);
+ ChromePopularSitesFactory() = delete;
};
-#endif // CHROME_BROWSER_ANDROID_NTP_POPULAR_SITES_H_
+#endif // CHROME_BROWSER_NTP_TILES_CHROME_POPULAR_SITES_FACTORY_H_

Powered by Google App Engine
This is Rietveld 408576698