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

Issue 2942183002: [CSS Typed OM] Make CSSRotation attributes mutable. (Closed)

Created:
3 years, 6 months ago by meade_UTC10
Modified:
3 years, 5 months ago
Reviewers:
Bugs Nash
CC:
darktears, apavlov+blink_chromium.org, blink-reviews, blink-reviews-css, chromium-reviews, dglazkov+blink, rwlbuis
Target Ref:
refs/heads/master
Project:
chromium
Visibility:
Public.

Description

[CSS Typed OM] Make CSSRotation attributes mutable. Spec: https://drafts.css-houdini.org/css-typed-om/#cssrotation This CL also removes the "is2d_" member of CSSRotation to make way for a future refactor for updated handling of "2D-ness" of CSSTransformComponents (WIP here: https://codereview.chromium.org/2943303002/ For now, some is2D tests will fail, but it's not worth fixing them in light of the future refactor. BUG=545318 Review-Url: https://codereview.chromium.org/2942183002 Cr-Commit-Position: refs/heads/master@{#482933} Committed: https://chromium.googlesource.com/chromium/src/+/5655311243673c93bc6f402206a3feff0d1d78b5

Patch Set 1 #

Patch Set 2 : Update idl file and tests #

Patch Set 3 : rebase #

Patch Set 4 : rebase #

Patch Set 5 : remove const #

Patch Set 6 : rebase #

Patch Set 7 : Add global interface expectations #

Patch Set 8 : rebase #

Messages

Total messages: 39 (24 generated)
meade_UTC10
3 years, 6 months ago (2017-06-16 07:08:07 UTC) #4
Bugs Nash
lgtm, but I am not familiar with cssom code please add to the description: - ...
3 years, 6 months ago (2017-06-18 23:12:56 UTC) #7
meade_UTC10
On 2017/06/18 23:12:56, Bugs Nash wrote: > lgtm, but I am not familiar with cssom ...
3 years, 6 months ago (2017-06-19 08:04:01 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/2942183002/80001
3 years, 6 months ago (2017-06-22 06:47:55 UTC) #13
commit-bot: I haz the power
Try jobs failed on following builders: win_clang on master.tryserver.chromium.win (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.win/builders/win_clang/builds/255485)
3 years, 6 months ago (2017-06-22 07:06:45 UTC) #15
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/2942183002/70006
3 years, 6 months ago (2017-06-23 03:19:28 UTC) #19
commit-bot: I haz the power
Try jobs failed on following builders: android_arm64_dbg_recipe on master.tryserver.chromium.android (JOB_FAILED, https://build.chromium.org/p/tryserver.chromium.android/builders/android_arm64_dbg_recipe/builds/295002)
3 years, 6 months ago (2017-06-23 03:35:19 UTC) #21
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/2942183002/70006
3 years, 6 months ago (2017-06-23 03:43:16 UTC) #23
commit-bot: I haz the power
Try jobs failed on following builders: android_arm64_dbg_recipe on master.tryserver.chromium.android (JOB_FAILED, https://build.chromium.org/p/tryserver.chromium.android/builders/android_arm64_dbg_recipe/builds/295025)
3 years, 6 months ago (2017-06-23 03:56:47 UTC) #25
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/2942183002/70006
3 years, 6 months ago (2017-06-23 09:08:13 UTC) #27
commit-bot: I haz the power
Try jobs failed on following builders: android_arm64_dbg_recipe on master.tryserver.chromium.android (JOB_FAILED, https://build.chromium.org/p/tryserver.chromium.android/builders/android_arm64_dbg_recipe/builds/295286)
3 years, 6 months ago (2017-06-23 09:24:44 UTC) #29
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/2942183002/70006
3 years, 6 months ago (2017-06-23 09:26:57 UTC) #31
commit-bot: I haz the power
Try jobs failed on following builders: android_arm64_dbg_recipe on master.tryserver.chromium.android (JOB_FAILED, https://build.chromium.org/p/tryserver.chromium.android/builders/android_arm64_dbg_recipe/builds/295313)
3 years, 6 months ago (2017-06-23 09:44:05 UTC) #33
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/2942183002/150001
3 years, 5 months ago (2017-06-28 07:53:26 UTC) #36
commit-bot: I haz the power
3 years, 5 months ago (2017-06-28 10:09:25 UTC) #39
Message was sent while issue was closed.
Committed patchset #8 (id:150001) as
https://chromium.googlesource.com/chromium/src/+/5655311243673c93bc6f402206a3...

Powered by Google App Engine
This is Rietveld 408576698