Insert pinch zoom virtual viewport layers to graphics layer tree.

Source/WebKit/chromium/src/PinchViewports.cpp

+/*
+ * Copyright (C) 2013 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
+ * copyright notice, this list of conditions and the following disclaimer
+ * in the documentation and/or other materials provided with the
+ * distribution.
+ *     * Neither the name of Google Inc. nor the names of its
+ * contributors may be used to endorse or promote products derived from
+ * this software without specific prior written permission.
+ *
+ * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
+ * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
+ * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
+ * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
+ * OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
+ * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
+ * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
+ * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
+ * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
+ * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
+ * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+ */
+
+#include "config.h"
+#include "WebKit/chromium/src/PinchViewports.h"
+
+#include "WebKit/chromium/src/WebSettingsImpl.h"
+#include "WebKit/chromium/src/WebViewImpl.h"
+#include "core/page/FrameView.h"
+#include "core/platform/graphics/GraphicsLayer.h"
+#include "core/rendering/RenderLayerCompositor.h"
+#include "public/platform/Platform.h"
+#include "public/platform/WebCompositorSupport.h"
+#include "public/platform/WebLayerTreeView.h"
+#include "public/platform/WebScrollbarLayer.h"
+
+namespace WebKit {
+
+WTF::PassOwnPtr<PinchViewports> PinchViewports::create(WebViewImpl* owner)

[jamesr, 2013/06/17 20:42:10]

No WTF::

[wjmaclean, 2013/06/17 22:09:04]

Done.

+{
+    return adoptPtr(new PinchViewports(owner));
+}
+
+PinchViewports::PinchViewports(WebViewImpl* owner)
+    : m_owner(owner)
+    , m_innerViewportClipLayer(GraphicsLayer::create(m_owner->graphicsLayerFactory(), this))
+    , m_pageScaleLayer(GraphicsLayer::create(m_owner->graphicsLayerFactory(), this))
+    , m_innerViewportScrollLayer(GraphicsLayer::create(m_owner->graphicsLayerFactory(), this))
+    , m_overlayScrollbarHorizontal(GraphicsLayer::create(m_owner->graphicsLayerFactory(), this))
+    , m_overlayScrollbarVertical(GraphicsLayer::create(m_owner->graphicsLayerFactory(), this))
+{
+    m_innerViewportClipLayer->setMasksToBounds(true);
+    m_innerViewportClipLayer->platformLayer()->setIsContainerForFixedPositionLayers(true);
+
+    m_innerViewportScrollLayer->platformLayer()->setScrollable(true);
+
+#ifndef NDEBUG
+    m_innerViewportClipLayer->setName("inner viewport clip layer");
+    m_pageScaleLayer->setName("page scale layer");
+    m_innerViewportScrollLayer->setName("inner viewport scroll layer");
+    m_overlayScrollbarHorizontal->setName("overlay scrollbar horizontal");
+    m_overlayScrollbarVertical->setName("overlay scrollbar vertical");
+#endif
+
+    m_innerViewportClipLayer->addChild(m_pageScaleLayer.get());
+    m_pageScaleLayer->addChild(m_innerViewportScrollLayer.get());
+    m_innerViewportClipLayer->addChild(m_overlayScrollbarHorizontal.get());
+    m_innerViewportClipLayer->addChild(m_overlayScrollbarVertical.get());
+
+    // Setup the inner viewport overlay scrollbars.
+    setupScrollbar(WebScrollbar::Horizontal);
+    setupScrollbar(WebScrollbar::Vertical);
+}
+
+PinchViewports::~PinchViewports() { }
+
+void PinchViewports::setViewportSize(const WebCore::FloatSize& newSize)
+{
+    ASSERT(m_innerViewportClipLayer);
+    m_innerViewportClipLayer->setSize(newSize);
+
+    // Need to re-compute sizes for the overlay scrollbars.
+    setupScrollbar(WebScrollbar::Horizontal);
+    setupScrollbar(WebScrollbar::Vertical);
+
+    // DO NOT SUBMIT: this next line will be removed before this CL lands, please ignore for review.
+    showGraphicsLayers();
+}
+
+// Modifies the top of the graphics layer tree to add layers needed to support
+// the inner/outer viewport fixed-position model for pinch zoom. When finished,
+// the tree will look like this (with * denoting added layers):
+//
+// *innerViewportClipLayer (fixed pos container)
+//  +- *pageScaleLayer
+//  |   +- *innerViewportScrollLayer
+//  |       +-- overflowControlsHostLayer (root layer)
+//  |           +-- outerViewportClipLayer (fixed pos container) [frame clip layer in RenderLayerCompositor]
+//  |           |   +-- outerViewportScrollLayer [frame scroll layer in RenderLayerCompositor]
+//  |           |       +-- content layers ...
+//  |           +-- horizontal ScrollbarLayer (non-overlay)
+//  |           +-- verticalScrollbarLayer (non-overlay)
+//  |           +-- scroll corner (non-overlay)
+//  +- *horizontalScrollbarLayer (overlay)
+//  +- *verticalScrollbarLayer (overlay)
+//
+void PinchViewports::setOverflowControlsHostLayer(GraphicsLayer* layer)
+{
+    if (layer) {
+        ASSERT(!m_innerViewportScrollLayer->children().size());
+        m_innerViewportScrollLayer->addChild(layer);
+    } else {
+        m_innerViewportScrollLayer->removeAllChildren();
+        return;
+    }
+
+    WebCore::Page* page = m_owner->page();
+    // We only need to disable the existing (outer viewport) scrollbars
+    // if the existing ones are already overlay.
+    if (!page || !page->mainFrame()->view()->hasOverlayScrollbars())
+        return;
+
+    // Disable the existing outer-viewport scrollbars, since we don't need to
+    // display two sets of overlay scrollbars.
+    // FIXME: If we knew in advance before the overflowControlsHostLayer goes
+    // away, we would re-enable the drawing of these scrollbars.
+    // FIXME: When scrollbar existence is no longer tied to scrollability, see
+    // issue crbug.com/247055, inhibit the creation of these scrollbars when they're not needed.
+    if (GraphicsLayer* scrollbar = m_owner->compositor()->layerForHorizontalScrollbar())
+        scrollbar->setDrawsContent(false);
+    if (GraphicsLayer* scrollbar = m_owner->compositor()->layerForVerticalScrollbar())
+        scrollbar->setDrawsContent(false);
+
+    // DO NOT SUBMIT: this next line will be removed before this CL lands, please ignore for review.
+    showGraphicsLayers();
+}
+
+static inline bool isHorizontal(WebScrollbar::Orientation orientation)
+{
+    return orientation == WebScrollbar::Horizontal;
+}
+
+void PinchViewports::setupScrollbar(WebScrollbar::Orientation orientation)
+{
+    OwnPtr<GraphicsLayer>& scrollbarGraphicsLayer = isHorizontal(orientation) ?

[jamesr, 2013/06/17 20:42:10]

i think having this be a reference to an OwnPtr is very confusing since there
isn't actually any change in ownership implied here. It'd be clearer to just
take a raw pointer to the appropriate scrollbar

[wjmaclean, 2013/06/17 22:09:04]

Done.

+        m_overlayScrollbarHorizontal : m_overlayScrollbarVertical;
+
+    const int kOverlayScrollbarThickness = m_owner->settingsImpl()->pinchOverlayScrollbarThickness();
+
+    ASSERT(scrollbarGraphicsLayer);
+
+    int xPosition = isHorizontal(orientation) ? 0 : m_innerViewportClipLayer->size().width() - kOverlayScrollbarThickness;
+    int yPosition = isHorizontal(orientation) ? m_innerViewportClipLayer->size().height() - kOverlayScrollbarThickness : 0;
+    int width = isHorizontal(orientation) ? m_innerViewportClipLayer->size().width() - kOverlayScrollbarThickness : kOverlayScrollbarThickness;
+    int height = isHorizontal(orientation) ? kOverlayScrollbarThickness : m_innerViewportClipLayer->size().height() - kOverlayScrollbarThickness;

[jamesr, 2013/06/17 20:42:10]

perhaps store isHorizontal(orientation) in a local bool instead of calling the
function 5 times?

[wjmaclean, 2013/06/17 22:09:04]

Done.

+
+    scrollbarGraphicsLayer->setPosition(WebCore::IntPoint(xPosition, yPosition));
+    scrollbarGraphicsLayer->setSize(WebCore::IntSize(width, height));
+}
+
+void PinchViewports::registerViewportLayersWithTreeView(WebLayerTreeView* treeView) const

[jamesr, 2013/06/17 20:42:10]

i don't think we use the variable name 'treeView' anywhere - how about
'layerTreeView' ?

[wjmaclean, 2013/06/17 22:09:04]

Done.

+{
+    if (!treeView)
+        return;
+
+    WebCore::RenderLayerCompositor* compositor = m_owner->compositor();
+    ASSERT(compositor);
+    treeView->registerPinchViewportLayers(
+        m_innerViewportClipLayer->platformLayer(),
+        m_pageScaleLayer->platformLayer(),
+        m_innerViewportScrollLayer->platformLayer(),
+        compositor->scrollLayer()->platformLayer(),
+        m_overlayScrollbarHorizontal->platformLayer(),
+        m_overlayScrollbarVertical->platformLayer());
+}
+
+void PinchViewports::clearViewportLayersForTreeView(WebLayerTreeView* treeView) const
+{
+    if (!treeView)
+        return;
+
+    treeView->registerPinchViewportLayers(0, 0, 0, 0, 0, 0);
+}
+
+void PinchViewports::notifyAnimationStarted(const GraphicsLayer*, double time)
+{
+}
+
+void PinchViewports::paintContents(const GraphicsLayer*, GraphicsContext&, GraphicsLayerPaintingPhase, const IntRect& inClip)
+{
+}
+
+// DO NOT SUBMIT: this function will be removed before this CL lands, please ignore for review.
+void PinchViewports::showGraphicsLayers()
+{
+#ifndef NDEBUG
+    if (m_innerViewportClipLayer) {
+        GraphicsLayer* root = m_innerViewportClipLayer.get();
+        while (root->parent())
+            root = root->parent();
+
+        showGraphicsLayerTree(root);
+    }
+#endif
+}
+
+} // namespace WebKit