The main window use ico from favicon.ico

[chenon]

Hi,
Is it possible to specify the icon of window from the HTML link tag?
<link rel="Shortcut Icon" href="favicon.ico">

Thanks a lot!

[magreenblatt]

There are three separate issues here.

1. Expose favicon change notifications from CEF. This can be done in browser_webview_delegate.[h|cc] by overriding WebFrameClient::didChangeIcons(WebFrame*) and retrieving the icon URL via WebFrame::favIconURL(). Patches to add this functionality welcome.

2. Retrieve the icon once you know the URL. This could be done client-side using the existing CefWebURLRequest interface.

3. Change the client application icon. This would depend on how your client application is written and is beyond the scope of this forum (I would suggest googling it).

Regards,
Marshall

[TalkiToaster]

Would the process of adding favicon support to CEF 3 still be the same as described above?

[magreenblatt]

Would the process of adding favicon support to CEF 3 still be the same as described above?

It is already done, see http://code.google.com/p/chromiumembedd ... ail?id=779

[TalkiToaster]

The patch file attached to the bug suggests I should see an OnUpdateFaveiconURL function inside CefDisplayHandler (just below OnTitleChange), however I see no such function in CEF 3.

[magreenblatt]

My mistake, it was added in CEF1 and not CEF3. The CEF3 implementatin will be similar but you need to identify the Content API callback.

[TalkiToaster]

I'm not at all familiar with the Chrome Content API. That said, I've had a quick look, and a couple of things look like they have potential:

Code: Select all

// Invoked when new FaviconURL candidates are received from the renderer
void WebContentsObserver::DidUpdateFaviconURL

// The favicon data and tracking information. See content::FaviconStatus
const FaviconStatus& NavigationEntry::GetFavicon

Aside from some functions to get the default favicon, these are the only functions that seem to deal with favicons within the public section of the Content API.

[TalkiToaster]

Should I open a new bug for this for CEF 3?

[magreenblatt]

Should I open a new bug for this for CEF 3?

Yes please.

[TalkiToaster]

Done.

http://code.google.com/p/chromiumembedd ... ail?id=964