Bug 23862

Summary: When interrupting accelerated animations, need to simulate style differences
Product: WebKit Reporter: Simon Fraser (smfr) <simon.fraser>
Component: Layout and RenderingAssignee: Simon Fraser (smfr) <simon.fraser>
Status: RESOLVED FIXED    
Severity: Normal CC: hyatt
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Mac   
OS: OS X 10.5   
Bug Depends on:    
Bug Blocks: 23359    
Attachments:
Description Flags
Patch, changelog
hyatt: review-
Updated patch hyatt: review+

Description Simon Fraser (smfr) 2009-02-09 18:13:37 PST
When accelerated animations are running, the softare animation timer is not firing, and we are not constantly changing the RenderStyles (because the animation is being run by the compositing system). This gives great performance, but forces the need to special-case some instances where we compare RenderStyels to know if things are changing.
Comment 1 Simon Fraser (smfr) 2009-02-11 13:23:38 PST
Created attachment 27570 [details]
Patch, changelog
Comment 2 Simon Fraser (smfr) 2009-02-11 13:25:02 PST
I'll fix the "as a child using inheriting the transform" comment before checkin.
Comment 3 Dave Hyatt 2009-02-11 13:37:08 PST
Comment on attachment 27570 [details]
Patch, changelog

Typo:

"propery" should be "property"

Why two bits?  It seems like a single flag is good enough, especially given they are both in the same struct.  We can revisit if more bits end up in different structs, but for now I'd just do one.
Comment 4 Simon Fraser (smfr) 2009-02-11 15:10:24 PST
Created attachment 27574 [details]
Updated patch
Comment 5 Dave Hyatt 2009-02-11 15:13:21 PST
Comment on attachment 27574 [details]
Updated patch

r=me
Comment 6 Simon Fraser (smfr) 2009-02-11 15:19:10 PST
Committing to http://svn.webkit.org/repository/webkit/trunk ...
	M	WebCore/ChangeLog
	M	WebCore/page/animation/ImplicitAnimation.cpp
	M	WebCore/rendering/style/RenderStyle.h
	M	WebCore/rendering/style/StyleRareNonInheritedData.cpp
	M	WebCore/rendering/style/StyleRareNonInheritedData.h
Committed r40876