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

Side by Side Diff: chrome/browser/ui/tab_contents/tab_contents_wrapper.cc

Issue 6905044: Refactor preference syncing. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix the previous fix Created 9 years, 7 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/ui/tab_contents/tab_contents_wrapper.h" 5 #include "chrome/browser/ui/tab_contents/tab_contents_wrapper.h"
6 6
7 #include "base/lazy_instance.h" 7 #include "base/lazy_instance.h"
8 #include "chrome/browser/autocomplete_history_manager.h" 8 #include "chrome/browser/autocomplete_history_manager.h"
9 #include "chrome/browser/autofill/autofill_manager.h" 9 #include "chrome/browser/autofill/autofill_manager.h"
10 #include "chrome/browser/automation/automation_tab_helper.h" 10 #include "chrome/browser/automation/automation_tab_helper.h"
(...skipping 77 matching lines...) Expand 10 before | Expand all | Expand 10 after
88 TabContentsWrapper::~TabContentsWrapper() { 88 TabContentsWrapper::~TabContentsWrapper() {
89 // We don't want any notifications while we're running our destructor. 89 // We don't want any notifications while we're running our destructor.
90 registrar_.RemoveAll(); 90 registrar_.RemoveAll();
91 } 91 }
92 92
93 PropertyAccessor<TabContentsWrapper*>* TabContentsWrapper::property_accessor() { 93 PropertyAccessor<TabContentsWrapper*>* TabContentsWrapper::property_accessor() {
94 return g_tab_contents_wrapper_property_accessor.Pointer(); 94 return g_tab_contents_wrapper_property_accessor.Pointer();
95 } 95 }
96 96
97 void TabContentsWrapper::RegisterUserPrefs(PrefService* prefs) { 97 void TabContentsWrapper::RegisterUserPrefs(PrefService* prefs) {
98 prefs->RegisterBooleanPref(prefs::kAlternateErrorPagesEnabled, true); 98 prefs->RegisterBooleanPref(prefs::kAlternateErrorPagesEnabled,
99 true,
100 true /* sync pref */);
99 101
100 WebPreferences pref_defaults; 102 WebPreferences pref_defaults;
101 prefs->RegisterBooleanPref(prefs::kWebKitJavascriptEnabled, 103 prefs->RegisterBooleanPref(prefs::kWebKitJavascriptEnabled,
102 pref_defaults.javascript_enabled); 104 pref_defaults.javascript_enabled,
105 false /* don't sync pref */);
103 prefs->RegisterBooleanPref(prefs::kWebKitWebSecurityEnabled, 106 prefs->RegisterBooleanPref(prefs::kWebKitWebSecurityEnabled,
104 pref_defaults.web_security_enabled); 107 pref_defaults.web_security_enabled,
108 false /* don't sync pref */);
105 prefs->RegisterBooleanPref( 109 prefs->RegisterBooleanPref(
106 prefs::kWebKitJavascriptCanOpenWindowsAutomatically, true); 110 prefs::kWebKitJavascriptCanOpenWindowsAutomatically,
111 true,
112 false /* don't sync pref */);
107 prefs->RegisterBooleanPref(prefs::kWebKitLoadsImagesAutomatically, 113 prefs->RegisterBooleanPref(prefs::kWebKitLoadsImagesAutomatically,
108 pref_defaults.loads_images_automatically); 114 pref_defaults.loads_images_automatically,
115 false /* don't sync pref */);
109 prefs->RegisterBooleanPref(prefs::kWebKitPluginsEnabled, 116 prefs->RegisterBooleanPref(prefs::kWebKitPluginsEnabled,
110 pref_defaults.plugins_enabled); 117 pref_defaults.plugins_enabled,
118 false /* don't sync pref */);
111 prefs->RegisterBooleanPref(prefs::kWebKitDomPasteEnabled, 119 prefs->RegisterBooleanPref(prefs::kWebKitDomPasteEnabled,
112 pref_defaults.dom_paste_enabled); 120 pref_defaults.dom_paste_enabled,
121 false /* don't sync pref */);
113 prefs->RegisterBooleanPref(prefs::kWebKitShrinksStandaloneImagesToFit, 122 prefs->RegisterBooleanPref(prefs::kWebKitShrinksStandaloneImagesToFit,
114 pref_defaults.shrinks_standalone_images_to_fit); 123 pref_defaults.shrinks_standalone_images_to_fit,
115 prefs->RegisterDictionaryPref(prefs::kWebKitInspectorSettings); 124 false /* don't sync pref */);
125 prefs->RegisterDictionaryPref(prefs::kWebKitInspectorSettings,
126 false /* don't sync pref */);
116 prefs->RegisterBooleanPref(prefs::kWebKitTextAreasAreResizable, 127 prefs->RegisterBooleanPref(prefs::kWebKitTextAreasAreResizable,
117 pref_defaults.text_areas_are_resizable); 128 pref_defaults.text_areas_are_resizable,
129 false /* don't sync pref */);
118 prefs->RegisterBooleanPref(prefs::kWebKitJavaEnabled, 130 prefs->RegisterBooleanPref(prefs::kWebKitJavaEnabled,
119 pref_defaults.java_enabled); 131 pref_defaults.java_enabled,
132 false /* don't sync pref */);
120 prefs->RegisterBooleanPref(prefs::kWebkitTabsToLinks, 133 prefs->RegisterBooleanPref(prefs::kWebkitTabsToLinks,
121 pref_defaults.tabs_to_links); 134 pref_defaults.tabs_to_links,
135 false /* don't sync pref */);
122 136
137 #if !defined(OS_MACOSX)
123 prefs->RegisterLocalizedStringPref(prefs::kAcceptLanguages, 138 prefs->RegisterLocalizedStringPref(prefs::kAcceptLanguages,
124 IDS_ACCEPT_LANGUAGES); 139 IDS_ACCEPT_LANGUAGES,
140 true /* sync pref */);
141 #else
142 // Not used in OSX.
143 prefs->RegisterLocalizedStringPref(prefs::kAcceptLanguages,
144 IDS_ACCEPT_LANGUAGES,
145 false /* don't sync pref */);
146 #endif
125 prefs->RegisterLocalizedStringPref(prefs::kDefaultCharset, 147 prefs->RegisterLocalizedStringPref(prefs::kDefaultCharset,
126 IDS_DEFAULT_ENCODING); 148 IDS_DEFAULT_ENCODING,
149 true /* sync pref */);
127 prefs->RegisterLocalizedStringPref(prefs::kWebKitStandardFontFamily, 150 prefs->RegisterLocalizedStringPref(prefs::kWebKitStandardFontFamily,
128 IDS_STANDARD_FONT_FAMILY); 151 IDS_STANDARD_FONT_FAMILY,
152 false /* don't sync pref */);
129 prefs->RegisterLocalizedStringPref(prefs::kWebKitFixedFontFamily, 153 prefs->RegisterLocalizedStringPref(prefs::kWebKitFixedFontFamily,
130 IDS_FIXED_FONT_FAMILY); 154 IDS_FIXED_FONT_FAMILY,
155 false /* don't sync pref */);
131 prefs->RegisterLocalizedStringPref(prefs::kWebKitSerifFontFamily, 156 prefs->RegisterLocalizedStringPref(prefs::kWebKitSerifFontFamily,
132 IDS_SERIF_FONT_FAMILY); 157 IDS_SERIF_FONT_FAMILY,
158 false /* don't sync pref */);
133 prefs->RegisterLocalizedStringPref(prefs::kWebKitSansSerifFontFamily, 159 prefs->RegisterLocalizedStringPref(prefs::kWebKitSansSerifFontFamily,
134 IDS_SANS_SERIF_FONT_FAMILY); 160 IDS_SANS_SERIF_FONT_FAMILY,
161 false /* don't sync pref */);
135 prefs->RegisterLocalizedStringPref(prefs::kWebKitCursiveFontFamily, 162 prefs->RegisterLocalizedStringPref(prefs::kWebKitCursiveFontFamily,
136 IDS_CURSIVE_FONT_FAMILY); 163 IDS_CURSIVE_FONT_FAMILY,
164 false /* don't sync pref */);
137 prefs->RegisterLocalizedStringPref(prefs::kWebKitFantasyFontFamily, 165 prefs->RegisterLocalizedStringPref(prefs::kWebKitFantasyFontFamily,
138 IDS_FANTASY_FONT_FAMILY); 166 IDS_FANTASY_FONT_FAMILY,
167 false /* don't sync pref */);
139 prefs->RegisterLocalizedIntegerPref(prefs::kWebKitDefaultFontSize, 168 prefs->RegisterLocalizedIntegerPref(prefs::kWebKitDefaultFontSize,
140 IDS_DEFAULT_FONT_SIZE); 169 IDS_DEFAULT_FONT_SIZE,
170 false /* don't sync pref */);
141 prefs->RegisterLocalizedIntegerPref(prefs::kWebKitDefaultFixedFontSize, 171 prefs->RegisterLocalizedIntegerPref(prefs::kWebKitDefaultFixedFontSize,
142 IDS_DEFAULT_FIXED_FONT_SIZE); 172 IDS_DEFAULT_FIXED_FONT_SIZE,
173 false /* don't sync pref */);
143 prefs->RegisterLocalizedIntegerPref(prefs::kWebKitMinimumFontSize, 174 prefs->RegisterLocalizedIntegerPref(prefs::kWebKitMinimumFontSize,
144 IDS_MINIMUM_FONT_SIZE); 175 IDS_MINIMUM_FONT_SIZE,
176 false /* don't sync pref */);
145 prefs->RegisterLocalizedIntegerPref(prefs::kWebKitMinimumLogicalFontSize, 177 prefs->RegisterLocalizedIntegerPref(prefs::kWebKitMinimumLogicalFontSize,
146 IDS_MINIMUM_LOGICAL_FONT_SIZE); 178 IDS_MINIMUM_LOGICAL_FONT_SIZE,
179 false /* don't sync pref */);
147 prefs->RegisterLocalizedBooleanPref(prefs::kWebKitUsesUniversalDetector, 180 prefs->RegisterLocalizedBooleanPref(prefs::kWebKitUsesUniversalDetector,
148 IDS_USES_UNIVERSAL_DETECTOR); 181 IDS_USES_UNIVERSAL_DETECTOR,
182 true /* sync pref */);
149 prefs->RegisterLocalizedStringPref(prefs::kStaticEncodings, 183 prefs->RegisterLocalizedStringPref(prefs::kStaticEncodings,
150 IDS_STATIC_ENCODING_LIST); 184 IDS_STATIC_ENCODING_LIST,
151 prefs->RegisterStringPref(prefs::kRecentlySelectedEncoding, ""); 185 false /* don't sync pref */);
186 prefs->RegisterStringPref(prefs::kRecentlySelectedEncoding,
187 "",
188 false /* don't sync pref */);
152 } 189 }
153 190
154 string16 TabContentsWrapper::GetDefaultTitle() { 191 string16 TabContentsWrapper::GetDefaultTitle() {
155 return l10n_util::GetStringUTF16(IDS_DEFAULT_TAB_TITLE); 192 return l10n_util::GetStringUTF16(IDS_DEFAULT_TAB_TITLE);
156 } 193 }
157 194
158 string16 TabContentsWrapper::GetStatusText() const { 195 string16 TabContentsWrapper::GetStatusText() const {
159 if (!tab_contents()->is_loading() || 196 if (!tab_contents()->is_loading() ||
160 tab_contents()->load_state() == net::LOAD_STATE_IDLE) { 197 tab_contents()->load_state() == net::LOAD_STATE_IDLE) {
161 return string16(); 198 return string16();
(...skipping 159 matching lines...) Expand 10 before | Expand all | Expand 10 after
321 } 358 }
322 359
323 void TabContentsWrapper::UpdateStarredStateForCurrentURL() { 360 void TabContentsWrapper::UpdateStarredStateForCurrentURL() {
324 BookmarkModel* model = tab_contents()->profile()->GetBookmarkModel(); 361 BookmarkModel* model = tab_contents()->profile()->GetBookmarkModel();
325 const bool old_state = is_starred_; 362 const bool old_state = is_starred_;
326 is_starred_ = (model && model->IsBookmarked(tab_contents()->GetURL())); 363 is_starred_ = (model && model->IsBookmarked(tab_contents()->GetURL()));
327 364
328 if (is_starred_ != old_state && delegate()) 365 if (is_starred_ != old_state && delegate())
329 delegate()->URLStarredChanged(this, is_starred_); 366 delegate()->URLStarredChanged(this, is_starred_);
330 } 367 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698