[Android WebView] Set WebView font renderer preferences

android_webview/native/aw_contents.cc

Index: android_webview/native/aw_contents.cc
diff --git a/android_webview/native/aw_contents.cc b/android_webview/native/aw_contents.cc
index 6b6752559922a5aab51d0eb4ec2d948ad38fec86..30fee70aec951a86cccfc6d0df8b0331fa022e13 100644
--- a/android_webview/native/aw_contents.cc
+++ b/android_webview/native/aw_contents.cc
@@ -147,8 +147,7 @@ AwContents::AwContents(scoped_ptr<WebContents> web_contents)
   if (autofill_manager_delegate)
     InitAutofillIfNecessary(autofill_manager_delegate->GetSaveFormData());
 
-  web_contents_->GetMutableRendererPrefs()->tap_multiple_targets_strategy =
-      content::TAP_MULTIPLE_TARGETS_STRATEGY_NONE;
+  SetAndroidWebViewRendererPrefs();
 }
 
 void AwContents::SetJavaPeers(JNIEnv* env,
@@ -214,6 +213,15 @@ void AwContents::InitAutofillIfNecessary(bool enabled) {
       AutofillManager::DISABLE_AUTOFILL_DOWNLOAD_MANAGER);
 }
 
+void AwContents::SetAndroidWebViewRendererPrefs() {
+  content::RendererPreferences* prefs =
+      web_contents_->GetMutableRendererPrefs();
+  prefs->tap_multiple_targets_strategy =
+      content::TAP_MULTIPLE_TARGETS_STRATEGY_NONE;
+  prefs->hinting = content::RENDERER_PREFERENCES_HINTING_SLIGHT;
+  web_contents_->GetRenderViewHost()->SyncRendererPrefs();

[boliu, 2013/08/29 16:48:31]

Hmm, I think we do have a RVH at this point, but not RV, and this just becomes a
no-op in that case? Guess that's ok.

[benm (inactive), 2013/08/29 16:53:36]

Yes, that's correct AFAIU. This doesn't crash, anyway :-)

But thinking about it, as this is now a separate function that could be called
from elsewhere, I will add the null check to be safe. Also gonna alphabetise the
prefs, it's bugging me.

[joth, 2013/08/29 18:19:40]

Sounds like AwRVHExt OnRVHCreated would be the place to do this then?

[boliu, 2013/08/29 18:26:08]

Not really. When render view is created, the existing preferences is passed rv
already, so no need to call sync there. It's only after render view is created
and we want to change the preference again that we *need* to call sync.

[benm (inactive), 2013/08/30 13:47:43]

We could move to AWRHExt::OnRVCreated, - this is actually where I had this
originally, but then noticed Bo's change for tap_targets was done here in
AwContents so moved it over. Any preference?

+}
+
 void AwContents::SetAwAutofillManagerDelegate(jobject delegate) {
   JNIEnv* env = AttachCurrentThread();
   ScopedJavaLocalRef<jobject> obj = java_ref_.get(env);