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

Unified Diff: chrome/browser/renderer_preferences_util.cc

Issue 413523004: 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 | « no previous file | content/common/view_messages.h » ('j') | content/public/common/renderer_preferences.cc » ('J')
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 82a37790215ba8a94ea65a5270f7f85dd6d578c8..3867fd69da65b9284d45f5d5e4c591c5fd902030 100644
--- a/chrome/browser/renderer_preferences_util.cc
+++ b/chrome/browser/renderer_preferences_util.cc
@@ -26,50 +26,6 @@
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) {
const PrefService* pref_service = profile->GetPrefs();
@@ -134,11 +90,10 @@ void UpdateFromSystemSettings(
const gfx::FontRenderParams& params = gfx::GetDefaultWebKitFontRenderParams();
prefs->should_antialias_text = params.antialiasing;
prefs->use_subpixel_positioning = params.subpixel_positioning;
- prefs->hinting = GetRendererPreferencesHintingEnum(params.hinting);
+ prefs->hinting = params.hinting;
prefs->use_autohinter = params.autohinter;
prefs->use_bitmaps = params.use_bitmaps;
- prefs->subpixel_rendering =
- GetRendererPreferencesSubpixelRenderingEnum(params.subpixel_rendering);
+ prefs->subpixel_rendering = params.subpixel_rendering;
#endif
#if !defined(OS_MACOSX)
« no previous file with comments | « no previous file | content/common/view_messages.h » ('j') | content/public/common/renderer_preferences.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698