summaryrefslogtreecommitdiffstats
path: root/components/devtools_http_handler
diff options
context:
space:
mode:
authorbrettw <brettw@chromium.org>2015-07-16 10:49:29 -0700
committerCommit bot <commit-bot@chromium.org>2015-07-16 17:50:12 +0000
commita7ff1b291cd328e9584779a4bb9101fa9a3c01ad (patch)
treed010374be1ca156ea26405ae7b0aa85e48bb4bba /components/devtools_http_handler
parent7befa9cf85342a0409502ed2c05d7610afec7844 (diff)
downloadchromium_src-a7ff1b291cd328e9584779a4bb9101fa9a3c01ad.zip
chromium_src-a7ff1b291cd328e9584779a4bb9101fa9a3c01ad.tar.gz
chromium_src-a7ff1b291cd328e9584779a4bb9101fa9a3c01ad.tar.bz2
Remove some legacy versions of StartsWith and EndsWith.
This just replaces true -> base::CompareCase::SENSITIVE false -> base::CompareCase::INSENSITIVE_ASCII I checked the insensitive cases to make sure they're not doing anything suspicious. The old version is a sometimes-correct Unicode comparison so converting to INSENSTITIVE_ASCII isn't a no-op. However, generally the prefix/suffix checking is done against a hardcoded string so there were very few cases to actually look at. extensions/browser/api/declarative_webrequest/webrequest_condition_attribute.cc has a not-quite search-and-replace change where I changed the type of a class variable. BUG=506255 TBR=jam Reland of http://crrev.com/1239493005 Review URL: https://codereview.chromium.org/1233043003 Cr-Commit-Position: refs/heads/master@{#339071}
Diffstat (limited to 'components/devtools_http_handler')
-rw-r--r--components/devtools_http_handler/devtools_http_handler.cc17
1 files changed, 11 insertions, 6 deletions
diff --git a/components/devtools_http_handler/devtools_http_handler.cc b/components/devtools_http_handler/devtools_http_handler.cc
index 0080f68..40f7edc 100644
--- a/components/devtools_http_handler/devtools_http_handler.cc
+++ b/components/devtools_http_handler/devtools_http_handler.cc
@@ -359,17 +359,22 @@ static std::string PathWithoutParams(const std::string& path) {
}
static std::string GetMimeType(const std::string& filename) {
- if (base::EndsWith(filename, ".html", false)) {
+ if (base::EndsWith(filename, ".html", base::CompareCase::INSENSITIVE_ASCII)) {
return "text/html";
- } else if (base::EndsWith(filename, ".css", false)) {
+ } else if (base::EndsWith(filename, ".css",
+ base::CompareCase::INSENSITIVE_ASCII)) {
return "text/css";
- } else if (base::EndsWith(filename, ".js", false)) {
+ } else if (base::EndsWith(filename, ".js",
+ base::CompareCase::INSENSITIVE_ASCII)) {
return "application/javascript";
- } else if (base::EndsWith(filename, ".png", false)) {
+ } else if (base::EndsWith(filename, ".png",
+ base::CompareCase::INSENSITIVE_ASCII)) {
return "image/png";
- } else if (base::EndsWith(filename, ".gif", false)) {
+ } else if (base::EndsWith(filename, ".gif",
+ base::CompareCase::INSENSITIVE_ASCII)) {
return "image/gif";
- } else if (base::EndsWith(filename, ".json", false)) {
+ } else if (base::EndsWith(filename, ".json",
+ base::CompareCase::INSENSITIVE_ASCII)) {
return "application/json";
}
LOG(ERROR) << "GetMimeType doesn't know mime type for: "