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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: chrome/browser/ui/website_settings/website_settings_ui.h

Issue 2732963005: Views/Permissions: Use shorter strings for site setting drop downs. (Closed)
Patch Set: Change all references to |info| in the constructor to references to |permission_| to be consistent. Created 3 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
« no previous file with comments | « chrome/browser/ui/website_settings/permission_menu_model.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/website_settings/website_settings_ui.h
diff --git a/chrome/browser/ui/website_settings/website_settings_ui.h b/chrome/browser/ui/website_settings/website_settings_ui.h
index 5689d53837616bd889966ce183b149b0e2c3a438..16bc72750d4feb9a209f8bdb4197c200e1ecd15d 100644
--- a/chrome/browser/ui/website_settings/website_settings_ui.h
+++ b/chrome/browser/ui/website_settings/website_settings_ui.h
@@ -146,7 +146,8 @@ class WebsiteSettingsUI {
// Returns the UI string describing the action taken for a permission,
// including why that action was taken. E.g. "Allowed by you",
- // "Blocked by default".
+ // "Blocked by default". If |setting| is default, specify the actual default
+ // setting using |default_setting|.
static base::string16 PermissionActionToUIString(
Profile* profile,
ContentSettingsType type,
« no previous file with comments | « chrome/browser/ui/website_settings/permission_menu_model.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698