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

Unified Diff: chrome/browser/extensions/extension_prefs.cc

Issue 3056003: Attemp 2 at: (Closed)
Patch Set: Fix chromeos breakage Created 10 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/extensions/extension_prefs.h ('k') | chrome/browser/extensions/extension_prefs_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/extensions/extension_prefs.cc
diff --git a/chrome/browser/extensions/extension_prefs.cc b/chrome/browser/extensions/extension_prefs.cc
index 93f29c41a62093b9a8a0c69390daa373ecadada5..ed44ec426b623c06ee47e28e5b2bff61686f7618 100644
--- a/chrome/browser/extensions/extension_prefs.cc
+++ b/chrome/browser/extensions/extension_prefs.cc
@@ -35,9 +35,6 @@ const wchar_t kPrefVersion[] = L"manifest.version";
// Indicates if an extension is blacklisted:
const wchar_t kPrefBlacklist[] = L"blacklist";
-// Indicates whether the toolbar should be shown on app tabs.
-const wchar_t kPrefAppTabToolbars[] = L"app_tab_toolbars";
-
// Indicates whether to show an install warning when the user enables.
const wchar_t kExtensionDidEscalatePermissions[] = L"install_warning_on_enable";
@@ -780,32 +777,6 @@ std::set<std::string> ExtensionPrefs::GetIdleInstallInfoIds() {
return result;
}
-bool ExtensionPrefs::AreAppTabToolbarsVisible(
- const std::string& extension_id) {
- // Default to hiding toolbars.
- bool show_toolbars = false;
- DictionaryValue* pref = GetExtensionPref(extension_id);
- if (!pref)
- return show_toolbars;
-
- pref->GetBoolean(
- ASCIIToWide(extension_id) + L"." + kPrefAppTabToolbars, &show_toolbars);
- return show_toolbars;
-}
-
-void ExtensionPrefs::SetAppTabToolbarVisibility(
- const std::string& extension_id, bool value) {
- DictionaryValue* pref = GetOrCreateExtensionPref(extension_id);
- std::wstring key = ASCIIToWide(extension_id) + L"." + kPrefAppTabToolbars;
-
- if (value)
- pref->SetBoolean(key, true);
- else
- pref->Remove(key, NULL); // False is the default value.
-
- prefs_->ScheduleSavePersistentPrefs();
-}
-
// static
void ExtensionPrefs::RegisterUserPrefs(PrefService* prefs) {
« no previous file with comments | « chrome/browser/extensions/extension_prefs.h ('k') | chrome/browser/extensions/extension_prefs_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698