Remove pinch viewports.

sky/engine/core/frame/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 189 matching lines @@
         return;
 
     Widget::setFrameRect(newRect);
 
     if (RenderView* renderView = this->renderView()) {
         if (renderView->usesCompositing())
             renderView->compositor()->frameViewDidChangeSize();
     }
 
     viewportConstrainedVisibleContentSizeChanged(newRect.width() != oldRect.width(), newRect.height() != oldRect.height());
-
-    if (oldRect.size() != newRect.size()
-        && m_frame->settings()->pinchVirtualViewportEnabled())
-        page()->frameHost().pinchViewport().mainFrameDidChangeSize();
 }
 
 Page* FrameView::page() const
 {
     return frame().page();
 }
 
 RenderView* FrameView::renderView() const
 {
     return frame().contentRenderer();
@@ skipping 528 matching lines @@
 LayoutRect FrameView::viewportConstrainedVisibleContentRect() const
 {
     LayoutRect viewportRect = visibleContentRect();
     // Ignore overhang. No-op when not using rubber banding.
     viewportRect.setLocation(clampScrollPosition(scrollPosition()));
     return viewportRect;
 }
 
 void FrameView::viewportConstrainedVisibleContentSizeChanged(bool widthChanged, bool heightChanged)
 {
-    if (!hasViewportConstrainedObjects())
-        return;
-
-    // If viewport is not enabled, frameRect change will cause layout size change and then layout.
-    // Otherwise, viewport constrained objects need their layout flags set separately to ensure
-    // they are positioned correctly. In the virtual-viewport pinch mode frame rect changes wont
-    // necessarily cause a layout size change so only take this early-out if we're in old-style
-    // pinch.
-    if (m_frame->settings()
-        && !m_frame->settings()->viewportEnabled()
-        && !m_frame->settings()->pinchVirtualViewportEnabled())
-        return;
-
-    ViewportConstrainedObjectSet::const_iterator end = m_viewportConstrainedObjects->end();
-    for (ViewportConstrainedObjectSet::const_iterator it = m_viewportConstrainedObjects->begin(); it != end; ++it) {
-        RenderObject* renderer = *it;
-        RenderStyle* style = renderer->style();
-        if (widthChanged) {
-            if (style->width().isFixed() && (style->left().isAuto() || style->right().isAuto()))
-                renderer->setNeedsPositionedMovementLayout();
-            else
-                renderer->setNeedsLayoutAndFullPaintInvalidation();
-        }
-        if (heightChanged) {
-            if (style->height().isFixed() && (style->top().isAuto() || style->bottom().isAuto()))
-                renderer->setNeedsPositionedMovementLayout();
-            else
-                renderer->setNeedsLayoutAndFullPaintInvalidation();
-        }
-    }
+    // FIXME(sky): Remove

[esprehn, 2014/10/25 07:33:27]

This is okay to remove because we got rid of position fixed?

[ojan, 2014/10/25 17:11:06]

I wasn't explicitly trying to break position:fixed in this patch...although it
might already be broken. In practice we were always early returning at lines
770-773 because both those settings are always false.

 }
 
 IntSize FrameView::scrollOffsetForFixedPosition() const
 {
     return toIntSize(clampScrollPosition(scrollPosition()));
 }
 
 IntPoint FrameView::lastKnownMousePosition() const
 {
     return m_frame->eventHandler().lastKnownMousePosition();
@@ skipping 77 matching lines @@
 void FrameView::scrollContentsSlowPath(const IntRect& updateRect)
 {
     // FIXME(sky): Remove
 }
 
 void FrameView::restoreScrollbar()
 {
     // FIXME(sky): Remove
 }
 
-void FrameView::scrollElementToRect(Element* element, const IntRect& rect)
-{
-    // FIXME(http://crbug.com/371896) - This method shouldn't be manually doing
-    // coordinate transformations to the PinchViewport.
-    IntRect targetRect(rect);
-
-    m_frame->document()->updateLayoutIgnorePendingStylesheets();
-
-    bool pinchVirtualViewportEnabled = m_frame->settings()->pinchVirtualViewportEnabled();
-
-    if (pinchVirtualViewportEnabled) {
-        PinchViewport& pinchViewport = m_frame->page()->frameHost().pinchViewport();
-
-        IntSize pinchViewportSize = expandedIntSize(pinchViewport.visibleRect().size());
-        targetRect.moveBy(ceiledIntPoint(pinchViewport.visibleRect().location()));
-        targetRect.setSize(pinchViewportSize.shrunkTo(targetRect.size()));
-    }
-
-    LayoutRect bounds = element->boundingBox();
-    int centeringOffsetX = (targetRect.width() - bounds.width()) / 2;
-    int centeringOffsetY = (targetRect.height() - bounds.height()) / 2;
-
-    IntPoint targetOffset(
-        bounds.x() - centeringOffsetX - targetRect.x(),
-        bounds.y() - centeringOffsetY - targetRect.y());
-
-    if (pinchVirtualViewportEnabled) {
-        IntPoint remainder = IntPoint(targetOffset - scrollPosition());
-        m_frame->page()->frameHost().pinchViewport().move(remainder);
-    }
-}
-
 // FIXME(sky): remove
 IntSize FrameView::layoutSize(IncludeScrollbarsInRect) const
 {
     return m_layoutSize;
 }
 
 void FrameView::setLayoutSize(const IntSize& size)
 {
     ASSERT(!layoutSizeFixedToFrameSize());
 
@@ skipping 427 matching lines @@
     m_inputEventsScaleFactorForEmulation = contentScaleFactor;
 }
 
 IntSize FrameView::inputEventsOffsetForEmulation() const
 {
     return m_inputEventsOffsetForEmulation;
 }
 
 float FrameView::inputEventsScaleFactor() const
 {
-    float pageScale = m_frame->settings()->pinchVirtualViewportEnabled()
-        ? m_frame->page()->frameHost().pinchViewport().scale()
-        : visibleContentScaleFactor();
+    float pageScale = visibleContentScaleFactor();
     return pageScale * m_inputEventsScaleFactorForEmulation;
 }
 
 bool FrameView::scrollbarsCanBeActive() const
 {
     if (m_frame->view() != this)
         return false;
 
     return !!m_frame->document();
 }
@@ skipping 424 matching lines @@
 void FrameView::setLayoutSizeInternal(const IntSize& size)
 {
     if (m_layoutSize == size)
         return;
 
     m_layoutSize = size;
     contentsResized();
 }
 
 } // namespace blink