[webkit-reviews] review denied: [Bug 38397] Add "preSubmitForm" hook to FrameLoader : [Attachment 54819] patch

[bugzilla-daemon at webkit.org]

Darin Fisher (:fishd, Google) <fishd at chromium.org> has denied Jens Alfke
<snej at chromium.org>'s request for review:
Bug 38397: Add "preSubmitForm" hook to FrameLoader
https://bugs.webkit.org/show_bug.cgi?id=38397

Attachment 54819: patch
https://bugs.webkit.org/attachment.cgi?id=54819&action=review

------- Additional Comments from Darin Fisher (:fishd, Google)
<fishd at chromium.org>
WebCore/html/HTMLFormElement.cpp:294
 +	frame->loader()->dispatchPreSubmitForm(this);
I would just do frame->loader()->client()->dispatch* here instead
of adding the intermediate FrameLoader method.

WebCore/loader/FrameLoaderClient.h:149
 +	    virtual void dispatchPreSubmitForm(HTMLFormElement*) { };
How about naming this dispatchWillSendSubmitEvent?  I think that name is more
accurate.

WebKit/chromium/public/WebFrameClient.h:143
 +	virtual void preSubmitForm(WebFrame*, const WebFormElement&) { }
willSendSubmitEvent