Resize only the virtual viewport when the OSK triggers a resize.

content/browser/android/content_view_core_impl.cc

Index: content/browser/android/content_view_core_impl.cc
diff --git a/content/browser/android/content_view_core_impl.cc b/content/browser/android/content_view_core_impl.cc
index 273d73e6af543418acd858a2dde8f1b36bc62fcf..bce87a2fe3bcfb5cff0ce8cdf39df9d8cd692da8 100644
--- a/content/browser/android/content_view_core_impl.cc
+++ b/content/browser/android/content_view_core_impl.cc
@@ -709,6 +709,22 @@ ScopedJavaLocalRef<jobject> ContentViewCoreImpl::GetContext() const {
   return Java_ContentViewCore_getContext(env, obj.obj());
 }
 
+gfx::Size ContentViewCoreImpl::GetViewSizeWithoutOSK() const {
+  gfx::Size size_pix;
+  JNIEnv* env = AttachCurrentThread();
+  ScopedJavaLocalRef<jobject> j_obj = java_ref_.get(env);
+  if (j_obj.is_null())

[bokan, 2015/10/08 22:38:21]

I think you should skip the bits below if this is null, yes?

[ymalik, 2015/10/09 01:05:25]

Absolutely. 

+    size_pix = gfx::Size();
+  size_pix = gfx::Size(
+      Java_ContentViewCore_getViewportWidthPix(env, j_obj.obj()),
+      Java_ContentViewCore_getViewportHeightWithoutOSKPix(env, j_obj.obj()));
+
+  gfx::Size size_dip = gfx::ScaleToCeiledSize(size_pix, 1.0f / dpi_scale());

[bokan, 2015/10/08 22:38:21]

Why not just make ContentViewCore.getViewportHeightWithoutOSK return DIPs?

[ymalik, 2015/10/09 01:05:25]

It seems to be the case that ContentViewCore.getViewportHeight returns the size
in pix. Its because some callers for that need the pix value. I tried to be
consistent, but don't really have a strong preference.

+  if (DoTopControlsShrinkBlinkSize())
+    size_dip.Enlarge(0, -GetTopControlsHeightDip());
+  return size_dip;
+}
+
 gfx::Size ContentViewCoreImpl::GetViewSize() const {
   gfx::Size size = GetViewportSizeDip();
   if (DoTopControlsShrinkBlinkSize())