Cleanup: Remove unused SearchTabHelper::GetOpenUrls

components/history/core/browser/top_sites.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef COMPONENTS_HISTORY_CORE_BROWSER_TOP_SITES_H_
 #define COMPONENTS_HISTORY_CORE_BROWSER_TOP_SITES_H_
 
 #include <vector>
 
 #include "base/callback.h"
 #include "base/macros.h"
 #include "base/observer_list.h"
 #include "base/task/cancelable_task_tracker.h"

[msw, 2016/12/13 22:31:53]

nit: remove (move to impl)

[Marc Treib, 2016/12/14 11:12:22]

Done. (impl already included it)

 #include "components/history/core/browser/history_types.h"
 #include "components/history/core/browser/top_sites_observer.h"
 #include "components/history/core/common/thumbnail_score.h"
 #include "components/keyed_service/core/refcounted_keyed_service.h"
 #include "third_party/skia/include/core/SkColor.h"
 #include "ui/gfx/image/image.h"
 
 class GURL;
 
 namespace base {
@@ skipping 34 matching lines @@
   TopSites();
 
   // Sets the given thumbnail for the given URL. Returns true if the thumbnail
   // was updated. False means either the URL wasn't known to us, or we felt
   // that our current thumbnail was superior to the given one. Should be called
   // from the UI thread.
   virtual bool SetPageThumbnail(const GURL& url,
                                 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;
 
   // Returns a list of most visited URLs via a callback, if
   // |include_forced_urls| is false includes only non-forced URLs. This may be
   // invoked on any thread. NOTE: the callback is called immediately if we have
   // the data cached. If data is not available yet, callback will later be
   // posted to the thread called this function.
   virtual void GetMostVisitedURLs(const GetMostVisitedURLsCallback& callback,
                                   bool include_forced_urls) = 0;
@@ skipping 37 matching lines @@
   // Removes a URL from the blacklist. Should be called from the UI thread.
   virtual void RemoveBlacklistedURL(const GURL& url) = 0;
 
   // Returns true if the URL is blacklisted. Should be called from the UI
   // thread.
   virtual bool IsBlacklisted(const GURL& url) = 0;
 
   // 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.
   virtual bool IsNonForcedFull() = 0;
 
   // Returns true if the top sites list of forced URLs is full (i.e. we already
   // have the maximum number of forced top sites).  This function also returns
   // false if TopSites isn't loaded yet.
   virtual bool IsForcedFull() = 0;
 
@@ skipping 27 matching lines @@
   friend class base::RefCountedThreadSafe<TopSites>;
 
   base::ObserverList<TopSitesObserver, true> observer_list_;
 
   DISALLOW_COPY_AND_ASSIGN(TopSites);
 };
 
 }  // namespace history
 
 #endif  // COMPONENTS_HISTORY_CORE_BROWSER_TOP_SITES_H_