Applied the IconType.

chrome/browser/favicon_service.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_SERVICE_H__
 #define CHROME_BROWSER_FAVICON_SERVICE_H__
 #pragma once
 
 #include <vector>
 
@@ skipping 20 matching lines @@
   // Callback for GetFavicon. If we have previously inquired about the favicon
   // for this URL, |know_favicon| will be true, and the rest of the fields will
   // be valid (otherwise they will be ignored).
   //
   // On |know_favicon| == true, |data| will either contain the PNG encoded
   // favicon data, or it will be NULL to indicate that the site does not have
   // a favicon (in other words, we know the site doesn't have a favicon, as
   // opposed to not knowing anything). |expired| will be set to true if we
   // refreshed the favicon "too long" ago and should be updated if the page
   // is visited again.
-  typedef Callback5<Handle,                           // handle
+  typedef Callback6<Handle,                           // handle

[sky, 2011/03/09 21:41:08]

6 parameters with 2 booleans is pretty confusing. Put this into a structure.

[michaelbai, 2011/03/09 23:11:45]

To be a structure makes a lot of code change. And 2 booleans were not introduced
in this CL.

Can we leave it as it?

[sky, 2011/03/09 23:34:52]

Is it really that many more code changes than you're already doing? That you had
to create a new Callback6 type is a strong indication that the callback should
be supplied a structure and not a bunch of args.

                     bool,                             // know_favicon
                     scoped_refptr<RefCountedMemory>,  // data
                     bool,                             // expired
-                    GURL>::Type                       // url of the favicon
+                    GURL,                             // url of the favicon
+                    history::IconType>::Type          // the type of favicon
                     FaviconDataCallback;
 
   typedef CancelableRequest<FaviconDataCallback> GetFaviconRequest;
 
-  // Requests the favicon. |consumer| is notified when the bits have been
-  // fetched.
+  // Requests the |icon_type| of favicon. |consumer| is notified when the bits
+  // have been fetched.
   Handle GetFavicon(const GURL& icon_url,
+                    history::IconType icon_type,
                     CancelableRequestConsumerBase* consumer,
                     FaviconDataCallback* callback);
 
-  // Fetches the favicon at |icon_url|, sending the results to the given
-  // |callback|. If the favicon has previously been set via SetFavicon(), then
-  // the favicon URL for |page_url| and all redirects is set to |icon_url|. If
-  // the favicon has not been set, the database is not updated.
+  // Fetches the |icon_type| of favicon at |icon_url|, sending the results to
+  // the given |callback|. If the favicon has previously been set via
+  // SetFavicon(), then the favicon URL for |page_url| and all redirects is set
+  // to |icon_url|. If the favicon has not been set, the database is not
+  // updated.
   Handle UpdateFaviconMappingAndFetch(const GURL& page_url,
                                       const GURL& icon_url,
+                                      history::IconType icon_type,
                                       CancelableRequestConsumerBase* consumer,
                                       FaviconDataCallback* callback);
 
-  // Requests a favicon for a web page URL. |consumer| is notified
-  // when the bits have been fetched.
+  // Requests any |icon_type| of favicon for a web page URL. |consumer| is
+  // notified when the bits have been fetched. |icon_type| could be any

[sky, 2011/03/09 21:41:08]

could->can

[michaelbai, 2011/03/09 23:11:45]

Done.

+  // combination of IconType value, but only one icon will be returned in the
+  // priority of TOUCH_PRECOMPOSED_ICON, TOUCH_ICON and FAV_ICON.
   //
   // Note: this version is intended to be used to retrieve the favicon of a
   // page that has been browsed in the past. |expired| in the callback is
   // always false.
   Handle GetFaviconForURL(const GURL& page_url,
+                          int icon_type,

[sky, 2011/03/09 21:41:08]

icon_types.

[michaelbai, 2011/03/09 23:11:45]

Done.

                           CancelableRequestConsumerBase* consumer,
                           FaviconDataCallback* callback);
 
-  // Marks the favicon for the page as being out of date.
+  // Marks the all type of favicon for the page as being out of date.
   void SetFaviconOutOfDateForPage(const GURL& page_url);
 
   // Allows the importer to set many favicons for many pages at once. The pages
   // must exist, any favicon sets for unknown pages will be discarded. Existing
   // favicons will not be overwritten.
   void SetImportedFavicons(
       const std::vector<history::ImportedFavIconUsage>& favicon_usage);
 
   // Sets the favicon for a page.
   void SetFavicon(const GURL& page_url,
                   const GURL& icon_url,
-                  const std::vector<unsigned char>& image_data);
+                  const std::vector<unsigned char>& image_data,
+                  history::IconType icon_type);
 
  private:
   friend class base::RefCountedThreadSafe<FaviconService>;
 
   ~FaviconService();
 
   Profile* profile_;
 
   // Helper to forward an empty result if we cannot get the history service.
   void ForwardEmptyResultAsync(GetFaviconRequest* request);
 
   DISALLOW_COPY_AND_ASSIGN(FaviconService);
 };
 
 #endif  // CHROME_BROWSER_FAVICON_SERVICE_H__