Merge "update callback mechanism, adds nativeDiscardAllTextures for TileBenchmark tool"

4 files changed, 27 insertions, 16 deletions

diff --git a/Source/WebCore/platform/graphics/android/BaseTileTexture.h b/Source/WebCore/platform/graphics/android/BaseTileTexture.h
index cd8e78b..fdd2b0e 100644
--- a/Source/WebCore/platform/graphics/android/BaseTileTexture.h
+++ b/Source/WebCore/platform/graphics/android/BaseTileTexture.h
@@ -83,9 +83,6 @@ public:
     bool acquire(TextureOwner* owner, bool force = false);
     bool release(TextureOwner* owner);
 
-    // removes Tile->Texture, and Texture->Tile links to fully discard the texture
-    void releaseAndRemoveFromTile();
-
     // set the texture owner if not busy. Return false if busy, true otherwise.
     bool setOwner(TextureOwner* owner, bool force = false);
 
diff --git a/Source/WebCore/platform/graphics/android/TilesManager.cpp b/Source/WebCore/platform/graphics/android/TilesManager.cpp
index 41ebfef..42a9a57 100644
--- a/Source/WebCore/platform/graphics/android/TilesManager.cpp
+++ b/Source/WebCore/platform/graphics/android/TilesManager.cpp
@@ -155,7 +155,7 @@ void TilesManager::allocateTiles()
          m_tilesTextures.size() * LAYER_TILE_WIDTH * LAYER_TILE_HEIGHT * 4 / 1024 / 1024);
 }
 
-void TilesManager::deallocateTextures(bool allTextures)
+void TilesManager::discardTextures(bool allTextures, bool glTextures)
 {
     const unsigned int max = m_textures.size();
 
@@ -169,24 +169,32 @@ void TilesManager::deallocateTextures(bool allTextures)
                 sparedDrawCount = std::max(sparedDrawCount, owner->drawCount());
         }
     }
-    deallocateTexturesVector(sparedDrawCount, m_textures);
-    deallocateTexturesVector(sparedDrawCount, m_tilesTextures);
+    discardTexturesVector(sparedDrawCount, m_textures, glTextures);
+    discardTexturesVector(sparedDrawCount, m_tilesTextures, glTextures);
 }
 
-void TilesManager::deallocateTexturesVector(unsigned long long sparedDrawCount,
-                                            WTF::Vector<BaseTileTexture*>& textures)
+void TilesManager::discardTexturesVector(unsigned long long sparedDrawCount,
+                                         WTF::Vector<BaseTileTexture*>& textures,
+                                         bool deallocateGLTextures)
 {
     const unsigned int max = textures.size();
     int dealloc = 0;
     for (unsigned int i = 0; i < max; i++) {
         TextureOwner* owner = textures[i]->owner();
         if (!owner || owner->drawCount() < sparedDrawCount) {
-            textures[i]->discardGLTexture();
+            if (deallocateGLTextures) {
+                // deallocate textures' gl memory
+                textures[i]->discardGLTexture();
+            } else if (owner) {
+                // simply detach textures from owner
+                static_cast<BaseTile*>(owner)->discardTextures();
+            }
             dealloc++;
         }
     }
-    XLOG("Deallocated %d gl textures (out of %d %s tiles)",
-         dealloc, max, (textures == m_textures) ? "base" : "layer");
+    XLOG("Discarded %d %s textures (out of %d %s tiles)",
+         dealloc, (deallocateGLTextures ? "gl" : ""),
+         max, (textures == m_textures) ? "base" : "layer");
 }
 
 void TilesManager::gatherTexturesNumbers(int* nbTextures, int* nbAllocatedTextures,
@@ -390,7 +398,8 @@ void TilesManager::setMaxLayerTextureCount(int max)
         double secondsSinceLayersUsed = WTF::currentTime() - m_lastTimeLayersUsed;
         if (secondsSinceLayersUsed > LAYER_TEXTURES_DESTROY_TIMEOUT) {
             unsigned long long sparedDrawCount = ~0; // by default, spare no textures
-            deallocateTexturesVector(sparedDrawCount, m_tilesTextures);
+            bool deleteGLTextures = true;
+            discardTexturesVector(sparedDrawCount, m_tilesTextures, deleteGLTextures);
             m_hasLayerTextures = false;
         }
         return;
diff --git a/Source/WebCore/platform/graphics/android/TilesManager.h b/Source/WebCore/platform/graphics/android/TilesManager.h
index 9782fbb..424e226 100644
--- a/Source/WebCore/platform/graphics/android/TilesManager.h
+++ b/Source/WebCore/platform/graphics/android/TilesManager.h
@@ -119,8 +119,8 @@ public:
 
     void allocateTiles();
 
-    // Called when webview is hidden to discard graphics memory
-    void deallocateTextures(bool allTextures);
+    // remove all tiles from textures (and optionally deallocate gl memory)
+    void discardTextures(bool allTextures, bool glTextures);
 
     bool getShowVisualIndicator()
     {
@@ -209,8 +209,9 @@ private:
             m_generatorReadyCond.wait(m_generatorLock);
     }
 
-    void deallocateTexturesVector(unsigned long long sparedDrawCount,
-                                  WTF::Vector<BaseTileTexture*>& textures);
+    void discardTexturesVector(unsigned long long sparedDrawCount,
+                               WTF::Vector<BaseTileTexture*>& textures,
+                               bool deallocateGLTextures);
 
     Vector<BaseTileTexture*> m_textures;
     Vector<BaseTileTexture*> m_availableTextures;
diff --git a/Source/WebCore/platform/graphics/android/TreeManager.cpp b/Source/WebCore/platform/graphics/android/TreeManager.cpp
index b7eaacf..35ddeb8 100644
--- a/Source/WebCore/platform/graphics/android/TreeManager.cpp
+++ b/Source/WebCore/platform/graphics/android/TreeManager.cpp
@@ -245,6 +245,10 @@ bool TreeManager::drawGL(double currentTime, IntRect& viewRect,
     if (m_drawingTree) {
         bool drawingReady = didTreeSwap || m_drawingTree->isReady();
 
+        // call the page swap callback if registration happened without more trees enqueued
+        if (treesSwappedPtr && drawingReady && !m_paintingTree)
+            *treesSwappedPtr = true;
+
         if (didTreeSwap || m_fastSwapMode || (drawingReady && !m_paintingTree))
             m_drawingTree->swapTiles();