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

Unified Diff: extensions/browser/api/management/management_api.cc

Issue 980353003: Extensions: Switch to new permission message system, part I (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: cleanup;rebase Created 5 years, 9 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
Index: extensions/browser/api/management/management_api.cc
diff --git a/extensions/browser/api/management/management_api.cc b/extensions/browser/api/management/management_api.cc
index c2fb30d84ef6d8359c8506334589d27400e57a6a..cdd7e4529c0c200ae7a1ca7d40f91e2dbd76fa61 100644
--- a/extensions/browser/api/management/management_api.cc
+++ b/extensions/browser/api/management/management_api.cc
@@ -61,12 +61,10 @@ AutoConfirmForTest auto_confirm_for_test = DO_NOT_SKIP;
std::vector<std::string> CreateWarningsList(const Extension* extension) {
std::vector<std::string> warnings_list;
- PermissionMessages warnings =
- extension->permissions_data()->GetPermissionMessages();
- for (PermissionMessages::const_iterator iter = warnings.begin();
- iter != warnings.end(); ++iter) {
- warnings_list.push_back(base::UTF16ToUTF8(iter->message()));
- }
+ CoalescedPermissionMessages warnings =
+ extension->permissions_data()->GetCoalescedPermissionMessages();
+ for (const CoalescedPermissionMessage& message : warnings)
+ warnings_list.push_back(base::UTF16ToUTF8(message.message()));
return warnings_list;
}

Powered by Google App Engine
This is Rietveld 408576698