Ewk_Context should be Ewk_Object, as part of bug101057 work.
Created attachment 172530 [details] patch
Comment on attachment 172530 [details] patch View in context: https://bugs.webkit.org/attachment.cgi?id=172530&action=review > Source/WebKit2/UIProcess/API/efl/ewk_context.cpp:199 > +void EwkContext::addVisitedLink(const String& visitedURL) Isn't this confusing now? What abut WebContextEfl ?
(In reply to comment #2) > (From update of attachment 172530 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=172530&action=review > > > Source/WebKit2/UIProcess/API/efl/ewk_context.cpp:199 > > +void EwkContext::addVisitedLink(const String& visitedURL) > > Isn't this confusing now? What abut WebContextEfl ? I don't mind, however how about other classes? WebURLRequsetEfl instead of present EwkUrlRequest?
Comment on attachment 172530 [details] patch View in context: https://bugs.webkit.org/attachment.cgi?id=172530&action=review >>> Source/WebKit2/UIProcess/API/efl/ewk_context.cpp:199 >>> +void EwkContext::addVisitedLink(const String& visitedURL) >> >> Isn't this confusing now? What abut WebContextEfl ? > > I don't mind, however how about other classes? WebURLRequsetEfl instead of present EwkUrlRequest? Why is it confusing? EwkContext is a Ewk Object. Kenneth, you want to call it "WebContextEfl"? why? I think then it would become confusing because we already have a WebContextEfl.cpp file (to provide EFL-specific implementation of some WebContext methods).
Comment on attachment 172530 [details] patch Clearing flags on attachment: 172530 Committed r133587: <http://trac.webkit.org/changeset/133587>
All reviewed patches have been landed. Closing bug.