CSS Independent Transform Properties

Source/core/style/StyleTransformData.cpp

 /*
  * Copyright (C) 1999 Antti Koivisto (koivisto@kde.org)
  * Copyright (C) 2004, 2005, 2006, 2007, 2008 Apple Inc. All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
  * version 2 of the License, or (at your option) any later version.
  *
  * This library is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
  * Library General Public License for more details.
  *
  * You should have received a copy of the GNU Library General Public License
  * along with this library; see the file COPYING.LIB.  If not, write to
  * the Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor,
  * Boston, MA 02110-1301, USA.
  *
  */
 
 #include "config.h"
 #include "core/style/StyleTransformData.h"
-
 #include "core/style/ComputedStyle.h"
+#include "core/style/DataEquivalency.h"
 
 namespace blink {
 
 StyleTransformData::StyleTransformData()
     : m_operations(ComputedStyle::initialTransform())
     , m_origin(ComputedStyle::initialTransformOrigin())
     , m_motion(nullptr, ComputedStyle::initialMotionOffset(), ComputedStyle::initialMotionRotation(), ComputedStyle::initialMotionRotationType())
 {
 }
 
 StyleTransformData::StyleTransformData(const StyleTransformData& o)
     : RefCounted<StyleTransformData>()
     , m_operations(o.m_operations)
     , m_origin(o.m_origin)
     , m_motion(o.m_motion)
+    , m_translate(o.m_translate)
+    , m_rotate(o.m_rotate)
+    , m_scale(o.m_scale)
 {
 }
 
 bool StyleTransformData::operator==(const StyleTransformData& o) const
 {
-    return m_origin == o.m_origin && m_operations == o.m_operations && m_motion == o.m_motion;
+    return m_origin == o.m_origin
+    && m_operations == o.m_operations
+    && m_motion == o.m_motion
+    && dataEquivalent(m_translate, o.m_translate)
+    && dataEquivalent(m_rotate, o.m_rotate)
+    && dataEquivalent(m_scale, o.m_scale);
+}
+
+bool StyleTransformData::has3DTransformProperties() const
+{
+    return (m_translate && m_translate->z() != 0)

[dstockwell, 2015/06/12 00:17:58]

Why doesn't this care about m_operations?

[soonm, 2015/06/12 01:54:13]

I wanted to separate the 3D checks for 'transform' and the individual ones
initially, but I think I can merge them.

CompositingReasonFinder.cpp:146 now uses this function instead of checking both
separatedly

+        || (m_rotate && (m_rotate->x() != 0 || m_rotate->y() != 0 || m_rotate->z() != 1))
+        || (m_scale && m_scale->z() != 1);
+}
+
+void StyleTransformData::setTranslate(PassRefPtr<TranslateTransformOperation> translate)
+{
+    m_translate = translate;
+}
+
+void StyleTransformData::setRotate(PassRefPtr<RotateTransformOperation> rotate)
+{
+    m_rotate = rotate;
+}
+
+void StyleTransformData::setScale(PassRefPtr<ScaleTransformOperation> scale)
+{
+    m_scale = scale;
 }
 
 } // namespace blink