[webkit-reviews] review cancelled: [Bug 16562] [gtk] Implement WebPolicyDelegate methods : [Attachment 18040] Suggested implementation

[bugzilla-daemon at webkit.org]

Pierre-Luc Beaudoin <pierre-luc.beaudoin at collabora.co.uk> has cancelled
Pierre-Luc Beaudoin <pierre-luc.beaudoin at collabora.co.uk>'s request for review:
Bug 16562: [gtk] Implement WebPolicyDelegate methods
http://bugs.webkit.org/show_bug.cgi?id=16562

Attachment 18040: Suggested implementation
http://bugs.webkit.org/attachment.cgi?id=18040&action=edit

------- Additional Comments from Pierre-Luc Beaudoin
<pierre-luc.beaudoin at collabora.co.uk>
bdash suggested, with reason, that the mime-type-requested signal name was not
clear, we settled on mime-type-policy-decision-requested.  Since
new-window-requested and navigation-requested are clear this way, I won't
change them although we loose on consistency.

I removed the _real_mime_type function to place the default behavior in
FrameLoaderClientGtk::dispatchDecidePolicyForMIMEType instead.	This is the
only place the signal is emitted, it makes sens to me.	

I added WEBKIT_POLICY_RESPONSE_NULL to allow an handler to return an error or
force the default behavior.  Also, when there is no handler to
mime-type-policy-decision-requested, the return value defaults to
WEBKIT_POLICY_RESPONSE_NULL which will trigger the default behavior.