Bug 78697 - Remove the need to have a database for getting the favicon
Summary: Remove the need to have a database for getting the favicon
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: Page Loading (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Enhancement
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-02-15 04:54 PST by andy.shaw
Modified: 2012-02-15 13:14 PST (History)
2 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description andy.shaw 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.
Comment 1 Alexey Proskuryakov 2012-02-15 10:47:30 PST
The database can be in-memory, why is this an issue?
Comment 2 andy.shaw 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?
Comment 3 Brady Eidson 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.
Comment 4 andy.shaw 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
Comment 5 Brady Eidson 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.
Comment 6 andy.shaw 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