Chromium Code Reviews| Index: Source/core/css/resolver/AnimatedStyleBuilder.cpp |
| diff --git a/Source/core/css/resolver/AnimatedStyleBuilder.cpp b/Source/core/css/resolver/AnimatedStyleBuilder.cpp |
| index 50f2f24cc60ba6c4bc0fa906c0fac3289b3be256..b2b1ad340fc484b499f0c10389d76e92c270ad0f 100644 |
| --- a/Source/core/css/resolver/AnimatedStyleBuilder.cpp |
| +++ b/Source/core/css/resolver/AnimatedStyleBuilder.cpp |
| @@ -345,8 +345,7 @@ void AnimatedStyleBuilder::applyProperty(CSSPropertyID property, StyleResolverSt |
| style->setVisitedLinkColor(toAnimatableColor(value)->visitedLinkColor()); |
| return; |
| case CSSPropertyFillOpacity: |
| - // Avoiding a value of 1 forces a layer to be created. |
| - style->setFillOpacity(clampTo<float>(toAnimatableDouble(value)->toDouble(), 0, nextafterf(1, 0))); |
| + style->setFillOpacity(clampTo<float>(toAnimatableDouble(value)->toDouble(), 0, 1)); |
| return; |
| case CSSPropertyFill: |
| { |
| @@ -581,8 +580,7 @@ void AnimatedStyleBuilder::applyProperty(CSSPropertyID property, StyleResolverSt |
| return; |
| case CSSPropertyWebkitTransform: { |
| const TransformOperations& operations = toAnimatableTransform(value)->transformOperations(); |
| - // FIXME: Using identity matrix here when the transform list is empty |
| - // forces a layer to be created in the presence of a transform animation. |
| + // FIXME: This normalization (handling of 'none' should be performed at input). |
|
Julien - ping for review
2014/03/17 22:26:16
Is the ending parenthesis badly placed?
Also not
dstockwell
2014/03/18 00:13:56
I'll rephrase this and fix the parenthesis. By 'in
|
| style->setTransform(operations.size() ? operations : TransformOperations(true)); |
| return; |
| } |