NEW 78697
Remove the need to have a database for getting the favicon 
https://bugs.webkit.org/show_bug.cgi?id=78697
Summary Remove the need to have a database for getting the favicon
andy.shaw
Reported 2012-02-15 04:54:56 PST
Remove the need to have a database for getting the favicon as in some cases it is not possible for a browser to write to the local system but it would still want to show the favicons. So it would be good if this could still be running without having a database to write to.
Attachments
Add attachment proposed patch, testcase, etc.
Alexey Proskuryakov
Comment 1 2012-02-15 10:47:30 PST
The database can be in-memory, why is this an issue?
andy.shaw
Comment 2 2012-02-15 12:21:41 PST
If it can be in-memory then I missed that and that would be sufficient, can you point me to where I can see that so I can look at it further?
Brady Eidson
Comment 3 2012-02-15 12:28:17 PST
I'm not sure there's currently a way to enable an in-memory icon database, but it should be possible to add a hook to tell sqlite to open an in-memory database instead of opening one on disk. It might be as simple as setting a special in-memory URL for the database path preference, I'm not sure if that's possible as I forget the semantics of SQLite in-memory databases.
andy.shaw
Comment 4 2012-02-15 12:30:58 PST
I am aware of the :memory: approach, but from what I could see it would validate the path on the system, and have its own filename set so it wasn't so straightforward to specify it should use :memory:. Though I would welcome that as being the solution, it would be sufficient enough
Brady Eidson
Comment 5 2012-02-15 13:07:17 PST
What port are you working on, btw? I'm curious about what your situation is where you want in-memory icons.
andy.shaw
Comment 6 2012-02-15 13:14:33 PST
I am actually working with QtWebKit, but my role is actually in the Digia Qt Commercial support team and the request came via something that came up there
Note You need to log in before you can comment on or make changes to this bug.