Comment #10 on issue 29354 by [email protected]: Starting earth download  
eats all your RAM
http://code.google.com/p/chromium/issues/detail?id=29354

It looks like the problem is in/around net::SniffMimeType().  LooksBinary()  
fails to
find any non-text bytes in the first few K of the document, so the code  
falls through
to SniffForMagicNumbers() which sees a "#!" at the start of the document  
and happily
sets the new MIME type hint to "text/plain" and returns true out of
net::SniffMimeType(), which is used as a signal by
BufferedResourceHandler::KeepBuffering() to override the current MIME type
(application/octet-stream) with the new hint (text/plain).  After this  
point the
request looks like any other text download and seems doomed to be displayed  
inline.

Anyone know how other browsers evade this?  Sniffing more characters into  
the
document is not a real fix since there could be an arbitrary amount  
of 'normal' text
up from before the tarball begins.  I think we should keep the MIME type
application/octet-stream and not text/plain but the magic numbers for "#!"  
seem to
indicate that that is explicitly supposed to be handled as text.  Is that  
for compat
reasons?

--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

-- 
Automated mail from issue updates at http://crbug.com/
Subscription options: http://groups.google.com/group/chromium-bugs

Reply via email to