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

Issue 212483003: CSS Transforms: Implement transform-origin (Closed)

Created:
6 years, 9 months ago by dstockwell
Modified:
6 years, 8 months ago
Reviewers:
eseidel
CC:
blink-reviews, ed+blinkwatch_opera.com, shans, rjwright, alancutter (OOO until 2018), Mike Lawther (Google), rwlbuis, rune+blink, dglazkov+blink, dstockwell, Timothy Loh, apavlov+blink_chromium.org, darktears, Steve Block, dino_apple.com, Eric Willigers, eseidel, esprehn, abarth-chromium, ojan, Julien - ping for review
Visibility:
Public.

Description

CSS Transforms: Implement transform-origin Implements transform-origin as defined in CSS Transforms[1]. Behavior is similar to -webkit-transform-origin, however transform-origin is not a shorthand. New parsing and serialization (mostly matching Gecko) has been introduced. The animation system is modified to avoid running animations and transitions on both the prefixed and unprefixed variants at the same time. Internally, animations are mapped to the unprefixed variant. As animations operate directly on the RenderStyle storage the result of specifying an animation on either -webkit-transform-origin or transform-origin is the same. However, support for independent animation/transition of -webkit-transform-origin-* is removed. The X, Y and Z coordinates now always transition together. Note, this property and changes in behavior are guarded by the CSSTransformsUnprefixed runtime feature (enabled only for testing). [1] http://www.w3.org/TR/css3-transforms/ BUG=154772 Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=170537

Patch Set 1 #

Patch Set 2 : Rebase. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+350 lines, -75 lines) Patch
M LayoutTests/animations/interpolation/transform-origin-interpolation.html View 1 chunk +9 lines, -9 lines 0 comments Download
M LayoutTests/animations/interpolation/transform-origin-interpolation-expected.txt View 1 chunk +12 lines, -12 lines 0 comments Download
A + LayoutTests/animations/interpolation/webkit-transform-origin-interpolation.html View 1 chunk +8 lines, -8 lines 0 comments Download
A LayoutTests/animations/interpolation/webkit-transform-origin-interpolation-expected.txt View 1 chunk +16 lines, -0 lines 0 comments Download
A LayoutTests/transforms/transform-origin-parsing.html View 1 chunk +53 lines, -0 lines 0 comments Download
A + LayoutTests/transitions/unprefixed-transform-origin.html View 2 chunks +9 lines, -4 lines 0 comments Download
A LayoutTests/transitions/unprefixed-transform-origin-expected.txt View 1 chunk +19 lines, -0 lines 0 comments Download
A + Source/core/animation/AnimatableLengthPoint3D.h View 2 chunks +18 lines, -15 lines 0 comments Download
A + Source/core/animation/AnimatableLengthPoint3D.cpp View 2 chunks +17 lines, -14 lines 0 comments Download
M Source/core/animation/AnimatableValue.h View 2 chunks +2 lines, -0 lines 0 comments Download
M Source/core/animation/css/CSSAnimatableValueFactory.cpp View 2 chunks +10 lines, -0 lines 0 comments Download
M Source/core/animation/css/CSSAnimations.cpp View 1 3 chunks +12 lines, -3 lines 0 comments Download
M Source/core/animation/css/CSSPropertyEquality.cpp View 1 chunk +2 lines, -0 lines 0 comments Download
M Source/core/core.gypi View 1 1 chunk +2 lines, -0 lines 0 comments Download
M Source/core/css/CSSComputedStyleDeclaration.cpp View 1 4 chunks +3 lines, -1 line 0 comments Download
M Source/core/css/CSSProperties.in View 1 1 chunk +1 line, -0 lines 0 comments Download
M Source/core/css/StylePropertySerializer.cpp View 1 chunk +1 line, -0 lines 0 comments Download
M Source/core/css/parser/CSSPropertyParser.h View 1 2 chunks +3 lines, -2 lines 0 comments Download
M Source/core/css/parser/CSSPropertyParser.cpp View 1 6 chunks +80 lines, -6 lines 0 comments Download
M Source/core/css/resolver/AnimatedStyleBuilder.cpp View 2 chunks +12 lines, -0 lines 0 comments Download
M Source/core/css/resolver/StyleBuilderCustom.cpp View 1 3 chunks +61 lines, -1 line 0 comments Download

Messages

Total messages: 12 (0 generated)
dstockwell
Depends on: https://codereview.chromium.org/212543003/
6 years, 9 months ago (2014-03-26 10:39:11 UTC) #1
eseidel
lgtm
6 years, 9 months ago (2014-03-27 23:18:47 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/dstockwell@chromium.org/212483003/1
6 years, 9 months ago (2014-03-27 23:18:59 UTC) #3
commit-bot: I haz the power
The CQ bit was unchecked by commit-bot@chromium.org
6 years, 9 months ago (2014-03-27 23:19:12 UTC) #4
commit-bot: I haz the power
Failed to apply patch for LayoutTests/transitions/unprefixed-transform-origin.html: While running svn copy LayoutTests/transitions/unprefixed-perspective.html LayoutTests/transitions/unprefixed-transform-origin.html --config-dir /b/commit-queue/subversion_config --non-interactive ...
6 years, 9 months ago (2014-03-27 23:19:12 UTC) #5
dstockwell
The CQ bit was checked by dstockwell@chromium.org
6 years, 8 months ago (2014-04-01 08:16:39 UTC) #6
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/dstockwell@chromium.org/212483003/80001
6 years, 8 months ago (2014-04-01 08:16:41 UTC) #7
commit-bot: I haz the power
The CQ bit was unchecked by commit-bot@chromium.org
6 years, 8 months ago (2014-04-01 08:32:04 UTC) #8
commit-bot: I haz the power
Try jobs failed on following builders: tryserver.blink on mac_blink_rel
6 years, 8 months ago (2014-04-01 08:32:04 UTC) #9
dstockwell
The CQ bit was checked by dstockwell@chromium.org
6 years, 8 months ago (2014-04-01 08:34:24 UTC) #10
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/dstockwell@chromium.org/212483003/80001
6 years, 8 months ago (2014-04-01 08:34:33 UTC) #11
commit-bot: I haz the power
6 years, 8 months ago (2014-04-01 09:49:44 UTC) #12
Message was sent while issue was closed.
Change committed as 170537

Powered by Google App Engine
This is Rietveld 408576698