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

Unified Diff: components/history/core/browser/top_sites.h

Issue 2567043002: Cleanup: Remove unused SearchTabHelper::GetOpenUrls (Closed)
Patch Set: more 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: components/history/core/browser/top_sites.h
diff --git a/components/history/core/browser/top_sites.h b/components/history/core/browser/top_sites.h
index 4ac6f97f6db496c0062f74f26d129be79494014a..76039434e25e0950562390305c3c332ff200d0c9 100644
--- a/components/history/core/browser/top_sites.h
+++ b/components/history/core/browser/top_sites.h
@@ -65,12 +65,6 @@ class TopSites : public RefcountedKeyedService {
const gfx::Image& thumbnail,
const ThumbnailScore& score) = 0;
- // While testing the history system, we want to set the thumbnail to a piece
- // of static memory.
- virtual bool SetPageThumbnailToJPEGBytes(const GURL& url,
- const base::RefCountedMemory* memory,
- const ThumbnailScore& score) = 0;
-
typedef base::Callback<void(const MostVisitedURLList&)>
GetMostVisitedURLsCallback;
@@ -128,20 +122,10 @@ class TopSites : public RefcountedKeyedService {
// Clear the blacklist. Should be called from the UI thread.
virtual void ClearBlacklistedURLs() = 0;
- // Query history service for the list of available thumbnails. Returns the
- // task id for the request, or |base::CancelableTaskTracker::kBadTaskId| if a
- // request could not be made. Public only for testing purposes.
- virtual base::CancelableTaskTracker::TaskId StartQueryForMostVisited() = 0;
-
// Returns true if the given URL is known to the top sites service.
// This function also returns false if TopSites isn't loaded yet.
virtual bool IsKnownURL(const GURL& url) = 0;
- // Follows the cached redirect chain to convert any URL to its
- // canonical version. If no redirect chain is known for the URL,
- // return it without modification.
- virtual const std::string& GetCanonicalURLString(const GURL& url) const = 0;
-
// Returns true if the top sites list of non-forced URLs is full (i.e. we
// already have the maximum number of non-forced top sites). This function
// also returns false if TopSites isn't loaded yet.

Powered by Google App Engine
This is Rietveld 408576698