Update WebViewImpl::contentsPreferredMinimumSize to be agnostic to the scrolling element

Source/web/WebViewImpl.cpp

 /*
  * Copyright (C) 2011, 2012 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 3379 matching lines @@
         return IntSize();
     LayoutView* root = page()->deprecatedLocalMainFrame()->contentLayoutObject();
     if (!root)
         return IntSize();
     return root->documentRect().size();
 }
 
 WebSize WebViewImpl::contentsPreferredMinimumSize()
 {
     Document* document = m_page->mainFrame()->isLocalFrame() ? m_page->deprecatedLocalMainFrame()->document() : 0;
-    if (!document || !document->layoutView() || !document->documentElement())
+    if (!document || !document->layoutView() || !document->documentElement() || !document->documentElement()->layoutBox())
         return WebSize();
 
     layout();
     int widthScaled = document->layoutView()->minPreferredLogicalWidth().round(); // Already accounts for zoom.
-    int heightScaled = static_cast<int>((document->documentElement()->scrollHeight() * zoomLevelToZoomFactor(zoomLevel())) + 0.5); // +0.5 to round rather than truncating
+    LayoutBox* box = document->documentElement()->layoutBox();
+    int heightScaled = static_cast<int>(adjustLayoutUnitForAbsoluteZoom(box->scrollHeight(), *box) * zoomLevelToZoomFactor(zoomLevel()) + 0.5); // +0.5 to round rather than truncating

[bokan, 2015/08/13 15:49:04]

Did you forget to remove the second zoom factor? It looks like we're double
applying. Also, just use .round() on the layout unit instead of the cast and '+
0.5'.

[Rick Byers, 2015/08/13 16:51:54]

I was attempting to leave the behavior exactly the same as before.  I _think_
adjustLayoutUnitForAbsoluteZoom is applying only css zoom, where
zoomLevelToZoomFactor is applying page zoom.  
The result of a LayoutUnit times a double is a double.  Of course I could use
'round' instead of this ugly addition, but the cast is still necessary.

     return IntSize(widthScaled, heightScaled);
 }
 
 void WebViewImpl::enableViewport()
 {
     settings()->setViewportEnabled(true);
 }
 
 void WebViewImpl::disableViewport()
 {
@@ skipping 1085 matching lines @@
     if (m_pageColorOverlay)
         m_pageColorOverlay->update();
     if (m_inspectorOverlay) {
         PageOverlay* inspectorPageOverlay = m_inspectorOverlay->pageOverlay();
         if (inspectorPageOverlay)
             inspectorPageOverlay->update();
     }
 }
 
 } // namespace blink