[Webkit-unassigned] [Bug 72258] [GTK] Implement download support in WebKit2

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=72258





--- Comment #3 from Carlos Garcia Campos <cgarcia at igalia.com>  2011-11-14 23:50:12 PST ---
(In reply to comment #2)
> (From update of attachment 114921 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=114921&action=review
> 
> Looks good, but I'm not sure the changes in Source/WebKit/gtk/webkit/webkitdownload.cpp are necessary...

No, they are not necessary, but since I moved the download errors to WebCore, I think wk1 should use them, so that we avoid having the same translatable strings in two different places.

> > Source/WebKit/gtk/webkit/webkitdownload.cpp:476
> > +        webkitDownloadEmitError(download, downloadDestinationError(core(priv->networkResponse), error->message));
> 
> It It seems like this code is just using the ResourceError to carry the message and the error code to webkitDownloadEmitError. Why not just pass the error code and the message directly?

In this case it would be possible, because DestinationError is built with the message, but for other errors we need the ResourceError to get the message from it, for example:

webkitDownloadEmitError(download, downloadCancelledByUserError(core(priv->networkResponse)));

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.