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

Unified Diff: components/history/core/test/history_client_fake_bookmarks.cc

Issue 1924773002: TopSites: filter out non-WebSafe URLs (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@topsites_cleanup
Patch Set: comment Created 4 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
Index: components/history/core/test/history_client_fake_bookmarks.cc
diff --git a/components/history/core/test/history_client_fake_bookmarks.cc b/components/history/core/test/history_client_fake_bookmarks.cc
index f251a6faa452845672667a39a928dafe6faf3929..2572f4a2df3fc2c29d50756718c723b287c6eddd 100644
--- a/components/history/core/test/history_client_fake_bookmarks.cc
+++ b/components/history/core/test/history_client_fake_bookmarks.cc
@@ -81,6 +81,7 @@ class HistoryBackendClientFakeBookmarks : public HistoryBackendClient {
bool IsBookmarked(const GURL& url) override;
void GetBookmarks(std::vector<URLAndTitle>* bookmarks) override;
bool ShouldReportDatabaseError() override;
+ bool IsWebSafe(const GURL& url) override;
#if defined(OS_ANDROID)
void OnHistoryBackendInitialized(HistoryBackend* history_backend,
HistoryDatabase* history_database,
@@ -117,6 +118,10 @@ bool HistoryBackendClientFakeBookmarks::ShouldReportDatabaseError() {
return false;
}
+bool HistoryBackendClientFakeBookmarks::IsWebSafe(const GURL& url) {
+ return true;
+}
+
#if defined(OS_ANDROID)
void HistoryBackendClientFakeBookmarks::OnHistoryBackendInitialized(
HistoryBackend* history_backend,
« no previous file with comments | « components/history/core/browser/visitsegment_database.cc ('k') | ios/chrome/browser/history/history_backend_client_impl.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698