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

Unified Diff: android_webview/native/aw_settings.cc

Issue 413523004: content: Remove RendererPreferences font-rendering enums. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix android compile, hopefully 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 | chrome/browser/renderer_preferences_util.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: android_webview/native/aw_settings.cc
diff --git a/android_webview/native/aw_settings.cc b/android_webview/native/aw_settings.cc
index 55b449b82622b75d296a754963f2bc83b99d5732..d28afaa28379a904d39bda6463b1df63ac993ea9 100644
--- a/android_webview/native/aw_settings.cc
+++ b/android_webview/native/aw_settings.cc
@@ -29,46 +29,6 @@ namespace android_webview {
namespace {
-// TODO(boliu): Deduplicate with chrome/ code.
-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;
- }
-}
-
-// TODO(boliu): Deduplicate with chrome/ code.
-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;
- }
-}
-
void PopulateFixedRendererPreferences(RendererPreferences* prefs) {
prefs->tap_multiple_targets_strategy =
content::TAP_MULTIPLE_TARGETS_STRATEGY_NONE;
@@ -77,11 +37,10 @@ void PopulateFixedRendererPreferences(RendererPreferences* prefs) {
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;
}
void PopulateFixedWebPreferences(WebPreferences* web_prefs) {
« no previous file with comments | « no previous file | chrome/browser/renderer_preferences_util.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698