Don't calculate a top controls adjustment if pinch viewport size is uninitialized.

Source/web/WebViewImpl.cpp

Index: Source/web/WebViewImpl.cpp
diff --git a/Source/web/WebViewImpl.cpp b/Source/web/WebViewImpl.cpp
index bb5da5197c3c6a016ec05a928e8a3939831f8384..22a3c370d51a7c3675ac7e423d290527402fdaf1 100644
--- a/Source/web/WebViewImpl.cpp
+++ b/Source/web/WebViewImpl.cpp
@@ -1729,6 +1729,10 @@ void WebViewImpl::didUpdateTopControls()
         view->setTopControlsViewportAdjustment(topControlsViewportAdjustment);
     } else {
         PinchViewport& pinchViewport = page()->frameHost().pinchViewport();
+
+        if (!pinchViewport.visibleRect().width() || !pinchViewport.visibleRect().height())

[Peter Kasting, 2014/10/22 19:32:21]

Can this just be "if (pinchViewport.visibleRect().isEmpty())"?  Or did you
actually want negative-size cases to proceed onwards?

[bokan, 2014/10/22 20:08:53]

Done.

+            return;
+
         pinchViewport.setTopControlsAdjustment(topControlsViewportAdjustment);
 
         // Shrink the FrameView by the amount that will maintain the aspect-ratio with the PinchViewport.