Changeset 172885 in webkit

Timestamp:

Aug 23, 2014, 12:11:00 AM (11 years ago)

Author:

zandobersek@gmail.com

Message:

[TexMap] Only notify of a required flush on the first layer property change
​https://bugs.webkit.org/show_bug.cgi?id=136104

Reviewed by Martin Robinson.

The GraphicsLayer's client should only be notified that a flush is required
when the first change of one of the layer's properties is initiated, instead
of requesting the flush on every such change.

• platform/graphics/texmap/GraphicsLayerTextureMapper.cpp:

(WebCore::GraphicsLayerTextureMapper::notifyChange):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• platform/graphics/texmap/GraphicsLayerTextureMapper.cpp (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2014-08-23  Zan Dobersek  <zdobersek@igalia.com>
+
+        [TexMap] Only notify of a required flush on the first layer property change
+        https://bugs.webkit.org/show_bug.cgi?id=136104
+
+        Reviewed by Martin Robinson.
+
+        The GraphicsLayer's client should only be notified that a flush is required
+        when the first change of one of the layer's properties is initiated, instead
+        of requesting the flush on every such change.
+
+        * platform/graphics/texmap/GraphicsLayerTextureMapper.cpp:
+        (WebCore::GraphicsLayerTextureMapper::notifyChange):
+
 2014-08-22  Dan Bernstein  <mitz@apple.com>
 

trunk/Source/WebCore/platform/graphics/texmap/GraphicsLayerTextureMapper.cpp

@@ -60 +60 @@
 void GraphicsLayerTextureMapper::notifyChange(ChangeMask changeMask)
 {
+    bool flushRequired = m_changeMask == NoChanges;
     m_changeMask |= changeMask;
-    client().notifyFlushRequired(this);
+
+    if (flushRequired)
+        client().notifyFlushRequired(this);
 }