[Webkit-unassigned] [Bug 50698] [GTK] Split webkitprivate.{cpp, h} in more manageable chunks
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Thu Dec 9 17:02:56 PST 2010
https://bugs.webkit.org/show_bug.cgi?id=50698
Martin Robinson <mrobinson at webkit.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #76100|review? |review+
Flag| |
--- Comment #9 from Martin Robinson <mrobinson at webkit.org> 2010-12-09 17:02:56 PST ---
(From update of attachment 76100)
View in context: https://bugs.webkit.org/attachment.cgi?id=76100&action=review
> WebKit/gtk/webkit/webkitnetworkrequest.cpp:264
> +WebKitNetworkRequest* kitNew(const WebCore::ResourceRequest& resourceRequest)
> +{
> + PlatformRefPtr<SoupMessage> soupMessage(adoptPlatformRef(resourceRequest.toSoupMessage()));
> + if (soupMessage)
> + return WEBKIT_NETWORK_REQUEST(g_object_new(WEBKIT_TYPE_NETWORK_REQUEST, "message", soupMessage.get(), NULL));
> +
> + return WEBKIT_NETWORK_REQUEST(g_object_new(WEBKIT_TYPE_NETWORK_REQUEST, "uri", resourceRequest.url().string().utf8().data(), NULL));
> +}
> +
It's a shame that the WebKitNetworkRequest doesn't store a copy of the original resource request. If it's possible perhaps it would make sense to do that in another patch. The kitNew name seems a little funky here, but makes sense.
> WebKit/gtk/webkit/webkitnetworkrequestprivate.h:28
> +WebCore::ResourceRequest core(WebKitNetworkRequest* request);
Please remove this variable name before landing.
--
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
More information about the webkit-unassigned
mailing list