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

Issue 2352273002: CSS Motion Path: offset-anchor and offset-position (Closed)

Created:
4 years, 3 months ago by Eric Willigers
Modified:
4 years, 3 months ago
CC:
darktears, apavlov+blink_chromium.org, asvitkine+watch_chromium.org, blink-reviews, blink-reviews-css, blink-reviews-style_chromium.org, chromium-reviews, dglazkov+blink, rwlbuis
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

CSS Motion Path: offset-anchor and offset-position offset-position is the initial position of the path in the containing element. https://drafts.fxtf.org/motion-1/#offset-position-property The default value of offset-position is auto (the element's current position). offset-anchor is the point in the element that is positioned along the path. https://drafts.fxtf.org/motion-1/#offset-anchor-property The default value of offset-anchor is center. This patch defines the new properties, and verifies they can be parsed, stored and read. Not yet implemented: - updating the 'offset' shorthand - animation - using the new properties when computing the motion path transforms. Intent to Implement: https://groups.google.com/a/chromium.org/d/msg/blink-dev/uqDmsIj6zJY/cS-Vp_XG2oYJ BUG=638055 Committed: https://crrev.com/72e3e282b889aa86071423648033f81d8bfc7b2d Cr-Commit-Position: refs/heads/master@{#420239}

Patch Set 1 #

Patch Set 2 : webexposed #

Patch Set 3 : rebase #

Unified diffs Side-by-side diffs Delta from patch set Stats (+228 lines, -4 lines) Patch
A third_party/WebKit/LayoutTests/css3/motion-path/offset-anchor.html View 1 chunk +74 lines, -0 lines 0 comments Download
A third_party/WebKit/LayoutTests/css3/motion-path/offset-position.html View 1 chunk +74 lines, -0 lines 0 comments Download
M third_party/WebKit/LayoutTests/fast/css/getComputedStyle/computed-style-listing-expected.txt View 1 1 chunk +2 lines, -0 lines 0 comments Download
M third_party/WebKit/LayoutTests/fast/css/getComputedStyle/computed-style-without-renderer-listing-expected.txt View 1 1 chunk +2 lines, -0 lines 0 comments Download
M third_party/WebKit/LayoutTests/platform/android/webexposed/css-properties-as-js-properties-expected.txt View 1 1 chunk +2 lines, -0 lines 0 comments Download
M third_party/WebKit/LayoutTests/svg/css/getComputedStyle-listing-expected.txt View 1 1 chunk +2 lines, -0 lines 0 comments Download
M third_party/WebKit/LayoutTests/webexposed/css-properties-as-js-properties-expected.txt View 1 1 chunk +2 lines, -0 lines 0 comments Download
M third_party/WebKit/Source/core/css/CSSComputedStyleDeclaration.cpp View 1 2 1 chunk +2 lines, -0 lines 0 comments Download
M third_party/WebKit/Source/core/css/CSSProperties.in View 1 chunk +2 lines, -0 lines 0 comments Download
M third_party/WebKit/Source/core/css/ComputedStyleCSSValueMapping.cpp View 2 chunks +18 lines, -0 lines 0 comments Download
M third_party/WebKit/Source/core/css/parser/CSSPropertyParser.cpp View 2 chunks +11 lines, -0 lines 0 comments Download
M third_party/WebKit/Source/core/css/resolver/StyleBuilderConverter.h View 1 chunk +1 line, -0 lines 0 comments Download
M third_party/WebKit/Source/core/css/resolver/StyleBuilderConverter.cpp View 1 chunk +8 lines, -0 lines 0 comments Download
M third_party/WebKit/Source/core/frame/UseCounter.cpp View 2 chunks +3 lines, -1 line 0 comments Download
M third_party/WebKit/Source/core/style/ComputedStyle.h View 2 chunks +10 lines, -0 lines 0 comments Download
M third_party/WebKit/Source/core/style/StyleMotionData.h View 3 chunks +7 lines, -2 lines 0 comments Download
M third_party/WebKit/Source/core/style/StyleMotionData.cpp View 1 chunk +3 lines, -1 line 0 comments Download
M third_party/WebKit/Source/core/style/StyleTransformData.cpp View 1 chunk +2 lines, -0 lines 0 comments Download
M third_party/WebKit/Source/platform/RuntimeEnabledFeatures.in View 1 chunk +1 line, -0 lines 0 comments Download
M tools/metrics/histograms/histograms.xml View 1 2 1 chunk +2 lines, -0 lines 0 comments Download

Messages

Total messages: 17 (8 generated)
Eric Willigers
The spec for offset-anchor still needs to be updated to show the default in center. ...
4 years, 3 months ago (2016-09-21 04:04:01 UTC) #3
sashab
lgtm
4 years, 3 months ago (2016-09-21 05:13:53 UTC) #4
sashab
dstockwell@ added for owner review at eric's request. doug - this patch is pretty straightforward, ...
4 years, 3 months ago (2016-09-21 05:16:27 UTC) #6
dstockwell
lgtm
4 years, 3 months ago (2016-09-21 05:21:38 UTC) #7
Eric Willigers
+vollick for platform OWNER
4 years, 3 months ago (2016-09-21 20:43:27 UTC) #9
Ian Vollick
On 2016/09/21 20:43:27, Eric Willigers wrote: > +vollick for platform OWNER platform/ lgtm.
4 years, 3 months ago (2016-09-21 21:22:38 UTC) #10
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2352273002/40001
4 years, 3 months ago (2016-09-21 22:33:03 UTC) #13
commit-bot: I haz the power
Committed patchset #3 (id:40001)
4 years, 3 months ago (2016-09-22 02:19:32 UTC) #15
commit-bot: I haz the power
4 years, 3 months ago (2016-09-22 02:21:13 UTC) #17
Message was sent while issue was closed.
Patchset 3 (id:??) landed as
https://crrev.com/72e3e282b889aa86071423648033f81d8bfc7b2d
Cr-Commit-Position: refs/heads/master@{#420239}

Powered by Google App Engine
This is Rietveld 408576698