Modified: trunk/Source/WebCore/ChangeLog (112449 => 112450)
--- trunk/Source/WebCore/ChangeLog 2012-03-28 22:07:25 UTC (rev 112449)
+++ trunk/Source/WebCore/ChangeLog 2012-03-28 22:09:28 UTC (rev 112450)
@@ -1,3 +1,20 @@
+2012-03-28 Emil A Eklund <[email protected]>
+
+ Remove unnecessary rounding in RenderLayerBacking
+ https://bugs.webkit.org/show_bug.cgi?id=82529
+
+ Reviewed by Eric Seidel.
+
+ Remove rounding that is no longer required now that borderBoxRect returns
+ an IntRect.
+
+ No new tests.
+
+ * rendering/RenderLayerBacking.cpp:
+ (WebCore::RenderLayerBacking::updateGraphicsLayerGeometry):
+ (WebCore::RenderLayerBacking::startAnimation):
+ (WebCore::RenderLayerBacking::startTransition):
+
2012-03-28 Sheriff Bot <[email protected]>
Unreviewed, rolling out r112432.
Modified: trunk/Source/WebCore/rendering/RenderLayerBacking.cpp (112449 => 112450)
--- trunk/Source/WebCore/rendering/RenderLayerBacking.cpp 2012-03-28 22:07:25 UTC (rev 112449)
+++ trunk/Source/WebCore/rendering/RenderLayerBacking.cpp 2012-03-28 22:09:28 UTC (rev 112450)
@@ -502,7 +502,7 @@
}
if (m_owningLayer->hasTransform()) {
- const IntRect borderBox = pixelSnappedIntRect(toRenderBox(renderer())->borderBoxRect());
+ const IntRect borderBox = toRenderBox(renderer())->borderBoxRect();
// Get layout bounds in the coords of compAncestor to match relativeCompositingBounds.
IntRect layerBounds = IntRect(delta, borderBox.size());
@@ -1277,7 +1277,7 @@
bool didAnimateFilter = false;
#endif
- if (hasTransform && m_graphicsLayer->addAnimation(transformVector, pixelSnappedIntRect(toRenderBox(renderer())->borderBoxRect()).size(), anim, keyframes.animationName(), timeOffset)) {
+ if (hasTransform && m_graphicsLayer->addAnimation(transformVector, toRenderBox(renderer())->borderBoxRect().size(), anim, keyframes.animationName(), timeOffset)) {
didAnimateTransform = true;
compositor()->didStartAcceleratedAnimation(CSSPropertyWebkitTransform);
}
@@ -1342,7 +1342,7 @@
KeyframeValueList transformVector(AnimatedPropertyWebkitTransform);
transformVector.insert(new TransformAnimationValue(0, &fromStyle->transform()));
transformVector.insert(new TransformAnimationValue(1, &toStyle->transform()));
- if (m_graphicsLayer->addAnimation(transformVector, pixelSnappedIntRect(toRenderBox(renderer())->borderBoxRect()).size(), transformAnim, GraphicsLayer::animationNameForTransition(AnimatedPropertyWebkitTransform), timeOffset)) {
+ if (m_graphicsLayer->addAnimation(transformVector, toRenderBox(renderer())->borderBoxRect().size(), transformAnim, GraphicsLayer::animationNameForTransition(AnimatedPropertyWebkitTransform), timeOffset)) {
// To ensure that the correct transform is visible when the animation ends, also set the final transform.
updateLayerTransform(toStyle);
didAnimateTransform = true;
