Expose more font size prefs to the Font Settings Extension API.

chrome/browser/extensions/extension_font_settings_api.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_EXTENSIONS_EXTENSION_FONT_SETTINGS_API_H__
 #define CHROME_BROWSER_EXTENSIONS_EXTENSION_FONT_SETTINGS_API_H__
 #pragma once
 
 #include "chrome/browser/extensions/extension_function.h"
 
@@ skipping 12 matching lines @@
 class GetFontListFunction : public AsyncExtensionFunction {
  public:
   virtual bool RunImpl() OVERRIDE;
   DECLARE_EXTENSION_FUNCTION_NAME("experimental.fontSettings.getFontList")
 
  private:
   void FontListHasLoaded(scoped_ptr<base::ListValue> list);
   bool CopyFontsToResult(base::ListValue* fonts);
 };
 
-class GetDefaultFontSizeFunction : public SyncExtensionFunction {
+// Base class for functions that get a font size.
+class GetFontSizeExtensionFunction : public SyncExtensionFunction {
+ protected:
+  virtual bool RunImpl() OVERRIDE;
+
+  // Implementations should return the name of the font size preference to get.
+  virtual const char* pref_name() = 0;

[Matt Perry, 2012/03/26 18:53:05]

underscore_style() is reserved for inline methods. Use GetPrefName()

[falken, 2012/03/27 02:26:01]

Done.

+};
+
+// Base class for functions that set a font size.
+class SetFontSizeExtensionFunction : public SyncExtensionFunction {
+ protected:
+  virtual bool RunImpl() OVERRIDE;
+
+  // Implementations should return the name of the font size preference to set.
+  virtual const char* pref_name() = 0;
+};
+
+class GetDefaultFontSizeFunction : public GetFontSizeExtensionFunction {
  public:
-  virtual bool RunImpl() OVERRIDE;
   DECLARE_EXTENSION_FUNCTION_NAME(
       "experimental.fontSettings.getDefaultFontSize")
+
+ protected:
+  virtual const char* pref_name() OVERRIDE;
 };
 
-class SetDefaultFontSizeFunction : public SyncExtensionFunction {
+class SetDefaultFontSizeFunction : public SetFontSizeExtensionFunction {
  public:
-  virtual bool RunImpl() OVERRIDE;
   DECLARE_EXTENSION_FUNCTION_NAME(
       "experimental.fontSettings.setDefaultFontSize")
+
+ protected:
+  virtual const char* pref_name() OVERRIDE;
+};
+
+class GetDefaultFixedFontSizeFunction : public GetFontSizeExtensionFunction {
+ public:
+  DECLARE_EXTENSION_FUNCTION_NAME(
+      "experimental.fontSettings.getDefaultFixedFontSize")
+
+ protected:
+  virtual const char* pref_name() OVERRIDE;
+};
+
+class SetDefaultFixedFontSizeFunction : public SetFontSizeExtensionFunction {
+ public:
+  DECLARE_EXTENSION_FUNCTION_NAME(
+      "experimental.fontSettings.setDefaultFixedFontSize")
+
+ protected:
+  virtual const char* pref_name() OVERRIDE;
+};
+
+class GetMinimumFontSizeFunction : public GetFontSizeExtensionFunction {
+ public:
+  DECLARE_EXTENSION_FUNCTION_NAME(
+      "experimental.fontSettings.getMinimumFontSize")
+
+ protected:
+  virtual const char* pref_name() OVERRIDE;
+};
+
+class SetMinimumFontSizeFunction : public SetFontSizeExtensionFunction {
+ public:
+  DECLARE_EXTENSION_FUNCTION_NAME(
+      "experimental.fontSettings.setMinimumFontSize")
+
+ protected:
+  virtual const char* pref_name() OVERRIDE;
 };
 
 #endif  // CHROME_BROWSER_EXTENSIONS_EXTENSION_FONT_SETTINGS_API_H__