summaryrefslogtreecommitdiffstats
path: root/base/android/library_loader
diff options
context:
space:
mode:
authorbrettw <brettw@chromium.org>2015-07-15 23:48:05 -0700
committerCommit bot <commit-bot@chromium.org>2015-07-16 06:48:41 +0000
commitedce9a33027cc5f73c4866d70e34f690f6720a56 (patch)
treea8dbfae7b9ebf5a5cac2bb8a02f59b1637ddfa9c /base/android/library_loader
parentfc4f1d1d95a221319f4c275e870e9b7cf2dc5040 (diff)
downloadchromium_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 'base/android/library_loader')
-rw-r--r--base/android/library_loader/library_prefetcher.cc6
1 files changed, 3 insertions, 3 deletions
diff --git a/base/android/library_loader/library_prefetcher.cc b/base/android/library_loader/library_prefetcher.cc
index 798a283..9b54843 100644
--- a/base/android/library_loader/library_prefetcher.cc
+++ b/base/android/library_loader/library_prefetcher.cc
@@ -36,7 +36,7 @@ bool IsReadableAndPrivate(const base::debug::MappedMemoryRegion& region) {
bool PathMatchesSuffix(const std::string& path) {
for (size_t i = 0; i < arraysize(kSuffixesToMatch); i++) {
- if (EndsWith(path, kSuffixesToMatch[i], true)) {
+ if (EndsWith(path, kSuffixesToMatch[i], CompareCase::SENSITIVE)) {
return true;
}
}
@@ -82,14 +82,14 @@ void NativeLibraryPrefetcher::FilterLibchromeRangesOnlyIfPossible(
std::vector<AddressRange>* ranges) {
bool has_libchrome_region = false;
for (const base::debug::MappedMemoryRegion& region : regions) {
- if (EndsWith(region.path, kLibchromeSuffix, true)) {
+ if (EndsWith(region.path, kLibchromeSuffix, CompareCase::SENSITIVE)) {
has_libchrome_region = true;
break;
}
}
for (const base::debug::MappedMemoryRegion& region : regions) {
if (has_libchrome_region &&
- !EndsWith(region.path, kLibchromeSuffix, true)) {
+ !EndsWith(region.path, kLibchromeSuffix, CompareCase::SENSITIVE)) {
continue;
}
ranges->push_back(std::make_pair(region.start, region.end));