Fix crashes due to gfx::Image unsafe thread passing

components/enhanced_bookmarks/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_ENHANCED_BOOKMARKS_IMAGE_STORE_H_
 #define COMPONENTS_ENHANCED_BOOKMARKS_IMAGE_STORE_H_
 
 #include <set>
 
 #include "base/sequence_checker.h"
 #include "components/enhanced_bookmarks/image_record.h"
 #include "ui/gfx/geometry/size.h"
 
 class GURL;
 
 // The ImageStore keeps an image for each URL.  This class is not thread safe,
 // and will check the thread using base::ThreadChecker, except the constructor.
 class ImageStore {
  public:
   ImageStore();
   virtual ~ImageStore();
 
   // Returns true if there is an image for this url.
   virtual bool HasKey(const GURL& page_url) = 0;
 
   // Inserts an ImageRecord in the store for the given page url. The image can
   // be null indicating that the download of the image at this URL or
   // encoding for insertion failed previously. On non-iOS platforms, |image|
   // must have exactly one representation with a scale factor of 1.
-  virtual void Insert(const GURL& page_url,
-                      const enhanced_bookmarks::ImageRecord& image_record) = 0;
+  virtual void Insert(
+      const GURL& page_url,
+      scoped_refptr<enhanced_bookmarks::ImageRecord> image_record) = 0;
 
   // Removes an image from the store.
   virtual void Erase(const GURL& page_url) = 0;
 
   // Returns the image associated with this url. Returns an ImageRecord with an
   // empty image if there is no image for this url. It also returns the
   // image_url where the image was downloaded from or failed to be downloaded
   // from. When the image is not empty, the dominant color of the image is also
   // filled.
-  virtual enhanced_bookmarks::ImageRecord Get(const GURL& page_url) = 0;
+  virtual scoped_refptr<enhanced_bookmarks::ImageRecord> Get(
+      const GURL& page_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 |urls| with all the urls that have an image in the store.
   virtual void GetAllPageUrls(std::set<GURL>* urls) = 0;
 
   // Removes all images.
   virtual void ClearAll() = 0;
 
   // Moves an image from one url to another.
   void ChangeImageURL(const GURL& from, const GURL& to);
 
   // Returns the saved images storage size in bytes. If the storage doesn't
   // exist yet or failed to read, returns -1.
   virtual int64 GetStoreSizeInBytes() = 0;
 
  protected:
   base::SequenceChecker sequence_checker_;
 
  private:
   DISALLOW_COPY_AND_ASSIGN(ImageStore);
 };
 
 #endif  // COMPONENTS_ENHANCED_BOOKMARKS_IMAGE_STORE_H_