[Webkit-unassigned] [Bug 125486] [EFL] Run-webkit-tests --efl raise ScriptError

[bugzilla-daemon at webkit.org]

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





--- Comment #7 from Ryuan Choi <ryuan.choi at samsung.com>  2013-12-19 05:07:50 PST ---
(In reply to comment #6)
> (In reply to comment #4)
> > (In reply to comment #3)
> > > First of all, thanks for the fast reply. I had found a documentation, that will answer all your questions: http://docs.enlightenment.org/auto/ecore/group__Ecore__Evas__Group.html#gab8fa311077f749190c9b622f672b2214
> > 
> > I can't get your point.
> > 
> > Because ewk_init() calls ecore_init() and evas_init(), we should call ecore_shutdown() and evas_shutdown() in ewk_shutdown().
> 
> I think Tibor is right. The ecore_evas_shutdown() wraps the ecore_shutdown() and evas_shutdown(). See: http://git.enlightenment.fr/~leif/ecore/tree/src/lib/ecore_evas/ecore_evas.c#n253
> 

_init()/_shutdown() are ref counted internally. so we should keep the pair _init()/_shutdown().

If not, _shutdown() just return after decreased the count.

I agree that it might be unnecessary, but it will not be the root cause.

> The problem is that when the EWebLauncher is closed, the following error will emit
> fccache.c:507: FcCacheFini: Assertion `fcCacheChains[i] == ((void *)0)' failed.
> 
> As I see, this is because we try to free the font config twice. I checked and this patch really removes the above error.
> 
> But you might have right, the ewk_init is still messy, we shouldn't call ecore_init and evas_init separate if use the ecore_evas_init wrapper function.

No, it is already raised and EFL fixed it.
(http://sourceforge.net/mailarchive/message.php?msg_id=30424686)

So since 1.7.6, it will be fine.

Anyway, I made Bug 125479 to bump EFL 1.8

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