Title: [193828] trunk/Source/WebCore
Revision
193828
Author
zandober...@gmail.com
Date
2015-12-09 06:03:34 -0800 (Wed, 09 Dec 2015)

Log Message

[TexMap] TextureMapperTiledBackingStore should notify the ImageObserver of the data access
https://bugs.webkit.org/show_bug.cgi?id=152053

Reviewed by Martin Robinson.

TextureMapperTiledBackingStore should call ImageObserver::didDraw() on the Image's
observer after updating the tile with the Image's data. This way the CachedImage
(i.e. the observer) can mark the data access with the current timestamp, avoiding
removing the decoded data in the very near future during a cache purge.

* platform/graphics/texmap/TextureMapperTiledBackingStore.cpp:
(WebCore::TextureMapperTiledBackingStore::updateContentsFromImageIfNeeded):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (193827 => 193828)


--- trunk/Source/WebCore/ChangeLog	2015-12-09 14:00:03 UTC (rev 193827)
+++ trunk/Source/WebCore/ChangeLog	2015-12-09 14:03:34 UTC (rev 193828)
@@ -1,5 +1,20 @@
 2015-12-09  Zan Dobersek  <zdober...@igalia.com>
 
+        [TexMap] TextureMapperTiledBackingStore should notify the ImageObserver of the data access
+        https://bugs.webkit.org/show_bug.cgi?id=152053
+
+        Reviewed by Martin Robinson.
+
+        TextureMapperTiledBackingStore should call ImageObserver::didDraw() on the Image's
+        observer after updating the tile with the Image's data. This way the CachedImage
+        (i.e. the observer) can mark the data access with the current timestamp, avoiding
+        removing the decoded data in the very near future during a cache purge.
+
+        * platform/graphics/texmap/TextureMapperTiledBackingStore.cpp:
+        (WebCore::TextureMapperTiledBackingStore::updateContentsFromImageIfNeeded):
+
+2015-12-09  Zan Dobersek  <zdober...@igalia.com>
+
         Make MainThreadSharedTimerGtk implementation GLib-specific
         https://bugs.webkit.org/show_bug.cgi?id=152044
 

Modified: trunk/Source/WebCore/platform/graphics/texmap/TextureMapperTiledBackingStore.cpp (193827 => 193828)


--- trunk/Source/WebCore/platform/graphics/texmap/TextureMapperTiledBackingStore.cpp	2015-12-09 14:00:03 UTC (rev 193827)
+++ trunk/Source/WebCore/platform/graphics/texmap/TextureMapperTiledBackingStore.cpp	2015-12-09 14:03:34 UTC (rev 193828)
@@ -23,6 +23,7 @@
 #include "TextureMapperTiledBackingStore.h"
 
 #include "ImageBuffer.h"
+#include "ImageObserver.h"
 #include "TextureMapper.h"
 
 namespace WebCore {
@@ -35,6 +36,9 @@
         return;
 
     updateContents(textureMapper, m_image.get(), m_image->size(), enclosingIntRect(m_image->rect()), BitmapTexture::UpdateCannotModifyOriginalImageData);
+
+    if (m_image->imageObserver())
+        m_image->imageObserver()->didDraw(m_image.get());
     m_image = nullptr;
 }
 
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to