Set root clip layer not to maskToBounds.

Source/web/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
@@ skipping 35 matching lines @@
 
 namespace WebKit {
 
 PassOwnPtr<PinchViewports> PinchViewports::create(WebViewImpl* owner)
 {
     return adoptPtr(new PinchViewports(owner));
 }
 
 PinchViewports::PinchViewports(WebViewImpl* owner)
     : m_owner(owner)
-    , m_innerViewportClipLayer(GraphicsLayer::create(m_owner->graphicsLayerFactory(), this))
+    , m_innerViewportContainerLayer(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_innerViewportContainerLayer->platformLayer()->setIsContainerForFixedPositionLayers(true);
+    // No need for the inner viewport to clip, since the compositing
+    // surface takes care of it -- and clipping here would interfere with
+    // dynamically-sized viewports on Android.
+    m_innerViewportContainerLayer->setMasksToBounds(false);
 
     m_innerViewportScrollLayer->platformLayer()->setScrollable(true);
 
 #ifndef NDEBUG
-    m_innerViewportClipLayer->setName("inner viewport clip layer");
+    m_innerViewportContainerLayer->setName("inner viewport container 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_innerViewportContainerLayer->addChild(m_pageScaleLayer.get());
     m_pageScaleLayer->addChild(m_innerViewportScrollLayer.get());
-    m_innerViewportClipLayer->addChild(m_overlayScrollbarHorizontal.get());
-    m_innerViewportClipLayer->addChild(m_overlayScrollbarVertical.get());
+    m_innerViewportContainerLayer->addChild(m_overlayScrollbarHorizontal.get());
+    m_innerViewportContainerLayer->addChild(m_overlayScrollbarVertical.get());
 
     // Setup the inner viewport overlay scrollbars.
     setupScrollbar(WebScrollbar::Horizontal);
     setupScrollbar(WebScrollbar::Vertical);
 }
 
 PinchViewports::~PinchViewports() { }
 
 void PinchViewports::setViewportSize(const WebCore::IntSize& newSize)
 {
-    m_innerViewportClipLayer->setSize(newSize);
+    m_innerViewportContainerLayer->setSize(newSize);
 
     // Need to re-compute sizes for the overlay scrollbars.
     setupScrollbar(WebScrollbar::Horizontal);
     setupScrollbar(WebScrollbar::Vertical);
 }
 
 // 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)
+// *innerViewportContainerLayer (fixed pos container)
 //  +- *pageScaleLayer
 //  |   +- *innerViewportScrollLayer
 //  |       +-- overflowControlsHostLayer (root layer)
-//  |           +-- outerViewportClipLayer (fixed pos container) [frame clip layer in RenderLayerCompositor]
+//  |           +-- outerViewportContainerLayer (fixed pos container) [frame container 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)
 {
@@ skipping 20 matching lines @@
 }
 
 void PinchViewports::setupScrollbar(WebScrollbar::Orientation orientation)
 {
     bool isHorizontal = orientation == WebScrollbar::Horizontal;
     GraphicsLayer* scrollbarGraphicsLayer = isHorizontal ?
         m_overlayScrollbarHorizontal.get() : m_overlayScrollbarVertical.get();
 
     const int overlayScrollbarThickness = m_owner->settingsImpl()->pinchOverlayScrollbarThickness();
 
-    int xPosition = isHorizontal ? 0 : m_innerViewportClipLayer->size().width() - overlayScrollbarThickness;
-    int yPosition = isHorizontal ? m_innerViewportClipLayer->size().height() - overlayScrollbarThickness : 0;
-    int width = isHorizontal ? m_innerViewportClipLayer->size().width() - overlayScrollbarThickness : overlayScrollbarThickness;
-    int height = isHorizontal ? overlayScrollbarThickness : m_innerViewportClipLayer->size().height() - overlayScrollbarThickness;
+    int xPosition = isHorizontal ? 0 : m_innerViewportContainerLayer->size().width() - overlayScrollbarThickness;
+    int yPosition = isHorizontal ? m_innerViewportContainerLayer->size().height() - overlayScrollbarThickness : 0;
+    int width = isHorizontal ? m_innerViewportContainerLayer->size().width() - overlayScrollbarThickness : overlayScrollbarThickness;
+    int height = isHorizontal ? overlayScrollbarThickness : m_innerViewportContainerLayer->size().height() - overlayScrollbarThickness;
 
     scrollbarGraphicsLayer->setPosition(WebCore::IntPoint(xPosition, yPosition));
     scrollbarGraphicsLayer->setSize(WebCore::IntSize(width, height));
 }
 
 void PinchViewports::registerViewportLayersWithTreeView(WebLayerTreeView* layerTreeView) const
 {
     if (!layerTreeView)
         return;
 
     WebCore::RenderLayerCompositor* compositor = m_owner->compositor();
     ASSERT(compositor);
     layerTreeView->registerPinchViewportLayers(
-        m_innerViewportClipLayer->platformLayer(),
+        m_innerViewportContainerLayer->platformLayer(),
         m_pageScaleLayer->platformLayer(),
         m_innerViewportScrollLayer->platformLayer(),
         compositor->scrollLayer()->platformLayer(),
         m_overlayScrollbarHorizontal->platformLayer(),
         m_overlayScrollbarVertical->platformLayer());
 }
 
 void PinchViewports::clearViewportLayersForTreeView(WebLayerTreeView* layerTreeView) const
 {
     if (!layerTreeView)
         return;
 
     layerTreeView->clearPinchViewportLayers();
 }
 
 void PinchViewports::notifyAnimationStarted(const GraphicsLayer*, double time)
 {
 }
 
 void PinchViewports::paintContents(const GraphicsLayer*, WebCore::GraphicsContext&, WebCore::GraphicsLayerPaintingPhase, const WebCore::IntRect& inClip)
 {
 }
 
 } // namespace WebKit