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

Unified Diff: extensions/browser/extension_prefs.cc

Issue 149213003: Cleanup: Fix some lint/style errors in extensions code. (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Created 6 years, 11 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
« no previous file with comments | « chrome/renderer/resources/extensions/permissions_custom_bindings.js ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: extensions/browser/extension_prefs.cc
===================================================================
--- extensions/browser/extension_prefs.cc (revision 247534)
+++ extensions/browser/extension_prefs.cc (working copy)
@@ -869,7 +869,6 @@
PermissionsInfo* info = PermissionsInfo::GetInstance();
for (ExtensionIdList::const_iterator ext_id =
extension_ids.begin(); ext_id != extension_ids.end(); ++ext_id) {
-
// An extension's granted permissions need to be migrated if the
// full_access bit is present. This bit was always present in the previous
// scheme and is never present now.
@@ -887,15 +886,13 @@
const base::ListValue* apis = NULL;
base::ListValue* new_apis = NULL;
- std::string granted_apis =
- JoinPrefs(kPrefGrantedPermissions, kPrefAPIs);
+ std::string granted_apis = JoinPrefs(kPrefGrantedPermissions, kPrefAPIs);
if (ext->GetList(kPrefOldGrantedAPIs, &apis))
new_apis = apis->DeepCopy();
else
new_apis = new base::ListValue();
- std::string plugin_name = info->GetByID(
- APIPermission::kPlugin)->name();
+ std::string plugin_name = info->GetByID(APIPermission::kPlugin)->name();
new_apis->Append(new base::StringValue(plugin_name));
UpdateExtensionPref(*ext_id, granted_apis, new_apis);
}
@@ -1671,7 +1668,7 @@
}
const base::DictionaryValue* ExtensionPrefs::GetInstallSignature() {
- return prefs_->GetDictionary(kInstallSignature);
+ return prefs_->GetDictionary(kInstallSignature);
}
void ExtensionPrefs::SetInstallSignature(
« no previous file with comments | « chrome/renderer/resources/extensions/permissions_custom_bindings.js ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698