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

Unified Diff: ui/gfx/font_render_params.h

Issue 382273002: ui/gfx: Allow for font-specific rendering settings. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: address review feedback Created 6 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
Index: ui/gfx/font_render_params.h
diff --git a/ui/gfx/font_render_params.h b/ui/gfx/font_render_params.h
index d4a224cd07a4972affd9df6483e3b9eaaece04c4..1df2c1073e08df6404f4609b656253ead5b30104 100644
--- a/ui/gfx/font_render_params.h
+++ b/ui/gfx/font_render_params.h
@@ -5,6 +5,9 @@
#ifndef UI_GFX_FONT_RENDER_PARAMS_H_
#define UI_GFX_FONT_RENDER_PARAMS_H_
+#include <string>
+#include <vector>
+
#include "ui/gfx/gfx_export.h"
namespace gfx {
@@ -58,13 +61,18 @@ struct GFX_EXPORT FontRenderParams {
GFX_EXPORT const FontRenderParams& GetDefaultFontRenderParams();
// Returns the system's default parameters for WebKit font rendering.
+// TODO(derat): s/WebKit/Blink/
GFX_EXPORT const FontRenderParams& GetDefaultWebKitFontRenderParams();
-// Returns the system's default parameters for WebKit subpixel positioning.
-// Subpixel positioning is special since neither GTK nor FontConfig currently
-// track it as a preference.
-// See https://bugs.freedesktop.org/show_bug.cgi?id=50736
-GFX_EXPORT bool GetDefaultWebkitSubpixelPositioning();
+// Returns the appropriate parameters for rendering the font described by the
+// passed-in-arguments, any of which may be NULL. If |family_out| is non-NULL,
+// it will be updated to contain the recommended font family from |family_list|.
+GFX_EXPORT FontRenderParams GetCustomFontRenderParams(
+ bool for_web_contents,
+ const std::vector<std::string>* family_list,
+ const int* pixel_size,
+ const int* point_size,
+ std::string* family_out);
} // namespace gfx

Powered by Google App Engine
This is Rietveld 408576698