Support touch icon in FaviconHelper

chrome/browser/favicon_helper.h

 // Copyright (c) 2011 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_FAVICON_HELPER_H__
 #define CHROME_BROWSER_FAVICON_HELPER_H__
 #pragma once
 
 #include <map>
 
 #include "base/basictypes.h"
 #include "base/callback.h"
 #include "base/ref_counted.h"
 #include "chrome/browser/favicon_service.h"
+#include "chrome/common/icon_messages.h"
 #include "chrome/common/ref_counted_util.h"
 #include "content/browser/cancelable_request.h"
 #include "content/browser/tab_contents/tab_contents_observer.h"
 #include "googleurl/src/gurl.h"
 
 class NavigationEntry;
 class Profile;
 class RefCountedMemory;
 class SkBitmap;
 class TabContents;
@@ skipping 35 matching lines @@
 // OnFaviconData either updates the favicon of the NavigationEntry (if the
 // db knew about the favicon), or requests the renderer to download the
 // favicon.
 //
 // When the renderer downloads the favicon SetFaviconImageData is invoked,
 // at which point we update the favicon of the NavigationEntry and notify
 // the database to save the favicon.
 
 class FaviconHelper : public TabContentsObserver {
  public:
-  explicit FaviconHelper(TabContents* tab_contents);
+  // |icon_types| defines this FaviconHelper supported icon types. It can be the

[sky, 2011/03/22 19:48:48]

|icon_types| is a bitmask of the supported icon types (history::IconType).

[michaelbai, 2011/03/22 23:59:03]

Done.

+  // combination of icon types. |preferred_icon_size| indicates the preferred
+  // icon size, the preferred icon is fetched from network and converted if the
+  // size is not matched; It can be 0 if there is no preference, the icon from

[sky, 2011/03/22 19:48:48]

; -> . or It -> it

[michaelbai, 2011/03/22 23:59:03]

Done.

+  // network will be left as it.
+  FaviconHelper(TabContents* tab_contents,
+                int icon_types,

[sky, 2011/03/22 19:48:48]

I don't think it makes sense to let folks specify an arbitrary set of types
here. Instead, define an enum in this class:

enum Types {
  FAVICON,
  TOUCH
};

and have the constructor take the enum. It shouldn't take a preferred_icon_size
either, that can be figured out from the type. Internally you'll map the type to
the set of IconTypes.

[michaelbai, 2011/03/22 23:59:03]

Done.

+                int preferred_icon_size);
   virtual ~FaviconHelper();
 
   // Initiates loading the favicon for the specified url.
   void FetchFavicon(const GURL& url);
 
   // Initiates loading an image from given |image_url|. Returns a download id
   // for caller to track the request. When download completes, |callback| is
   // called with the three params: the download_id, a boolean flag to indicate
   // whether the download succeeds and a SkBitmap as the downloaded image.
   // Note that |image_size| is a hint for images with multiple sizes. The
   // downloaded image is not resized to the given image_size. If 0 is passed,
   // the first frame of the image is returned.
   typedef Callback3<int, bool, const SkBitmap&>::Type ImageDownloadCallback;
-  int DownloadImage(const GURL& image_url, int image_size,
+  int DownloadImage(const GURL& image_url,
+                    int image_size,
+                    history::IconType icon_type,
                     ImageDownloadCallback* callback);
 
   // Message Handler.  Must be public, because also called from
   // PrerenderContents.
-  void OnUpdateFaviconURL(int32 page_id, const GURL& icon_url);
+  void OnUpdateFaviconURL(int32 page_id,
+                          std::vector<FaviconURL> candidates);

[sky, 2011/03/22 19:48:48]

const std::vector<>&

[michaelbai, 2011/03/22 23:59:03]

Done.

 
  private:
   struct DownloadRequest {
     DownloadRequest() {}

[sky, 2011/03/22 19:48:48]

initialize icon_type and callback here.

[michaelbai, 2011/03/22 23:59:03]

Done.

     DownloadRequest(const GURL& url,
                     const GURL& image_url,
-                    ImageDownloadCallback* callback)
+                    ImageDownloadCallback* callback,
+                    history::IconType icon_type)
         : url(url),
           image_url(image_url),
-          callback(callback) { }
+          callback(callback),
+          icon_type(icon_type) { }

[sky, 2011/03/22 19:48:48]

{ } -> {}

[michaelbai, 2011/03/22 23:59:03]

Done.

 
     GURL url;
     GURL image_url;
     ImageDownloadCallback* callback;
+    history::IconType icon_type;
   };
 
+  static bool IsSameFaviconURL(const FaviconURL& favicon_url,

[sky, 2011/03/22 19:48:48]

This should be unix_hacker_style. Also, the name is a bit misleading. How about,
do_url_and_icon_match

[michaelbai, 2011/03/22 23:59:03]

Done.

+                               const GURL& url,
+                               history::IconType icon_type) {
+    return favicon_url.icon_url == url &&
+        favicon_url.icon_type == static_cast<IconType>(icon_type);
+  }
+
   // TabContentsObserver implementation.
   virtual bool OnMessageReceived(const IPC::Message& message);
 
   void OnDidDownloadFavicon(int id,
                             const GURL& image_url,
                             bool errored,
                             const SkBitmap& image);
 
   // Return the NavigationEntry for the active entry, or NULL if the active
   // entries URL does not match that of the URL last passed to FetchFavicon.
   NavigationEntry* GetEntry();
 
   Profile* profile();
 
   FaviconService* GetFaviconService();
 
   // See description above class for details.
   void OnFaviconDataForInitialURL(FaviconService::Handle handle,
                                   history::FaviconData favicon);
 
   // If the favicon has expired, asks the renderer to download the favicon.
   // Otherwise asks history to update the mapping between page url and icon
   // url with a callback to OnFaviconData when done.
-  void DownloadFaviconOrAskHistory(NavigationEntry* entry);
+  void DownloadFaviconOrAskHistory(const GURL& page_url,
+                                   const GURL& icon_url,
+                                   history::IconType icon_type);
 
   // See description above class for details.
   void OnFaviconData(FaviconService::Handle handle,
                      history::FaviconData favicon);
 
   // Schedules a download for the specified entry. This adds the request to
   // download_requests_.
-  int ScheduleDownload(const GURL& url, const GURL& image_url, int image_size,
+  int ScheduleDownload(const GURL& url,
+                       const GURL& image_url,
+                       int image_size,
+                       history::IconType icon_type,
                        ImageDownloadCallback* callback);
 
   // Sets the image data for the favicon. This is invoked asynchronously after
   // we request the TabContents to download the favicon.
-  void SetFavicon(const GURL& url, const GURL& icon_url, const SkBitmap& image);
+  void SetFavicon(const GURL& url,
+                  const GURL& icon_url,
+                  const SkBitmap& image,
+                  history::IconType icon_type);
 
-  // Converts the image data to an SkBitmap and sets it on the NavigationEntry.
+  // Converts the FAVICON's image data to an SkBitmap and sets it on the
+  // NavigationEntry.
   // If the TabContents has a delegate, it is notified of the new favicon
   // (INVALIDATE_FAVICON).
   void UpdateFavicon(NavigationEntry* entry,
                      scoped_refptr<RefCountedMemory> data);
   void UpdateFavicon(NavigationEntry* entry, const SkBitmap& image);
 
   // Scales the image such that either the width and/or height is 16 pixels
   // wide. Does nothing if the image is empty.
   SkBitmap ConvertToFaviconSize(const SkBitmap& image);
 
   // Returns true if the favicon should be saved.
   bool ShouldSaveFavicon(const GURL& url);
 
+  // Return the current candidate if any.
+  FaviconURL* candidate() {

[sky, 2011/03/22 19:48:48]

current_candidate()

[michaelbai, 2011/03/22 23:59:03]

Done.

+    return (favicon_candidates_.size() > current_candidate_idx_) ?
+        &favicon_candidates_[current_candidate_idx_] : NULL;
+  }
+
   // Used for history requests.
   CancelableRequestConsumer cancelable_consumer_;
 
   // URL of the page we're requesting the favicon for.
   GURL url_;
 
-  // Whether we got the url for the page back from the renderer.
-  // See "Favicon Details" in tab_contents.cc for more details.
-  bool got_favicon_url_;
-
   // Whether we got the initial response for the favicon back from the renderer.
   // See "Favicon Details" in tab_contents.cc for more details.
   bool got_favicon_from_history_;
 
   // Whether the favicon is out of date. If true, it means history knows about
   // the favicon, but we need to download the favicon because the icon has
   // expired.
   // See "Favicon Details" in tab_contents.cc for more details.
   bool favicon_expired_;
 
   // Requests to the renderer to download favicons.
   typedef std::map<int, DownloadRequest> DownloadRequests;
   DownloadRequests download_requests_;
 
+  // The combination of the supported icon types.
+  const int icon_types_;
+
+  // The prioritized favicon candidates from the page back from the renderer.
+  std::vector<FaviconURL> favicon_candidates_;

[sky, 2011/03/22 19:48:48]

How about naming this urls_ ? If you change it, update current_candidate_idx_
too.

[michaelbai, 2011/03/22 23:59:03]

Done.

+
+  // The current candidate's index in favicon_candidates_.
+  size_t current_candidate_idx_;

[sky, 2011/03/22 19:48:48]

idx -> index

[michaelbai, 2011/03/22 23:59:03]

Done.

+
+  // The FaviconData from history.
+  history::FaviconData history_icon_;
+
+  // The preferred icon size for supported icon types.
+  int preferred_icon_size_;
+
   DISALLOW_COPY_AND_ASSIGN(FaviconHelper);
 };
 
 #endif  // CHROME_BROWSER_FAVICON_HELPER_H__