[Webkit-unassigned] [Bug 103510] Reduce the children repaints when moved multiple times during the layout
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Fri Nov 30 10:55:14 PST 2012
https://bugs.webkit.org/show_bug.cgi?id=103510
Tony Chang <tony at chromium.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #176952|review? |review+, commit-queue-
Flag| |
--- Comment #7 from Tony Chang <tony at chromium.org> 2012-11-30 10:57:33 PST ---
(From update of attachment 176952)
View in context: https://bugs.webkit.org/attachment.cgi?id=176952&action=review
> LayoutTests/css3/flexbox/repaint-column-reverse.html:23
> + if (window.testRunner) {
> + testRunner.waitUntilDone();
> + testRunner.display();
Please add testRunner.dumpAsText(true) so we get the pixel result, but no render tree dump which isn't important for this test.
> LayoutTests/css3/flexbox/repaint-column-reverse.html:29
> + document.body.appendChild(document.createTextNode(
> + "This test checks that for flex items that are moved multiple times during the layout "
> + + "only the initial and final positions are repainted. Only the blue flex item should be repainted "
> + + "after changing its position. If the other flex items are repainted, this test fails."));
> + }
I would always include this text. It makes it easier for gardeners.
> Source/WebCore/rendering/RenderFlexibleBox.cpp:395
> + childIndex++;
Nit: ++childIndex
> Source/WebCore/rendering/RenderFlexibleBox.cpp:735
> + iterator.first();
Ah right, previously the constructor would do this for us. Can we also remove the call to first() in the OrderIterator constructor?
--
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