Changeset 139894 in webkit


Ignore:
Timestamp:
Jan 16, 2013 10:32:46 AM (11 years ago)
Author:
reed@google.com
Message:

Use SkMatrix::I() when we need to pass identity, rather than constructing a new matrix.
https://bugs.webkit.org/show_bug.cgi?id=107016

Reviewed by James Robinson.

No new tests. existing tests exercise this code.

The default constructor is being deprecated, so this change is also part of that effort,
though it is a perf-improvement regardless, since we avoid making a new (temp) matrix.

  • platform/graphics/chromium/GraphicsLayerChromium.cpp:

(WebCore::GraphicsLayerChromium::updateLayerPreserves3D):

Location:
trunk/Source/WebCore
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • trunk/Source/WebCore/ChangeLog

    r139891 r139894  
     12013-01-16  Mike Reed  <reed@google.com>
     2
     3        Use SkMatrix::I() when we need to pass identity, rather than constructing a new matrix.
     4        https://bugs.webkit.org/show_bug.cgi?id=107016
     5
     6        Reviewed by James Robinson.
     7
     8        No new tests. existing tests exercise this code.
     9
     10        The default constructor is being deprecated, so this change is also part of that effort,
     11        though it is a perf-improvement regardless, since we avoid making a new (temp) matrix.
     12
     13        * platform/graphics/chromium/GraphicsLayerChromium.cpp:
     14        (WebCore::GraphicsLayerChromium::updateLayerPreserves3D):
     15
    1162013-01-16  Ojan Vafai  <ojan@chromium.org>
    217
  • trunk/Source/WebCore/platform/graphics/chromium/GraphicsLayerChromium.cpp

    r139829 r139894  
    736736
    737737        m_layer->layer()->setAnchorPoint(FloatPoint(0.5f, 0.5f));
    738         m_layer->layer()->setTransform(SkMatrix44());
     738        m_layer->layer()->setTransform(SkMatrix44::I());
    739739
    740740        // Set the old layer to opacity of 1. Further down we will set the opacity on the transform layer.
Note: See TracChangeset for help on using the changeset viewer.