A signal needs to be emitted in EFL FrameLoaderClient's dispatchDidCancelClientRedirect(). This signal needs to be caught in the DumpRenderTree so that "didCancelClientRedirectForFrame" are printed if the LayoutTestControler's dumpFrameLoadCallbacks is set to true.
Created attachment 137867 [details] Patch This patch improves the dumpFrameLoadCallbacks() output expected by the following test cases: didCancelClientRedirectForFrame: http/tests/loading/onload-vs-immediate-refresh.html http/tests/loading/deleted-host-in-resource-load-delegate-callback.html http/tests/loading/307-after-303-after-post.html http/tests/loading/redirect-methods.html http/tests/loading/authentication-after-redirect-stores-wrong-credentials/authentication-after-redirect-stores-wrong-credentials.html http/tests/globalhistory/history-delegate-basic-refresh-redirect.html http/tests/globalhistory/history-delegate-basic-302-redirect.html webarchive/loading/javascript-url-iframe-crash.html Unfortunately, none of these tests can be unskipped yet because they require other features (in particular dumpResourceLoadCallbacks() from #81891).
Generally is OK. View in context: https://bugs.webkit.org/attachment.cgi?id=137867&action=review > Source/WebKit/efl/ChangeLog:3 > + Bug 83669 - [EFL] dispatchDidCancelClientRedirect needs implementation Unnecessary line. > Tools/ChangeLog:3 > + Bug 83669 - [EFL] dispatchDidCancelClientRedirect needs implementation Ditto.
Created attachment 137870 [details] Patch Fixed Changelogs
Thanks for changes, LGTM.
Comment on attachment 137870 [details] Patch Clearing flags on attachment: 137870 Committed r114633: <http://trac.webkit.org/changeset/114633>
All reviewed patches have been landed. Closing bug.