Provide assumed favicon for intents if service provider page was never visited.

chrome/browser/history/history.h

Index: chrome/browser/history/history.h
diff --git a/chrome/browser/history/history.h b/chrome/browser/history/history.h
index c6f257bbdf75852c49034a9c574e542875eb5431..74b5ce8851121309c46c9c0365fb3faae4ed87f6 100644
--- a/chrome/browser/history/history.h
+++ b/chrome/browser/history/history.h
@@ -666,6 +666,10 @@ class HistoryService : public CancelableRequestProvider,
   // of date.
   void SetFaviconOutOfDateForPage(const GURL& page_url);
 
+  // Used by the FaviconService to clone favicons from one page to another,
+  // provided that other page does not already have favicons.

[Greg Billock, 2011/11/11 17:47:56]

Is the purpose for this a cross-domain mapping? That is, wouldn't we load the
right favicon (speculatively, anyway) for a page we'd never seen, if we knew its
domain's favicon?

[groby-ooo-7-16, 2011/11/14 23:53:36]

Actually, we don't know the domain's favicon. We only know favicons that have
been explicitly associated with a URL in the ThumbnailDatabase.

If no icon is recorded for a given URL, the default icon (the gray globe) is
shown. That's the whole reason for this patch :)

On 2011/11/11 17:47:56, Greg Billock wrote:

+  void CloneFavicon(const GURL& old_page_url, const GURL& new_page_url);
+
   // Used by the FaviconService for importing 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.