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

Unified Diff: chrome/browser/renderer_preferences_util.cc

Issue 410413008: Revert of content: Remove RendererPreferences font-rendering enums. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: 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
« no previous file with comments | « android_webview/native/aw_settings.cc ('k') | content/common/view_messages.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/renderer_preferences_util.cc
diff --git a/chrome/browser/renderer_preferences_util.cc b/chrome/browser/renderer_preferences_util.cc
index 3867fd69da65b9284d45f5d5e4c591c5fd902030..82a37790215ba8a94ea65a5270f7f85dd6d578c8 100644
--- a/chrome/browser/renderer_preferences_util.cc
+++ b/chrome/browser/renderer_preferences_util.cc
@@ -25,6 +25,50 @@
#endif
namespace renderer_preferences_util {
+
+namespace {
+
+#if defined(OS_LINUX) || defined(OS_ANDROID)
+content::RendererPreferencesHintingEnum GetRendererPreferencesHintingEnum(
+ gfx::FontRenderParams::Hinting hinting) {
+ switch (hinting) {
+ case gfx::FontRenderParams::HINTING_NONE:
+ return content::RENDERER_PREFERENCES_HINTING_NONE;
+ case gfx::FontRenderParams::HINTING_SLIGHT:
+ return content::RENDERER_PREFERENCES_HINTING_SLIGHT;
+ case gfx::FontRenderParams::HINTING_MEDIUM:
+ return content::RENDERER_PREFERENCES_HINTING_MEDIUM;
+ case gfx::FontRenderParams::HINTING_FULL:
+ return content::RENDERER_PREFERENCES_HINTING_FULL;
+ default:
+ NOTREACHED() << "Unhandled hinting style " << hinting;
+ return content::RENDERER_PREFERENCES_HINTING_SYSTEM_DEFAULT;
+ }
+}
+
+content::RendererPreferencesSubpixelRenderingEnum
+GetRendererPreferencesSubpixelRenderingEnum(
+ gfx::FontRenderParams::SubpixelRendering subpixel_rendering) {
+ switch (subpixel_rendering) {
+ case gfx::FontRenderParams::SUBPIXEL_RENDERING_NONE:
+ return content::RENDERER_PREFERENCES_SUBPIXEL_RENDERING_NONE;
+ case gfx::FontRenderParams::SUBPIXEL_RENDERING_RGB:
+ return content::RENDERER_PREFERENCES_SUBPIXEL_RENDERING_RGB;
+ case gfx::FontRenderParams::SUBPIXEL_RENDERING_BGR:
+ return content::RENDERER_PREFERENCES_SUBPIXEL_RENDERING_BGR;
+ case gfx::FontRenderParams::SUBPIXEL_RENDERING_VRGB:
+ return content::RENDERER_PREFERENCES_SUBPIXEL_RENDERING_VRGB;
+ case gfx::FontRenderParams::SUBPIXEL_RENDERING_VBGR:
+ return content::RENDERER_PREFERENCES_SUBPIXEL_RENDERING_VBGR;
+ default:
+ NOTREACHED() << "Unhandled subpixel rendering style "
+ << subpixel_rendering;
+ return content::RENDERER_PREFERENCES_SUBPIXEL_RENDERING_SYSTEM_DEFAULT;
+ }
+}
+#endif // defined(OS_LINUX) || defined(OS_ANDROID)
+
+} // namespace
void UpdateFromSystemSettings(
content::RendererPreferences* prefs, Profile* profile) {
@@ -90,10 +134,11 @@
const gfx::FontRenderParams& params = gfx::GetDefaultWebKitFontRenderParams();
prefs->should_antialias_text = params.antialiasing;
prefs->use_subpixel_positioning = params.subpixel_positioning;
- prefs->hinting = params.hinting;
+ prefs->hinting = GetRendererPreferencesHintingEnum(params.hinting);
prefs->use_autohinter = params.autohinter;
prefs->use_bitmaps = params.use_bitmaps;
- prefs->subpixel_rendering = params.subpixel_rendering;
+ prefs->subpixel_rendering =
+ GetRendererPreferencesSubpixelRenderingEnum(params.subpixel_rendering);
#endif
#if !defined(OS_MACOSX)
« no previous file with comments | « android_webview/native/aw_settings.cc ('k') | content/common/view_messages.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698