EFL's FrameLoaderClient currently emits the "load,finished" signal from the postProgressFinishedNotification() method. It apparently causes the signal to be emitted only for the main frame and not for other frame. It is more suited to emit the signal from dispatchDidFinishLoad() and dispatchDidFailLoad() so that we can send notification for all frames. This change is needed to unskip several test cases which rely on the DumpRenderTree support for LayoutTestController's dumpFrameLoadCallbacks() and expect notifications for other frame than the main one: * http/tests/security/mixedContent/empty-url-plugin-in-frame.html * http/tests/security/mixedContent/about-blank-iframe-in-main-frame.html * http/tests/security/mixedContent/data-url-iframe-in-main-frame.html * http/tests/security/mixedContent/data-url-script-in-iframe.html * http/tests/security/mixedContent/insecure-css-in-iframe.html * http/tests/security/mixedContent/insecure-iframe-in-iframe.html * http/tests/security/mixedContent/insecure-iframe-in-main-frame.html * http/tests/security/mixedContent/insecure-image-in-iframe.html * http/tests/security/mixedContent/redirect-http-to-https-script-in-iframe.html * http/tests/security/mixedContent/redirect-https-to-http-script-in-iframe.html
Created attachment 137366 [details] Proposed patch
Comment on attachment 137366 [details] Proposed patch If you submit rebased patch again, looks fine to me.
Comment on attachment 137366 [details] Proposed patch be sure it applies. bubbles are purple.
Created attachment 137551 [details] Proposed patch Rebase on master for landing.
Comment on attachment 137551 [details] Proposed patch Clearing flags on attachment: 137551 Committed r114420: <http://trac.webkit.org/changeset/114420>
All reviewed patches have been landed. Closing bug.