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

Issue 39353003: Web Animations CSS: Add transitions tests to virtual/web-animations-css (Closed)

Created:
7 years, 2 months ago by dstockwell
Modified:
7 years, 2 months ago
CC:
blink-reviews, shans, rjwright, alancutter (OOO until 2018), Mike Lawther (Google), Timothy Loh, darktears, Steve Block, dino_apple.com, Eric Willigers
Visibility:
Public.

Description

Web Animations CSS: Add transitions tests to virtual/web-animations-css BUG=271815 NOTRY=tried Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=160435

Patch Set 1 #

Total comments: 2

Patch Set 2 : Crashes from debug #

Messages

Total messages: 7 (0 generated)
dstockwell
7 years, 2 months ago (2013-10-24 05:23:51 UTC) #1
Steve Block
lgtm
7 years, 2 months ago (2013-10-24 05:26:14 UTC) #2
Timothy Loh
lgtm https://codereview.chromium.org/39353003/diff/1/LayoutTests/TestExpectations File LayoutTests/TestExpectations (right): https://codereview.chromium.org/39353003/diff/1/LayoutTests/TestExpectations#newcode195 LayoutTests/TestExpectations:195: crbug.com/303430 virtual/web-animations-css/transitions/shadow.html [ Crash ] I'm not too ...
7 years, 2 months ago (2013-10-24 05:29:03 UTC) #3
Steve Block
https://codereview.chromium.org/39353003/diff/1/LayoutTests/virtual/web-animations-css/transitions/svg-transitions-expected.txt File LayoutTests/virtual/web-animations-css/transitions/svg-transitions-expected.txt (right): https://codereview.chromium.org/39353003/diff/1/LayoutTests/virtual/web-animations-css/transitions/svg-transitions-expected.txt#newcode1 LayoutTests/virtual/web-animations-css/transitions/svg-transitions-expected.txt:1: PASS - "fill-opacity" property for "rect1" element at 1s ...
7 years, 2 months ago (2013-10-24 05:30:53 UTC) #4
dstockwell
On 2013/10/24 05:30:53, Steve Block wrote: > Do you need this expected result? - you ...
7 years, 2 months ago (2013-10-24 05:33:36 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/dstockwell@chromium.org/39353003/90001
7 years, 2 months ago (2013-10-24 08:42:56 UTC) #6
commit-bot: I haz the power
7 years, 2 months ago (2013-10-24 09:55:35 UTC) #7
Message was sent while issue was closed.
Change committed as 160435

Powered by Google App Engine
This is Rietveld 408576698