I want to call some other application when user click mailto. I think that FrameLoaderClientEfl::dispatchDecidePolicyForNavigationAction is proper function.
Created attachment 63565 [details] Patch
Created attachment 63573 [details] Patch
Created attachment 63681 [details] Patch
Created attachment 63694 [details] proposed patch Sorry I found wrong EAPI option in ewk_private.h. I removed
Comment on attachment 63694 [details] proposed patch I think it looks good. I have one question: > + > + PolicyAction policy; > + if (!ret) > + policy = PolicyIgnore; > + else { > + Frame* f = ewk_frame_core_get(m_frame); > + f->loader()->resetMultipleFormSubmissionProtection(); > + policy = PolicyUse; > + } You are unconditionally calling resetMultipleFormSubmissionProtection() . Should not you do so only with the NavicationAction involves that? Se Qt's code below: if (!page->d->acceptNavigationRequest(m_webFrame, r, QWebPage::NavigationType(action.type()))) { if (action.type() == NavigationTypeFormSubmitted || action.type() == NavigationTypeFormResubmitted) m_frame->loader()->resetMultipleFormSubmissionProtection();
(In reply to comment #5) > (From update of attachment 63694 [details]) > I think it looks good. I have one question: > > > + > > + PolicyAction policy; > > + if (!ret) > > + policy = PolicyIgnore; > > + else { > > + Frame* f = ewk_frame_core_get(m_frame); > > + f->loader()->resetMultipleFormSubmissionProtection(); > > + policy = PolicyUse; > > + } > > > You are unconditionally calling resetMultipleFormSubmissionProtection() . Should not you do so only with the NavicationAction involves that? > > Se Qt's code below: > > if (!page->d->acceptNavigationRequest(m_webFrame, r, QWebPage::NavigationType(action.type()))) { > if (action.type() == NavigationTypeFormSubmitted || action.type() == NavigationTypeFormResubmitted) > m_frame->loader()->resetMultipleFormSubmissionProtection(); Ok, I'll add action related code
Created attachment 63859 [details] Patch
Comment on attachment 63859 [details] Patch r=me
Comment on attachment 63859 [details] Patch Clearing flags on attachment: 63859 Committed r65163: <http://trac.webkit.org/changeset/65163>
All reviewed patches have been landed. Closing bug.