diff options
author | brettw <brettw@chromium.org> | 2015-07-15 23:48:05 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2015-07-16 06:48:41 +0000 |
commit | edce9a33027cc5f73c4866d70e34f690f6720a56 (patch) | |
tree | a8dbfae7b9ebf5a5cac2bb8a02f59b1637ddfa9c /ios | |
parent | fc4f1d1d95a221319f4c275e870e9b7cf2dc5040 (diff) | |
download | chromium_src-edce9a33027cc5f73c4866d70e34f690f6720a56.zip chromium_src-edce9a33027cc5f73c4866d70e34f690f6720a56.tar.gz chromium_src-edce9a33027cc5f73c4866d70e34f690f6720a56.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
Review URL: https://codereview.chromium.org/1239493005
Cr-Commit-Position: refs/heads/master@{#338996}
Diffstat (limited to 'ios')
-rw-r--r-- | ios/web/webui/web_ui_ios_data_source_impl.cc | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/ios/web/webui/web_ui_ios_data_source_impl.cc b/ios/web/webui/web_ui_ios_data_source_impl.cc index ee0ed42..e20b660 100644 --- a/ios/web/webui/web_ui_ios_data_source_impl.cc +++ b/ios/web/webui/web_ui_ios_data_source_impl.cc @@ -109,19 +109,19 @@ std::string WebUIIOSDataSourceImpl::GetSource() const { } std::string WebUIIOSDataSourceImpl::GetMimeType(const std::string& path) const { - if (base::EndsWith(path, ".js", false)) + if (base::EndsWith(path, ".js", base::CompareCase::INSENSITIVE_ASCII)) return "application/javascript"; - if (base::EndsWith(path, ".json", false)) + if (base::EndsWith(path, ".json", base::CompareCase::INSENSITIVE_ASCII)) return "application/json"; - if (base::EndsWith(path, ".pdf", false)) + if (base::EndsWith(path, ".pdf", base::CompareCase::INSENSITIVE_ASCII)) return "application/pdf"; - if (base::EndsWith(path, ".css", false)) + if (base::EndsWith(path, ".css", base::CompareCase::INSENSITIVE_ASCII)) return "text/css"; - if (base::EndsWith(path, ".svg", false)) + if (base::EndsWith(path, ".svg", base::CompareCase::INSENSITIVE_ASCII)) return "image/svg+xml"; return "text/html"; |