WKPageLoaderClient has empty functions after the refactorings, let's clean that.
Created attachment 102124 [details] Patch
Comment on attachment 102124 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=102124&action=review r=me > Source/WebKit2/UIProcess/qt/ClientImpl.cpp:116 > + QUrl qUrl = urlStr; > + toQtWebPageProxy(clientInfo)->updateNavigationActions(); > + toQtWebPageProxy(clientInfo)->didChangeUrl(qUrl); I don't really see the need for the qUrl variable here.
Comment on attachment 102124 [details] Patch This qUrl is just adding clarity, no arm here :)
(In reply to comment #3) > (From update of attachment 102124 [details]) > This qUrl is just adding clarity, no arm here :) If you hinsist :)
Comment on attachment 102124 [details] Patch Clearing flags on attachment: 102124 Committed r91840: <http://trac.webkit.org/changeset/91840>
All reviewed patches have been landed. Closing bug.