 Chromium Code Reviews
 Chromium Code Reviews Issue 33753004:
  Add link to the Advanced Font Settings extension in Font Settings  (Closed) 
  Base URL: svn://svn.chromium.org/chrome/trunk/src
    
  
    Issue 33753004:
  Add link to the Advanced Font Settings extension in Font Settings  (Closed) 
  Base URL: svn://svn.chromium.org/chrome/trunk/src| Index: chrome/browser/ui/webui/options/font_settings_handler.h | 
| diff --git a/chrome/browser/ui/webui/options/font_settings_handler.h b/chrome/browser/ui/webui/options/font_settings_handler.h | 
| index 32ca8c49d2a37ff1fd5b573931b3532e561a56b9..1f779c47c7a974662a33c405e2f9f6f9c989d87e 100644 | 
| --- a/chrome/browser/ui/webui/options/font_settings_handler.h | 
| +++ b/chrome/browser/ui/webui/options/font_settings_handler.h | 
| @@ -13,6 +13,10 @@ namespace base { | 
| class ListValue; | 
| } | 
| +namespace extensions { | 
| +class Extension; | 
| +} | 
| + | 
| namespace options { | 
| // Font settings overlay page UI handler. | 
| @@ -23,12 +27,18 @@ class FontSettingsHandler : public OptionsPageUIHandler { | 
| // OptionsPageUIHandler implementation. | 
| virtual void GetLocalizedValues(DictionaryValue* localized_strings) OVERRIDE; | 
| + virtual void InitializeHandler() OVERRIDE; | 
| virtual void InitializePage() OVERRIDE; | 
| // WebUIMessageHandler implementation. | 
| virtual void RegisterMessages() OVERRIDE; | 
| private: | 
| + // content::NotificationObserver (from OptionsPageUIHandler) implementation | 
| 
Dan Beam
2013/10/23 18:04:00
// OptionsPageUIHandler implementation.
 
falken
2013/10/24 12:09:41
Done.
 | 
| + virtual void Observe(int type, | 
| + const content::NotificationSource& source, | 
| + const content::NotificationDetails& details) OVERRIDE; | 
| + | 
| void HandleFetchFontsData(const ListValue* args); | 
| void FontsListHasLoaded(scoped_ptr<base::ListValue> list); | 
| @@ -38,6 +48,11 @@ class FontSettingsHandler : public OptionsPageUIHandler { | 
| void SetUpSansSerifFontSample(); | 
| void SetUpFixedFontSample(); | 
| void SetUpMinimumFontSample(); | 
| + | 
| 
Dan Beam
2013/10/23 18:04:00
though the rest of this class is lacking in docume
 
falken
2013/10/24 12:09:41
Done.
 | 
| + const extensions::Extension* GetAdvancedFontSettingsExtension(); | 
| + void NotifyAdvancedFontSettingsAvailability(); | 
| + void OpenAdvancedFontSettingsOptions(const base::ListValue* args); | 
| + | 
| void OnWebKitDefaultFontSizeChanged(); | 
| StringPrefMember standard_font_; | 
| @@ -49,6 +64,8 @@ class FontSettingsHandler : public OptionsPageUIHandler { | 
| IntegerPrefMember default_fixed_font_size_; | 
| IntegerPrefMember minimum_font_size_; | 
| + content::NotificationRegistrar registrar_; | 
| + | 
| DISALLOW_COPY_AND_ASSIGN(FontSettingsHandler); | 
| }; |