Bug 20328 - REGRESSION: When transition property is 'all', some transitions are broken after the first time.
Summary: REGRESSION: When transition property is 'all', some transitions are broken af...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: 528+ (Nightly build)
Hardware: Mac OS X 10.5
: P1 Normal
Assignee: Simon Fraser (smfr)
URL:
Keywords: NeedsReduction, Regression
Depends on:
Blocks:
 
Reported: 2008-08-07 23:07 PDT by Simon Fraser (smfr)
Modified: 2008-08-11 15:45 PDT (History)
1 user (show)

See Also:


Attachments
Manual testcase (2.08 KB, text/html)
2008-08-07 23:07 PDT, Simon Fraser (smfr)
no flags Details
Automated testcase (1.63 KB, text/html)
2008-08-08 15:26 PDT, Simon Fraser (smfr)
no flags Details
Patch, testcase, changlog (20.74 KB, patch)
2008-08-11 13:56 PDT, Simon Fraser (smfr)
hyatt: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Simon Fraser (smfr) 2008-08-07 23:07:24 PDT
In the testcase to be attached, hovering over the list items no longer shows scale or color transitions after they have been selected once. The list items go "dead" to hover.

Adding an explicit -webkit-transition-property rule makes it work again.
Comment 1 Simon Fraser (smfr) 2008-08-07 23:07:45 PDT
Created attachment 22703 [details]
Manual testcase
Comment 2 Simon Fraser (smfr) 2008-08-07 23:23:39 PDT
Behavior works in r35532, broken in r35620, broken in r35625.
Comment 3 Simon Fraser (smfr) 2008-08-08 00:03:42 PDT
This broke in r35545 (of course).
Comment 4 Simon Fraser (smfr) 2008-08-08 15:26:48 PDT
Created attachment 22713 [details]
Automated testcase
Comment 5 Simon Fraser (smfr) 2008-08-08 17:37:19 PDT
That automated testcase doesn't catch all the issues.
Comment 6 Simon Fraser (smfr) 2008-08-11 13:56:00 PDT
Created attachment 22733 [details]
Patch, testcase, changlog
Comment 7 Dave Hyatt 2008-08-11 15:30:04 PDT
Comment on attachment 22733 [details]
Patch, testcase, changlog

r=me
Comment 8 Simon Fraser (smfr) 2008-08-11 15:44:26 PDT
Committed r35672
	M	WebCore/ChangeLog
	M	WebCore/page/AnimationController.cpp
	M	LayoutTests/ChangeLog
	A	LayoutTests/transitions/interrupted-all-transition-expected.txt
	A	LayoutTests/transitions/interrupted-all-transition.html
r35672 = e0d44797ca91baba9b32df622363f49b88fd0c98 (trunk)
Comment 9 Simon Fraser (smfr) 2008-08-11 15:44:48 PDT
Reopen to assign to me (sorry for the noise)