[Webkit-unassigned] [Bug 85253] Apply animations and transitions for first-letter element
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Thu May 31 14:45:59 PDT 2012
https://bugs.webkit.org/show_bug.cgi?id=85253
Simon Fraser (smfr) <simon.fraser at apple.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #144960|review?, commit-queue? |review-, commit-queue-
Flag| |
--- Comment #13 from Simon Fraser (smfr) <simon.fraser at apple.com> 2012-05-31 14:45:59 PST ---
(From update of attachment 144960)
View in context: https://bugs.webkit.org/attachment.cgi?id=144960&action=review
r- pending answers to my questions.
> LayoutTests/animations/first-letter-animation.html:11
> + -webkit-animation: boxAnimation 0.5s 0.2s linear;
0.5s is still pretty long for one test. Imagine 100 of these; you've eaten 50s of every developer's time who runs tests.
> LayoutTests/animations/first-letter-animation.html:28
> + }
Style rules would suggest this paren be aligned with the 'from'.
> LayoutTests/animations/first-letter-play-state.html:5
> +<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN" "http://www.w3.org/TR/html4/loose.dtd">
> +<html lang="en">
> +<head>
> + <meta http-equiv="Content-Type" content="text/html; charset=utf-8">
> + <title>Test of -webkit-animation-play-state</title>
Could be simplified at match the other test.
> Source/WebCore/ChangeLog:8
> + Instead of call RenderOject::node() in the animations code,
"of calling"
> Source/WebCore/rendering/RenderBlock.cpp:6020
> - RenderStyle* pseudoStyle = styleForFirstLetter(firstLetterBlock, firstLetterContainer);
> + RefPtr<RenderStyle> pseudoStyle = animation()->updateAnimations(firstLetter, styleForFirstLetter(firstLetterBlock, firstLetterContainer));
Why do we have to call updateAnimations() explicitly in this case, but in createFirstLetterRenderer() we can rely on setAnimatableStyle() to call updateAnimations() for us?
> Source/WebCore/rendering/RenderObject.cpp:2227
> +Node* RenderObject::generatingNode() const
> +{
> + Node* node = m_node == document() ? 0 : m_node;
> + if (node)
> + return node;
> +
> + for (RenderObject* object = parent(); object; object = object->parent()) {
> + if (Node* node = (object->node() == object->document() ? 0 : object->node()))
> + return node;
> + }
> + return 0;
> +}
Does this behavior change affect other calls to generatingNode(), like the counter code?
--
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
More information about the webkit-unassigned
mailing list