Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(3080)

Unified Diff: chrome/browser/download/download_query.cc

Issue 16924017: A few minor changes to the chrome.downloads extension API (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: @r208716 Created 7 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/download/download_query.cc
diff --git a/chrome/browser/download/download_query.cc b/chrome/browser/download/download_query.cc
index 3f7ed1002f52d8f3e62f964ac4543fced44b128f..76448d49a9c366dacc1568f84151898956353de6 100644
--- a/chrome/browser/download/download_query.cc
+++ b/chrome/browser/download/download_query.cc
@@ -58,8 +58,6 @@ static bool MatchesQuery(const string16& query, const DownloadItem& item) {
if (query.empty())
return true;
- DCHECK_EQ(query, base::i18n::ToLower(query));
-
string16 url_raw(UTF8ToUTF16(item.GetOriginalUrl().spec()));
if (base::i18n::StringSearchIgnoringCaseAndAccents(
query, url_raw, NULL, NULL)) {
@@ -270,7 +268,7 @@ bool DownloadQuery::AddFilter(DownloadQuery::FilterType type,
case FILTER_QUERY: {
string16 query;
return GetAs(value, &query) &&
- AddFilter(base::Bind(&MatchesQuery, query));
+ AddFilter(base::Bind(&MatchesQuery, base::i18n::ToLower(query)));
}
case FILTER_ENDED_AFTER:
return AddFilter(BuildFilter<std::string>(value, GT, &GetEndTime));

Powered by Google App Engine
This is Rietveld 408576698