Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(16)

Unified Diff: third_party/WebKit/Source/core/style/ComputedStyle.h

Issue 2891453003: Incorporate ComputedStyle::hasTransform when diffing transform styles. (Closed)
Patch Set: Created 3 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/core/style/ComputedStyle.h
diff --git a/third_party/WebKit/Source/core/style/ComputedStyle.h b/third_party/WebKit/Source/core/style/ComputedStyle.h
index 2d7f8a0870f8c69cf0cb0d9ae45af76830263322..d524f6bf3a961221c0661ac004c7f92b78770569 100644
--- a/third_party/WebKit/Source/core/style/ComputedStyle.h
+++ b/third_party/WebKit/Source/core/style/ComputedStyle.h
@@ -3318,6 +3318,11 @@ class CORE_EXPORT ComputedStyle : public ComputedStyleBase,
ETransformStyle3D usedTransformStyle3D() const {
return hasGroupingProperty() ? TransformStyle3DFlat : transformStyle3D();
}
+ // Returns whether the transform operations for |otherStyle| differ from the
+ // operations for this style instance. Note that callers may want to also
+ // check hasTransform(), as it is possible for two styles to have matching
+ // transform operations but differ in other transform-impacting style
+ // respects.
bool transformDataEquivalent(const ComputedStyle& otherStyle) const {
return m_rareNonInheritedData->m_transform ==
otherStyle.m_rareNonInheritedData->m_transform;
@@ -3728,6 +3733,10 @@ class CORE_EXPORT ComputedStyle : public ComputedStyleBase,
StyleInheritedVariables& mutableInheritedVariables();
StyleNonInheritedVariables& mutableNonInheritedVariables();
+
+ FRIEND_TEST_ALL_PREFIXES(
+ ComputedStyleTest,
+ UpdatePropertySpecificDifferencesRespectsTransformAnimation);
};
// FIXME: Reduce/remove the dependency on zoom adjusted int values.
« no previous file with comments | « third_party/WebKit/Source/core/paint/PaintLayerTest.cpp ('k') | third_party/WebKit/Source/core/style/ComputedStyle.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698