Enabled using viewport on desktop browsers behind experimental flag

Source/core/page/FrameView.cpp

 /*
  * Copyright (C) 1998, 1999 Torben Weis <weis@kde.org>
  *                     1999 Lars Knoll <knoll@kde.org>
  *                     1999 Antti Koivisto <koivisto@kde.org>
  *                     2000 Dirk Mueller <mueller@kde.org>
  * Copyright (C) 2004, 2005, 2006, 2007, 2008 Apple Inc. All rights reserved.
  *           (C) 2006 Graham Dennis (graham.dennis@gmail.com)
  *           (C) 2006 Alexey Proskuryakov (ap@nypop.com)
  * Copyright (C) 2009 Google Inc. All rights reserved.
  *
@@ skipping 456 matching lines @@
     // make it update the rendering area when set
     m_margins.setWidth(w);
 }
 
 void FrameView::setMarginHeight(LayoutUnit h)
 {
     // make it update the rendering area when set
     m_margins.setHeight(h);
 }
 
+IntSize FrameView::layoutSize(VisibleContentRectIncludesScrollbars scrollbarInclusion) const
+{
+    if (m_layoutSize.isEmpty())
+        return ScrollView::layoutSize(scrollbarInclusion);

[aelias_OOO_until_Jul13, 2013/07/31 22:31:21]

I don't think we should keep using the ScrollView::layoutSize logic on
FrameViews.  Just always return m_layoutSize.  And make sure that it's always
set correctly (for example, call it on every WebViewImpl::resize()).

+
+    return m_layoutSize;
+}
+
+void FrameView::setLayoutSize(const IntSize& newSize)

[aelias_OOO_until_Jul13, 2013/07/31 22:31:21]

Could you just layout() when this is called and remove all other layout calls
from FrameView?

+{
+    if (m_layoutSize == newSize)
+        return;
+
+    m_layoutSize = newSize;
+    updateScrollbars(scrollOffset());
+    contentsResized();
+}
+
 void FrameView::setCanHaveScrollbars(bool canHaveScrollbars)
 {
     m_canHaveScrollbars = canHaveScrollbars;
     ScrollView::setCanHaveScrollbars(canHaveScrollbars);
 }
 
 void FrameView::updateCanHaveScrollbars()
 {
     ScrollbarMode hMode;
     ScrollbarMode vMode;
@@ skipping 1218 matching lines @@
 
 void FrameView::visibleContentsResized()
 {
     // We check to make sure the view is attached to a frame() as this method can
     // be triggered before the view is attached by Frame::createView(...) setting
     // various values such as setScrollBarModes(...) for example.  An ASSERT is
     // triggered when a view is layout before being attached to a frame().
     if (!frame()->view())
         return;
 
-    if (!useFixedLayout() && needsLayout())
+    if (m_layoutSize.isEmpty() && needsLayout())
         layout();
 
     if (RenderView* renderView = this->renderView()) {
         if (renderView->usesCompositing())
             renderView->compositor()->frameViewDidChangeSize();
     }
 }
 
 void FrameView::beginDeferredRepaints()
 {
@@ skipping 1647 matching lines @@
 }
 
 AXObjectCache* FrameView::axObjectCache() const
 {
     if (frame() && frame()->document())
         return frame()->document()->existingAXObjectCache();
     return 0;
 }
 
 } // namespace WebCore