Restrict salient image size before storing

chrome/browser/enhanced_bookmarks/android/bookmark_image_service_android.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 CHROME_BROWSER_ENHANCED_BOOKMARKS_ANDROID_BOOKMARK_IMAGE_SERVICE_ANDROID_H_
 #define CHROME_BROWSER_ENHANCED_BOOKMARKS_ANDROID_BOOKMARK_IMAGE_SERVICE_ANDROID_H_
 
 #include "components/enhanced_bookmarks/bookmark_image_service.h"
 #include "chrome/browser/bitmap_fetcher/bitmap_fetcher.h"
 
 namespace chrome {
 class BitmapFetcher;
 }
 
 namespace content {
 class BrowserContext;
 class RenderFrameHost;
 class WebContents;
 }
 
 namespace enhanced_bookmarks {
 
 class BookmarkImageServiceAndroid : public BookmarkImageService {
  public:
-  explicit BookmarkImageServiceAndroid(content::BrowserContext* browserContext);
+  explicit BookmarkImageServiceAndroid(
+      content::BrowserContext* browserContext,
+      scoped_refptr<base::SequencedWorkerPool> pool);
+
+  ~BookmarkImageServiceAndroid() override;
 
   void RetrieveSalientImage(const GURL& page_url,
                             const GURL& image_url,
                             const std::string& referrer,
                             net::URLRequest::ReferrerPolicy referrer_policy,
                             bool update_bookmark) override;
 
   // Searches the current page for a salient image, if a url is found the image
   // is fetched and stored.
   void RetrieveSalientImageFromContext(
       content::RenderFrameHost* render_frame_host,
       const GURL& page_url,
       bool update_bookmark);
 
   // Investigates if the tab points to a bookmarked url in needs of an updated
   // image. If it is, invokes RetrieveSalientImageFromContext() for the relevant
   // urls.
   void FinishSuccessfulPageLoadForTab(content::WebContents* web_contents,
                                       bool update_bookmark);
 
  private:
   void RetrieveSalientImageFromContextCallback(const GURL& page_url,
                                                bool update_bookmark,
                                                const base::Value* result);
 
+  // Resizes the image, if it is larger than device display.
+  gfx::Image ResizeImage(gfx::Image image);
+
   content::BrowserContext* browser_context_;
   // The script injected in a page to extract image urls.
   base::string16 script_;
+  scoped_refptr<base::SequencedWorkerPool> pool_;
+  scoped_ptr<gfx::Size> max_size_;

[Kibeom Kim (inactive), 2015/02/12 23:24:15]

looks like this doesn't have to be scoped_ptr? (Also since ios will be doing it
too, I think this can just go BookmarkImageService. But maybe in a follow-up ios
CL)

[Kibeom Kim (inactive), 2015/02/13 05:08:53]

ping?

 
   class BitmapFetcherHandler : private chrome::BitmapFetcherDelegate {
    public:
     explicit BitmapFetcherHandler(BookmarkImageServiceAndroid* service,
                                   const GURL& image_url)
         : service_(service), bitmap_fetcher_(image_url, this) {}
     void Start(content::BrowserContext* browser_context,
                const std::string& referrer,
                net::URLRequest::ReferrerPolicy referrer_policy,
                int load_flags,
@@ skipping 12 matching lines @@
 
     DISALLOW_COPY_AND_ASSIGN(BitmapFetcherHandler);
   };
 
   DISALLOW_COPY_AND_ASSIGN(BookmarkImageServiceAndroid);
 };
 
 }  // namespace enhanced_bookmarks
 
 #endif  // CHROME_BROWSER_ENHANCED_BOOKMARKS_ANDROID_BOOKMARK_IMAGE_SERVICE_ANDROID_H_