[Webkit-unassigned] [Bug 71611] [GTK] simplify ResourceHandleSoup
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Sat Nov 5 12:36:57 PDT 2011
https://bugs.webkit.org/show_bug.cgi?id=71611
--- Comment #12 from Martin Robinson <mrobinson at webkit.org> 2011-11-05 12:36:57 PST ---
(From update of attachment 113761)
View in context: https://bugs.webkit.org/attachment.cgi?id=113761&action=review
This patch is awesome. I'm sad to see the soup dependency go up again, but we were using unstable API so it seem unavoidable.
> Source/WebCore/platform/network/soup/ResourceHandleSoup.cpp:316
> + if (handle->shouldContentSniff()
> + && soupMessage->status_code != SOUP_STATUS_NOT_MODIFIED) {
This can be one line. In WebKit we generally go to about 120 character before splitting them.
> Source/WebCore/platform/network/soup/ResourceHandleSoup.cpp:321
> + const char* officialType = soup_message_headers_get_one(soupMessage->response_headers, "Content-Type");
> + const char* sniffedType = soup_request_get_content_type(d->m_soupRequest.get());
> + if (sniffedType && (!officialType || strcmp(officialType, sniffedType)))
> + d->m_response.setSniffedContentType(sniffedType);
> + }
I think it would make sense to always set the sniffed content type and move the logic about whether or not the sniffed type should override the advertised type to ResourceResponse::updateFromSoupMessage. I say this because otherwise there might be times when sniffedContentType returns an empty string even when Soup did sniff a content type.
--
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