[android] Resize the android_webview if it's 0x0 initially.

android_webview/native/aw_settings.cc

Index: android_webview/native/aw_settings.cc
diff --git a/android_webview/native/aw_settings.cc b/android_webview/native/aw_settings.cc
index ef10859c229cad7d911ca1003a28c147fa47e098..65277485b9c5cc3a799e746d9b3bf695136bfc7c 100644
--- a/android_webview/native/aw_settings.cc
+++ b/android_webview/native/aw_settings.cc
@@ -6,6 +6,8 @@
 
 #include "android_webview/browser/renderer_host/aw_render_view_host_ext.h"
 #include "android_webview/native/aw_contents.h"
+#include "content/public/browser/render_view_host.h"
+#include "content/public/browser/web_contents.h"
 #include "jni/AwSettings_jni.h"
 #include "webkit/glue/webkit_glue.h"
 
@@ -58,10 +60,16 @@ void AwSettings::SetTextZoom(JNIEnv* env, jobject obj, jint text_zoom_percent) {
   UpdateTextZoom();
 }
 
-void AwSettings::SetWebContents(JNIEnv* env, jobject obj, jint web_contents) {
-  Observe(reinterpret_cast<content::WebContents*>(web_contents));
-}
+void AwSettings::SetWebContents(JNIEnv* env, jobject obj, jint jweb_contents) {
+  content::WebContents* web_contents =
+      reinterpret_cast<content::WebContents*>(jweb_contents);
+  Observe(web_contents);
 
+  content::RenderViewHost* rvh = web_contents->GetRenderViewHost();
+  if (rvh) {
+    rvh->EnablePreferredSizeMode();
+  }
+}
 
 void AwSettings::UpdateEnableFixedLayoutMode() {
   AwRenderViewHostExt* rvhe = GetAwRenderViewHostExt();
@@ -95,6 +103,7 @@ void AwSettings::RenderViewCreated(content::RenderViewHost* render_view_host) {
   UpdateEnableFixedLayoutMode();
   UpdateInitialPageScale();
   UpdateTextZoom();
+  render_view_host->EnablePreferredSizeMode();

[joth, 2013/03/26 22:12:49]

for consistency I wonder about having UpdatePreferredSizeMode(), called here and
in SetWebContents

That way there's a single place to look for the policy about what it is set to
(i.e. always enabled).

also, I now wonder if the other UpdateFooBar methods need calling from the
SetWebContents? maybe check with mikhail/ben (it's mostly relevant for the
pop-up window case)

[mnaganov (inactive), 2013/03/27 11:06:31]

Yes, a good point! I have completely missed the pop-up case here. We also need
to be able to copy those setting from the embedder-provided WebView. Currently,
we only do that for the ContentSettings part.

I will shortly turn to refactoring ContentSettings / AwSettings mayhem and will
sort the things out.

 }
 
 static jint Init(JNIEnv* env,