Add mime type information to the download database

chrome/browser/download/download_history.cc

Index: chrome/browser/download/download_history.cc
diff --git a/chrome/browser/download/download_history.cc b/chrome/browser/download/download_history.cc
index ea2f2e13c015240c7426e5c2bdfa1ec43465d284..af7abf908722bc1e839a6e8913b9a6df6b5a39ad 100644
--- a/chrome/browser/download/download_history.cc
+++ b/chrome/browser/download/download_history.cc
@@ -114,6 +114,8 @@ history::DownloadRow GetDownloadRow(
       item->GetTargetFilePath(),
       item->GetUrlChain(),
       item->GetReferrerUrl(),
+      item->GetMimeType(),
+      item->GetOriginalMimeType(),
       item->GetStartTime(),
       item->GetEndTime(),
       item->GetETag(),
@@ -131,7 +133,8 @@ history::DownloadRow GetDownloadRow(
 
 bool ShouldUpdateHistory(const history::DownloadRow* previous,
                          const history::DownloadRow& current) {
-  // Ignore url, referrer, start_time, id, which don't change.
+  // Ignore url, referrer, mime_type, original_mime_type, start_time,
+  // id, db_handle, which don't change.

[Randy Smith (Not in Mondays), 2013/12/16 21:23:19]

Question for Asanka: Might mime_type/original_mime_type change on download
resumption?

[asanka, 2013/12/16 21:51:48]

Nothing prevents a subsequent response from changing the MIME type. But, I think
it's okay to go with the first MIME type seen the URL.

   return ((previous == NULL) ||
           (previous->current_path != current.current_path) ||
           (previous->target_path != current.target_path) ||
@@ -235,6 +238,8 @@ void DownloadHistory::QueryCallback(scoped_ptr<InfoVector> infos) {
         it->target_path,
         it->url_chain,
         it->referrer_url,
+        it->mime_type,
+        it->original_mime_type,
         it->start_time,
         it->end_time,
         it->etag,