[Webkit-unassigned] [Bug 60361] [EFL] EWebLauncher creates only one settings databases directory at startup.

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue May 17 14:21:55 PDT 2011


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





--- Comment #17 from Lucas De Marchi <demarchi at webkit.org>  2011-05-17 14:21:55 PST ---
(From update of attachment 93081)
View in context: https://bugs.webkit.org/attachment.cgi?id=93081&action=review

For the reason pointed above, my informal r-

>> Tools/EWebLauncher/main.c:880
>> +    if (!ewk_init(path))
> 
> I think this is really API related so I would like Lucas input on it

This is not how EFL works. There's no EFL lib with arguments in *_init() functions because you have to do only the bare minimum to initialize the lib. If you feel like you need an argument in init, you better split that functionality to another function that you call later.

In this case, IMO an acceptable patch would be not to set the cache/database/icon directory until the application (EWebLauncher) explicitly tells webkit to do so.

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



More information about the webkit-unassigned mailing list