[Webkit-unassigned] [Bug 117625] [EFL][WK2] Fix crash issue on ewk_context_favicon_database_directory_set()

[bugzilla-daemon at webkit.org]

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





--- Comment #5 from Changhyup Jwa <ch.jwa at samsung.com>  2013-06-13 23:19:16 PST ---
(In reply to comment #2)
> (From update of attachment 204674 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=204674&action=review
> 
> > Source/WebKit2/UIProcess/API/C/WKContext.cpp:287
> > +bool WKContextSetIconDatabasePath(WKContextRef contextRef, WKStringRef iconDatabasePath)
> 
> You should avoid changing C API if possible. Most C API setters don't return bool so this seems a bit inconsistent.
> 

Without changing C API(returning value), can I add WK C APIs to enable and check IconDatabase's enabled flag? Because isOpen() returns false before IconDatabase's sync thread opens DB.

> > Source/WebKit2/UIProcess/API/efl/ewk_context.cpp:-161
> > -    if (iconDatabase->isOpen())
> 
> I remember we added this check for a specific reason. iirc the icon database path can only be set once. When the path is set the first time, the icon database gets enabled / opened. Therefore, checking for isOpen() was used to make sure we set the icon database path only once (and avoid assertion hit in debug mode).

As you commented, icon database path should be set once. But without this patch, EwkContext::setFaviconDatabaseDirectoryPath() tries to open DB before sync thread's DB opening is done.

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