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

Side by Side Diff: extensions/common/permissions/permissions_info.cc

Issue 1242023005: Remove legacy StartsWithASCII function. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: y Created 5 years, 5 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 unified diff | Download patch
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "extensions/common/permissions/permissions_info.h" 5 #include "extensions/common/permissions/permissions_info.h"
6 6
7 #include "base/lazy_instance.h" 7 #include "base/lazy_instance.h"
8 #include "base/logging.h" 8 #include "base/logging.h"
9 #include "base/stl_util.h" 9 #include "base/stl_util.h"
10 #include "base/strings/string_util.h" 10 #include "base/strings/string_util.h"
(...skipping 46 matching lines...) Expand 10 before | Expand all | Expand 10 after
57 const APIPermissionInfo* permission_info = GetByName(*i); 57 const APIPermissionInfo* permission_info = GetByName(*i);
58 if (permission_info) 58 if (permission_info)
59 permissions.insert(permission_info->id()); 59 permissions.insert(permission_info->id());
60 } 60 }
61 return permissions; 61 return permissions;
62 } 62 }
63 63
64 bool PermissionsInfo::HasChildPermissions(const std::string& name) const { 64 bool PermissionsInfo::HasChildPermissions(const std::string& name) const {
65 NameMap::const_iterator i = name_map_.lower_bound(name + '.'); 65 NameMap::const_iterator i = name_map_.lower_bound(name + '.');
66 if (i == name_map_.end()) return false; 66 if (i == name_map_.end()) return false;
67 return base::StartsWithASCII(i->first, name + '.', true); 67 return base::StartsWith(i->first, name + '.', base::CompareCase::SENSITIVE);
68 } 68 }
69 69
70 PermissionsInfo::PermissionsInfo() 70 PermissionsInfo::PermissionsInfo()
71 : permission_count_(0) { 71 : permission_count_(0) {
72 } 72 }
73 73
74 PermissionsInfo::~PermissionsInfo() { 74 PermissionsInfo::~PermissionsInfo() {
75 STLDeleteContainerPairSecondPointers(id_map_.begin(), id_map_.end()); 75 STLDeleteContainerPairSecondPointers(id_map_.begin(), id_map_.end());
76 } 76 }
77 77
78 void PermissionsInfo::RegisterAlias( 78 void PermissionsInfo::RegisterAlias(
79 const char* name, 79 const char* name,
80 const char* alias) { 80 const char* alias) {
81 DCHECK(ContainsKey(name_map_, name)); 81 DCHECK(ContainsKey(name_map_, name));
82 DCHECK(!ContainsKey(name_map_, alias)); 82 DCHECK(!ContainsKey(name_map_, alias));
83 name_map_[alias] = name_map_[name]; 83 name_map_[alias] = name_map_[name];
84 } 84 }
85 85
86 void PermissionsInfo::RegisterPermission(APIPermissionInfo* permission) { 86 void PermissionsInfo::RegisterPermission(APIPermissionInfo* permission) {
87 DCHECK(!ContainsKey(id_map_, permission->id())); 87 DCHECK(!ContainsKey(id_map_, permission->id()));
88 DCHECK(!ContainsKey(name_map_, permission->name())); 88 DCHECK(!ContainsKey(name_map_, permission->name()));
89 89
90 id_map_[permission->id()] = permission; 90 id_map_[permission->id()] = permission;
91 name_map_[permission->name()] = permission; 91 name_map_[permission->name()] = permission;
92 92
93 permission_count_++; 93 permission_count_++;
94 } 94 }
95 95
96 } // namespace extensions 96 } // namespace extensions
OLDNEW
« no previous file with comments | « extensions/common/extension.cc ('k') | extensions/shell/common/shell_content_client_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698