Title: [92331] trunk/Source/WebCore
Revision
92331
Author
[email protected]
Date
2011-08-03 16:43:15 -0700 (Wed, 03 Aug 2011)

Log Message

[chromium] Fix issue where switching composited tabs evicts root textures
https://bugs.webkit.org/show_bug.cgi?id=65562

Reviewed by James Robinson.

LayerTilerChromium::protectTileTextures was hard-coding texture sizes
and texture formats. Because this didn't always match textures' actual
sizes and formats, they weren't getting protected during eviction.

* platform/graphics/chromium/LayerTilerChromium.cpp:
(WebCore::LayerTilerChromium::protectTileTextures):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (92330 => 92331)


--- trunk/Source/WebCore/ChangeLog	2011-08-03 23:42:25 UTC (rev 92330)
+++ trunk/Source/WebCore/ChangeLog	2011-08-03 23:43:15 UTC (rev 92331)
@@ -1,3 +1,17 @@
+2011-08-02  Adrienne Walker  <[email protected]>
+
+        [chromium] Fix issue where switching composited tabs evicts root textures
+        https://bugs.webkit.org/show_bug.cgi?id=65562
+
+        Reviewed by James Robinson.
+
+        LayerTilerChromium::protectTileTextures was hard-coding texture sizes
+        and texture formats. Because this didn't always match textures' actual
+        sizes and formats, they weren't getting protected during eviction.
+
+        * platform/graphics/chromium/LayerTilerChromium.cpp:
+        (WebCore::LayerTilerChromium::protectTileTextures):
+
 2011-08-03  Ryosuke Niwa  <[email protected]>
 
         select-all, copy, paste of specialAncestorElements (e.g. pre, h1, etc) nests the element inside itself

Modified: trunk/Source/WebCore/platform/graphics/chromium/LayerTilerChromium.cpp (92330 => 92331)


--- trunk/Source/WebCore/platform/graphics/chromium/LayerTilerChromium.cpp	2011-08-03 23:42:25 UTC (rev 92330)
+++ trunk/Source/WebCore/platform/graphics/chromium/LayerTilerChromium.cpp	2011-08-03 23:43:15 UTC (rev 92331)
@@ -242,10 +242,10 @@
     for (int j = top; j <= bottom; ++j) {
         for (int i = left; i <= right; ++i) {
             Tile* tile = tileAt(i, j);
-            if (!tile || !tile->texture()->isValid(m_tileSize, GraphicsContext3D::RGBA))
+            if (!tile || !tile->texture()->isValid(m_tileSize, m_textureFormat))
                 continue;
 
-            tile->texture()->reserve(m_tileSize, GraphicsContext3D::RGBA);
+            tile->texture()->reserve(m_tileSize, m_textureFormat);
         }
     }
 }
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to