The Resource load client is currently tightly integrated to the view and uses C style. We should move all the resource load client code to its own C++ class.
Created attachment 169904 [details] Patch
Comment on attachment 169904 [details] Patch Will move the class to WebKit namespace.
Created attachment 169929 [details] Patch Move client to WebKit namespace.
Comment on attachment 169929 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=169929&action=review r=me > Source/WebKit2/UIProcess/API/efl/ewk_view.cpp:839 > + priv->resourceLoadClient = ResourceLoadClientEfl::create(ewkView); why not call it ResourceLoaderClientEfl to show the relation to FrameLoaderClient > Source/WebKit2/UIProcess/efl/ResourceLoadClientEfl.cpp:131 > + WKPageResourceLoadClient wkResourceLoadClient; Ah I see now...
Comment on attachment 169929 [details] Patch Clearing flags on attachment: 169929 Committed r132122: <http://trac.webkit.org/changeset/132122>
All reviewed patches have been landed. Closing bug.