Commit da10ffa3 authored by ananta@chromium.org's avatar ananta@chromium.org

Fix incorrect map usage in the WebPluginImpl::didReceiveData function, which caused

a response to be incorrectly treated as a multipart response. This would eventually
end up sending the ViewHostMsg_DidStopLoading IPC to the browser, which would treat the
page as loaded, when it has not. 

This fixes http://code.google.com/p/chromium/issues/detail?id=7916, which would cause
the favicon to not show up on trunk.

Bug=7916


Review URL: http://codereview.chromium.org/28199

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@10499 0039d316-1c4b-4281-b951-d872f2087c98
parent 4792dc2b
......@@ -1042,9 +1042,11 @@ void WebPluginImpl::didReceiveData(WebCore::ResourceHandle* handle,
int length, int) {
WebPluginResourceClient* client = GetClientFromHandle(handle);
if (client) {
MultipartResponseDelegate* multi_part_handler =
multi_part_response_map_[client];
if (multi_part_handler) {
MultiPartResponseHandlerMap::iterator index =
multi_part_response_map_.find(client);
if (index != multi_part_response_map_.end()) {
MultipartResponseDelegate* multi_part_handler = (*index).second;
DCHECK(multi_part_handler != NULL);
multi_part_handler->OnReceivedData(buffer, length);
} else {
client->DidReceiveData(buffer, length, 0);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment