Treat ERR_CONNECTION_CLOSED as end-of-data marker for downloads.

chrome/browser/safe_browsing/safe_browsing_database.cc

Index: chrome/browser/safe_browsing/safe_browsing_database.cc
diff --git a/chrome/browser/safe_browsing/safe_browsing_database.cc b/chrome/browser/safe_browsing/safe_browsing_database.cc
index 16c4b38715cd1b85c09511e83ff27aec79e151fe..c4d764981ed1811b7f212b8be9dd384230780771 100644
--- a/chrome/browser/safe_browsing/safe_browsing_database.cc
+++ b/chrome/browser/safe_browsing/safe_browsing_database.cc
@@ -449,7 +449,6 @@ FilePath SafeBrowsingDatabase::CsdWhitelistDBFilename(
 }
 
 SafeBrowsingStore* SafeBrowsingDatabaseNew::GetStore(const int list_id) {
-  DVLOG(3) << "Get store for list: " << list_id;

[Randy Smith (Not in Mondays), 2011/05/03 21:30:41]

I doubt this is part of your CL, so you probably don't want to change this file?

[Scott Hess - ex-Googler, 2011/05/03 21:50:29]

If this is the log line you refer to in the CC to me, then LGTM to remove.

BTW, I'd generally un-support adding new log lines unless you're convinced that
they would be totally useful for cases where you can't just re-compile Chromium.
 They may-or-may-not be cheap in execution time, but often enough the log line
which was obvious at point of insertion isn't obvious six months later when
someone is debugging things, so it's just a bunch of extra noise to read through
in the code.

[ahendrickson, 2011/05/04 15:08:25]

Since all our debugging happens at VLOG(20), this outputs a massive amount of
logging (making it look like Chrome is hung) when I try to debug.

I'd be happy to leave it in at VLOG(30) or something like that, if people think
it's necessary.

Oh, and list_id is always 1.

   if (list_id == safe_browsing_util::PHISH ||
       list_id == safe_browsing_util::MALWARE) {
     return browse_store_.get();