Summary: | REGRESSION(r111639): delayed animation start can be postponed | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | dstockwell | ||||
Component: | New Bugs | Assignee: | dstockwell | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | alancutter, dglazkov, dino, simon.fraser, webkit.review.bot | ||||
Priority: | P2 | ||||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Bug Depends on: | |||||||
Bug Blocks: | 107521 | ||||||
Attachments: |
|
Description
dstockwell
2013-01-31 22:20:42 PST
Created attachment 185948 [details]
Patch
Comment on attachment 185948 [details] Patch Attachment 185948 [details] did not pass cr-linux-debug-ews (chromium-xvfb): Output: http://queues.webkit.org/results/16302453 (In reply to comment #2) > (From update of attachment 185948 [details]) > Attachment 185948 [details] did not pass cr-linux-debug-ews (chromium-xvfb): > Output: http://queues.webkit.org/results/16302453 This reported failure was due to this change in the cr-linux-debug-ews bots: bug 108512 Comment on attachment 185948 [details]
Patch
cr-linux-debug failure is spurious
Comment on attachment 185948 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=185948&action=review > Source/WebCore/page/animation/AnimationController.cpp:129 > + if (m_animationTimer.isActive() && (m_animationTimer.repeatInterval() || m_animationTimer.nextFireInterval() <= timeToNextService)) > + return; The use of repeat interval in this class is peculiar. I don’t understand the comment at all. It says “to reduce the overhead of starting”. Comment on attachment 185948 [details] Patch Clearing flags on attachment: 185948 Committed r144935: <http://trac.webkit.org/changeset/144935> All reviewed patches have been landed. Closing bug. |