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

Unified Diff: chrome/browser/extensions/api/font_settings/font_settings_api.cc

Issue 11366074: Coalesce event router Init calls into their constructors. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: aa Created 8 years, 1 month 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: chrome/browser/extensions/api/font_settings/font_settings_api.cc
diff --git a/chrome/browser/extensions/api/font_settings/font_settings_api.cc b/chrome/browser/extensions/api/font_settings/font_settings_api.cc
index 053e417839f4828fe33ceda934af9d5219a3f5d7..a3ec6519a431379c1b2e8828a5eec13a18deede2 100644
--- a/chrome/browser/extensions/api/font_settings/font_settings_api.cc
+++ b/chrome/browser/extensions/api/font_settings/font_settings_api.cc
@@ -98,11 +98,7 @@ void RegisterFontFamilyMapObserver(PrefChangeRegistrar* registrar,
} // namespace
FontSettingsEventRouter::FontSettingsEventRouter(
- Profile* profile) : profile_(profile) {}
-
-FontSettingsEventRouter::~FontSettingsEventRouter() {}
-
-void FontSettingsEventRouter::Init() {
+ Profile* profile) : profile_(profile) {
registrar_.Init(profile_->GetPrefs());
AddPrefToObserve(prefs::kWebKitDefaultFixedFontSize,
@@ -131,6 +127,8 @@ void FontSettingsEventRouter::Init() {
prefs::kWebKitPictographFontFamilyMap, this);
}
+FontSettingsEventRouter::~FontSettingsEventRouter() {}
+
void FontSettingsEventRouter::AddPrefToObserve(const char* pref_name,
const char* event_name,
const char* key) {

Powered by Google App Engine
This is Rietveld 408576698