Index: chrome/browser/extensions/extension_prefs.cc |
diff --git a/chrome/browser/extensions/extension_prefs.cc b/chrome/browser/extensions/extension_prefs.cc |
index 2246e49a0413563493c7c38cd8ac057d76f6d812..09fad3346b358ef83bd700cb05a42fe2bf4f109a 100644 |
--- a/chrome/browser/extensions/extension_prefs.cc |
+++ b/chrome/browser/extensions/extension_prefs.cc |
@@ -280,6 +280,16 @@ bool ScopeToPrefKey(ExtensionPrefsScope scope, std::string* result) { |
return false; |
} |
+const char* GetToolbarOrderKeyName() { |
+ return switch_utils::IsExtensionsInActionBoxEnabled() ? |
+ kExtensionActionBoxBar : kExtensionToolbar; |
+} |
+ |
+const char* GetToolbarVisibilityKeyName() { |
+ return switch_utils::IsExtensionsInActionBoxEnabled() ? |
+ kBrowserActionPinned : kBrowserActionVisible; |
+} |
+ |
} // namespace |
ExtensionPrefs::ExtensionPrefs( |
@@ -1318,14 +1328,11 @@ bool ExtensionPrefs::IsExtensionDisabled( |
} |
ExtensionPrefs::ExtensionIdSet ExtensionPrefs::GetToolbarOrder() { |
- bool action_box_enabled = extensions::switch_utils::IsActionBoxEnabled(); |
- return GetExtensionPrefAsVector( |
- action_box_enabled ? kExtensionActionBoxBar : kExtensionToolbar); |
+ return GetExtensionPrefAsVector(GetToolbarOrderKeyName()); |
} |
void ExtensionPrefs::SetToolbarOrder(const ExtensionIdSet& extension_ids) { |
- SetExtensionPrefFromVector(extensions::switch_utils::IsActionBoxEnabled() ? |
- kExtensionActionBoxBar : kExtensionToolbar, extension_ids); |
+ SetExtensionPrefFromVector(GetToolbarOrderKeyName(), extension_ids); |
} |
ExtensionPrefs::ExtensionIdSet ExtensionPrefs::GetActionBoxOrder() { |
@@ -1421,17 +1428,17 @@ void ExtensionPrefs::SetExtensionState(const std::string& extension_id, |
} |
bool ExtensionPrefs::GetBrowserActionVisibility(const Extension* extension) { |
- bool action_box_enabled = switch_utils::IsActionBoxEnabled(); |
- bool default_value = !action_box_enabled; |
+ bool extensions_in_action_box_enabled = |
+ switch_utils::IsExtensionsInActionBoxEnabled(); |
+ bool default_value = !extensions_in_action_box_enabled; |
const DictionaryValue* extension_prefs = GetExtensionPref(extension->id()); |
if (!extension_prefs) |
return default_value; |
bool visible = false; |
- const char* browser_action_pref = action_box_enabled ? kBrowserActionPinned : |
- kBrowserActionVisible; |
- bool pref_exists = extension_prefs->GetBoolean(browser_action_pref, &visible); |
+ bool pref_exists = extension_prefs->GetBoolean(GetToolbarVisibilityKeyName(), |
+ &visible); |
if (!pref_exists) |
return default_value; |
@@ -1443,10 +1450,7 @@ void ExtensionPrefs::SetBrowserActionVisibility(const Extension* extension, |
if (GetBrowserActionVisibility(extension) == visible) |
return; |
- bool action_box_enabled = switch_utils::IsActionBoxEnabled(); |
- const char* browser_action_pref = action_box_enabled ? kBrowserActionPinned : |
- kBrowserActionVisible; |
- UpdateExtensionPref(extension->id(), browser_action_pref, |
+ UpdateExtensionPref(extension->id(), GetToolbarVisibilityKeyName(), |
Value::CreateBooleanValue(visible)); |
content::NotificationService::current()->Notify( |
chrome::NOTIFICATION_EXTENSION_BROWSER_ACTION_VISIBILITY_CHANGED, |