[Webkit-unassigned] [Bug 89186] [EFL][WK2] Add APIs to create, delete and get ewk_context.

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Jun 28 03:10:05 PDT 2012


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





--- Comment #8 from EunMi Lee <eunmi15.lee at samsung.com>  2012-06-28 03:10:03 PST ---
(In reply to comment #5)
> (In reply to comment #3)
> > (From update of attachment 147953 [details] [details])
> > View in context: https://bugs.webkit.org/attachment.cgi?id=147953&action=review
> > 
> > > Source/WebKit2/UIProcess/API/efl/ewk_context.cpp:70
> > > +void ewk_context_delete(Ewk_Context* ewkContext)
> > > +{
> > > +    EINA_SAFETY_ON_NULL_RETURN(ewkContext);
> > > +    EINA_SAFETY_ON_TRUE_RETURN(ewkContext == ewk_context_default_get());
> > > +
> > > +    delete ewkContext;
> > > +}
> > 
> > Can we delete context which have live ewk_view?
> 
> Good question :)
> I think application should not call ewk_context_delete() for context which have live ewk_view.
> so, I will add codes to check whether context has ewk_view or not.

Ryuan,

I added code to check WKContext is referred by someome using refCount().
Please review new patch again :)

-- 
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