[Suggestions] Make ThumbnailManager implement ImageManager

chrome/browser/search/suggestions/thumbnail_manager.h

Index: chrome/browser/search/suggestions/thumbnail_manager.h
diff --git a/chrome/browser/search/suggestions/thumbnail_manager.h b/chrome/browser/search/suggestions/thumbnail_manager.h
index dab023766cb828acae2964a8c6998ff8740e25d1..8974057cdbc07ebbe97fc5487fe098621ca8219d 100644
--- a/chrome/browser/search/suggestions/thumbnail_manager.h
+++ b/chrome/browser/search/suggestions/thumbnail_manager.h
@@ -15,6 +15,7 @@
 #include "base/memory/scoped_ptr.h"
 #include "base/memory/weak_ptr.h"
 #include "chrome/browser/bitmap_fetcher/bitmap_fetcher.h"
+#include "chrome/browser/search/suggestions/image_manager.h"
 #include "chrome/browser/search/suggestions/proto/suggestions.pb.h"
 #include "components/leveldb_proto/proto_database.h"
 #include "ui/gfx/image/image_skia.h"
@@ -31,7 +32,8 @@ class ThumbnailData;
 
 // A class used to fetch server thumbnails asynchronously and manage the caching
 // layer (both in memory and on disk).
-class ThumbnailManager : public chrome::BitmapFetcherDelegate {
+class ThumbnailManager : public ImageManager,
+                         public chrome::BitmapFetcherDelegate {
  public:
   typedef std::vector<ThumbnailData> ThumbnailVector;
 
@@ -43,23 +45,23 @@ class ThumbnailManager : public chrome::BitmapFetcherDelegate {
 
   // Initializes the |thumbnail_url_map_| with the proper mapping from website

[huangs, 2014/07/17 18:23:35]

Add something similar to
// ImageManager interface.
Update comment to match image_manager.h's, or simply remove it.

[Mathieu, 2014/07/17 18:29:13]

Done.

   // URL to thumbnail URL.
-  void InitializeThumbnailMap(const SuggestionsProfile& suggestions);
+  virtual void Initialize(const SuggestionsProfile& suggestions) OVERRIDE;
 
   // Retrieves stored thumbnail for website |url| asynchronously. Calls
   // |callback| with Bitmap pointer if found, and NULL otherwise. Should be
   // called from the UI thread.
-  void GetPageThumbnail(
+  virtual void GetImageForURL(

[huangs, 2014/07/17 18:23:35]

// ImageManager interface.

The UI thread constraint departs from the base classes.  If it's a property of
this class, then comment as such.  Otherwise leave things the way they are.

[Mathieu, 2014/07/17 18:29:13]

Done.

       const GURL& url,
-      base::Callback<void(const GURL&, const SkBitmap*)> callback);
+      base::Callback<void(const GURL&, const SkBitmap*)> callback) OVERRIDE;
 
  private:
   friend class MockThumbnailManager;
   friend class ThumbnailManagerBrowserTest;
-  FRIEND_TEST_ALL_PREFIXES(ThumbnailManagerTest, InitializeThumbnailMapTest);
+  FRIEND_TEST_ALL_PREFIXES(ThumbnailManagerTest, InitializeTest);
   FRIEND_TEST_ALL_PREFIXES(ThumbnailManagerBrowserTest,
-                           GetPageThumbnailNetworkCacheHit);
+                           GetImageForURLNetworkCacheHit);
   FRIEND_TEST_ALL_PREFIXES(ThumbnailManagerBrowserTest,
-                           GetPageThumbnailNetworkCacheNotInitialized);
+                           GetImageForURLNetworkCacheNotInitialized);
 
   // Used for testing.
   ThumbnailManager();