Local salient image storage for enhanced bookmark experiment.

components/bookmarks_enhanced/image_store.h

+// Copyright 2014 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_BOOKMARKS_ENHANCED_IMAGE_STORE_H_
+#define COMPONENTS_BOOKMARKS_ENHANCED_IMAGE_STORE_H_
+
+#include <map>
+#include <vector>
+
+#include "base/memory/ref_counted.h"
+#include "base/memory/scoped_ptr.h"
+#include "base/threading/non_thread_safe.h"
+#include "ui/gfx/image/image.h"
+
+class GURL;
+
+// The ImageStore keeps an image for each URL.
+class ImageStore : public base::NonThreadSafe,

[tfarina, 2014/05/05 17:42:59]

No, do not use NonThreadSafe. Instead, could you change this to have a member
variable of base::ThreadChecker (composition is generally preferred over
inheritance)?

Please, refer to lines 56-60 from non_thread_safe.h header for my above comment.

Thanks,

[Kibeom Kim (inactive), 2014/05/05 17:55:50]

Done. Thanks!

+                   public base::RefCounted<ImageStore> {
+ public:
+  ImageStore();
+
+  // Returns true if there is an image for this url.
+  virtual bool HasKey(const GURL& page_url) = 0;
+
+  // Inserts an image and its url in the store for the the given page url. The
+  // image can be null indicating that the download of the image at this URL
+  // failed previously.
+  virtual void Insert(const GURL& page_url,
+                      const GURL& image_url,
+                      const gfx::Image& image) = 0;
+
+  // Removes an image from the store.
+  virtual void Erase(const GURL& page_url) = 0;
+
+  // Returns the image associated with this url. Returns nil if there are no
+  // image for this url. It also returns the image_url where the image was
+  // downloaded from or failed to be downloaded from.
+  virtual gfx::Image Get(const GURL& page_url, GURL* image_url) = 0;
+
+  // Returns the size of the image stored for this URL or empty size if no
+  // images are present.
+  virtual gfx::Size GetSize(const GURL& page_url) = 0;
+
+  // Populates |url| with  all the urls that have an image in the store.
+  virtual void GetAllPageUrls(std::vector<GURL>* urls) = 0;
+
+  // Moves an image from one url to another.
+  virtual void ChangeImageURL(const GURL& from, const GURL& to);
+
+  // Removes all images.
+  void ClearAll();
+
+ protected:
+  virtual ~ImageStore();
+
+ private:
+  friend class base::RefCounted<ImageStore>;
+};
+
+#endif  // COMPONENTS_BOOKMARKS_ENHANCED_IMAGE_STORE_H_