LinearTimingFunction should have control points 0,0,1,1 not 0,0,0,0

[Simon Fraser (smfr)]

getComputedStyle() for transition-timing-function or animation-timing-function returns a cubic-bezier() always. This is OK, but for a linear timing function, it returns cubic-bezier(0,0,0,0). That should be cubic-bezier(0,0,1,1)

[Simon Fraser (smfr)]

Created attachment 23013 [details]
Patch, fixed tests, changelog.

[Eric Seidel (no email)]

Comment on attachment 23013 [details]
Patch, fixed tests, changelog.

Why change CSSStyleSelector if you've changed the default constructor too?

Otherwise looks fine.

[Simon Fraser (smfr)]

I changed CSSStyleSelector because I wanted to make it obvious when creating the linear timing function that the correct control points are being passed.

[Darin Adler]

Comment on attachment 23013 [details]
Patch, fixed tests, changelog.

".0" is strange syntax for a 0 of type double. We normally use "0.0".

[Simon Fraser (smfr)]

I'll fix before committing.

[Simon Fraser (smfr)]

Committed r35943
	M	WebCore/rendering/style/RenderStyle.h
	M	WebCore/ChangeLog
	M	WebCore/css/CSSStyleSelector.cpp
	M	LayoutTests/ChangeLog
	M	LayoutTests/transitions/inherit-other-props-expected.txt
	M	LayoutTests/transitions/inherit-other-props.html
r35943 = e74d671ade9a443887ec1e832811b06adcf182e8 (trunk)