Changeset 140519 in webkit


Ignore:
Timestamp:
Jan 23, 2013 12:34:16 AM (11 years ago)
Author:
commit-queue@webkit.org
Message:

[TexMap] Don't call GraphicsLayer::setNeedsDisplay() from TextureMapperLayer.
​https://bugs.webkit.org/show_bug.cgi?id=107629

Patch by Huang Dongsung <​luxtella@company100.net> on 2013-01-23
Reviewed by Noam Rosenthal.

We are removing the dependency on GraphicsLayer from TextureMapperLayer.

This is in preparation for refactoring TextureMapper to work in an actor
model (​http://webkit.org/b/103854).

No new tests, refactoring only.

  • platform/graphics/texmap/GraphicsLayerTextureMapper.cpp:

(WebCore::GraphicsLayerTextureMapper::setDrawsContent):

  • platform/graphics/texmap/TextureMapperLayer.cpp:

(WebCore::TextureMapperLayer::flushCompositingStateForThisLayerOnly):

Location:
trunk/Source/WebCore
Files:
3 edited

Legend:

Unmodified
Added
Removed

  • trunk/Source/WebCore/ChangeLog

    r140518 r140519  
     12013-01-23  Huang Dongsung  <luxtella@company100.net>
     2
     3        [TexMap] Don't call GraphicsLayer::setNeedsDisplay() from TextureMapperLayer.
     4        https://bugs.webkit.org/show_bug.cgi?id=107629
     5
     6        Reviewed by Noam Rosenthal.
     7
     8        We are removing the dependency on GraphicsLayer from TextureMapperLayer.
     9
     10        This is in preparation for refactoring TextureMapper to work in an actor
     11        model (http://webkit.org/b/103854).
     12
     13        No new tests, refactoring only.
     14
     15        * platform/graphics/texmap/GraphicsLayerTextureMapper.cpp:
     16        (WebCore::GraphicsLayerTextureMapper::setDrawsContent):
     17        * platform/graphics/texmap/TextureMapperLayer.cpp:
     18        (WebCore::TextureMapperLayer::flushCompositingStateForThisLayerOnly):
     19
    1202013-01-23  No'am Rosenthal  <noam@webkit.org>
    221

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

    r138291 r140519  
    281281    notifyChange(TextureMapperLayer::DrawsContentChange);
    282282    GraphicsLayer::setDrawsContent(value);
     283
     284    if (value && m_hasOwnBackingStore)
     285        setNeedsDisplay();
    283286}
    284287

  • trunk/Source/WebCore/platform/graphics/texmap/TextureMapperLayer.cpp

    r140347 r140519  
    388388    if (changeMask & ChildrenChange)
    389389        setChildren(graphicsLayer->children());
    390 
    391     if ((changeMask & DrawsContentChange) && graphicsLayer->drawsContent())
    392         graphicsLayer->setNeedsDisplay();
    393390
    394391    if (changeMask & MaskLayerChange) {
Note: See TracChangeset for help on using the changeset viewer.