Fixed single-surface rendering with fixed-background elements

In single-surface rendering, we did not paint the surface collection
correctly if there was more than one child on the base surface
(which is the case with fixed-background elements).

Change-Id: I6e8d34630060414159b5f13cd302c1dc2165a048

1 files changed, 7 insertions, 5 deletions

diff --git a/Source/WebCore/platform/graphics/android/rendering/Surface.cpp b/Source/WebCore/platform/graphics/android/rendering/Surface.cpp
index 652c165..437b8ff 100644
--- a/Source/WebCore/platform/graphics/android/rendering/Surface.cpp
+++ b/Source/WebCore/platform/graphics/android/rendering/Surface.cpp
@@ -192,10 +192,10 @@ void Surface::prepareGL(bool layerTilesDisabled, bool updateWithBlit)
         ALOGV("prepareGL on Surf %p, no SurfBack, needsTexture? %d",
               this, m_surfaceBacking, needsTexture());
 
-        if (!needsTexture())
+        if (needsTexture() || (isBase() && layerTilesDisabled))
+            m_surfaceBacking = new SurfaceBacking(isBase());
+        else
             return;
-
-        m_surfaceBacking = new SurfaceBacking(isBase());
     }
 
     if (tilesDisabled) {
@@ -351,8 +351,10 @@ bool Surface::paint(SkCanvas* canvas)
         // In single surface mode, draw layer content onto the base layer
         if (isBase()
             && getFirstLayer()->countChildren()
-            && getFirstLayer()->state()->layersRenderingMode() > GLWebViewState::kClippedTextures)
-            getFirstLayer()->getChild(0)->drawCanvas(canvas, true, Layer::FlattenedLayers);
+            && getFirstLayer()->state()->layersRenderingMode() > GLWebViewState::kClippedTextures) {
+            for (unsigned int i = 0; i < getFirstLayer()->countChildren(); i++)
+                getFirstLayer()->getChild(i)->drawCanvas(canvas, true, Layer::FlattenedLayers);
+        }
     } else {
         SkAutoCanvasRestore acr(canvas, true);
         SkMatrix matrix;