[Webkit-unassigned] [Bug 35072] [Mac] Move Icon::iconForFiles() code to WebChromeClient::iconForFiles().

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=35072





--- Comment #5 from TAMURA, Kent <tkent at chromium.org>  2010-02-18 17:48:52 PST ---
I tried an idea of icon loading via a special URL several months ago.  However
it failed because of some security restrictions.

The current behavior is:
 1. FileChooser calls Icon::iconForFiles()
 2. FileChooser calls ChromeClient::iconForFiles() if the above failed.
Darin and Maciej didn't like such duplicated API situation.

How about changing it to:
 * FileChooser calls just ChromeClient::iconForFiles().
  So, all platforms need to implement ChromeClient::iconForFiles()

* Provide optional sub-API, Icon::iconForFiles().
  Chromium won't implement it.
  ChromeClient::iconForFiles() for other platforms call iconForFiles()

We don't need to move existing iconForFiles() implementations from WebCore in
this idea.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.