Chromium Code Reviews| Index: content/browser/download/download_stats.cc |
| diff --git a/content/browser/download/download_stats.cc b/content/browser/download/download_stats.cc |
| index d88eef2c4e2c039e61add7820cc94b88b562cfbe..a033f7046382c243e065f9bfa8f80a707ffa0c36 100644 |
| --- a/content/browser/download/download_stats.cc |
| +++ b/content/browser/download/download_stats.cc |
| @@ -39,6 +39,8 @@ enum ContentDispositionCountTypes { |
| // net::HttpContentDisposition::ParseResult. |
| CONTENT_DISPOSITION_HAS_DISPOSITION_TYPE, |
| CONTENT_DISPOSITION_HAS_UNKNOWN_TYPE, |
| + |
| + // Support removed. Kept for UMA compatiblity. |
| CONTENT_DISPOSITION_HAS_NAME, |
|
asanka
2015/04/28 00:33:44
It looks like the comment applies to all the enum
|
| CONTENT_DISPOSITION_HAS_FILENAME, |
| CONTENT_DISPOSITION_HAS_EXT_FILENAME, |
| @@ -46,7 +48,7 @@ enum ContentDispositionCountTypes { |
| CONTENT_DISPOSITION_HAS_PERCENT_ENCODED_STRINGS, |
| CONTENT_DISPOSITION_HAS_RFC2047_ENCODED_STRINGS, |
| - // Only have the 'name' attribute is present. |
| + // Support removed. Kept for UMA compatiblity. |
| CONTENT_DISPOSITION_HAS_NAME_ONLY, |
| CONTENT_DISPOSITION_LAST_ENTRY |
| @@ -505,9 +507,6 @@ void RecordDownloadContentDisposition( |
| CONTENT_DISPOSITION_HAS_UNKNOWN_TYPE, result, |
| net::HttpContentDisposition::HAS_UNKNOWN_DISPOSITION_TYPE); |
| RecordContentDispositionCountFlag( |
| - CONTENT_DISPOSITION_HAS_NAME, result, |
| - net::HttpContentDisposition::HAS_NAME); |
| - RecordContentDispositionCountFlag( |
| CONTENT_DISPOSITION_HAS_FILENAME, result, |
| net::HttpContentDisposition::HAS_FILENAME); |
| RecordContentDispositionCountFlag( |
| @@ -522,13 +521,6 @@ void RecordDownloadContentDisposition( |
| RecordContentDispositionCountFlag( |
| CONTENT_DISPOSITION_HAS_RFC2047_ENCODED_STRINGS, result, |
| net::HttpContentDisposition::HAS_RFC2047_ENCODED_STRINGS); |
| - |
| - RecordContentDispositionCount( |
| - CONTENT_DISPOSITION_HAS_NAME_ONLY, |
| - (result & (net::HttpContentDisposition::HAS_NAME | |
| - net::HttpContentDisposition::HAS_FILENAME | |
| - net::HttpContentDisposition::HAS_EXT_FILENAME)) == |
| - net::HttpContentDisposition::HAS_NAME); |
| } |
| void RecordFileThreadReceiveBuffers(size_t num_buffers) { |