77375 2012-01-30 15:18:42 -0800 (seemingly) Unecessary repainting activity during layer animation 2012-02-15 14:47:02 -0800 1 1 1 Unclassified WebKit WebCore Misc. 528+ (Nightly build) Unspecified Unspecified RESOLVED DUPLICATE 78020 P2 Normal --- 1 nduca webkit-unassigned jchaffraix reveman simon.fraser twiz vangelis oldest_to_newest 545475 0 nduca 2012-01-30 15:18:42 -0800 See attached example. I haven't studied it in detail, but it seems that when the div is half-outside the viewport, we have to repaint. The repaints themselves are light, but the texture uploads they cause are not. 545476 1 124612 nduca 2012-01-30 15:19:14 -0800 Created attachment 124612 Test case that repaints during animation 551432 2 reveman 2012-02-07 15:03:24 -0800 *** This bug has been marked as a duplicate of bug 78020 *** 553132 3 nduca 2012-02-09 06:55:19 -0800 Based on this, I think we should add some TRACE macros to the compositor when texture eviction occurs so we can tell the difference between invalidation-driven repainting and cache-driven repainting. David, can you do a quick patch for that? 553201 4 reveman 2012-02-09 08:49:14 -0800 (In reply to comment #3) > Based on this, I think we should add some TRACE macros to the compositor when texture eviction occurs so we can tell the difference between invalidation-driven repainting and cache-driven repainting. David, can you do a quick patch for that? Sure! Should we just make eviction show up in the traces or should we try to make paint show up differently in traces as a result eviction compared to invalidation? 557575 5 nduca 2012-02-15 14:47:02 -0800 (In reply to comment #4) > Sure! Should we just make eviction show up in the traces or should we try to make paint show up differently in traces as a result eviction compared to invalidation? Lets just get the eviction to show up. Its better than nothing. :) 124612 2012-01-30 15:19:14 -0800 2012-01-30 15:19:14 -0800 Test case that repaints during animation nested_layer_animation.html text/html 1323 nduca PGh0bWw+CjxoZWFkPgogIDxzdHlsZSB0eXBlPSJ0ZXh0L2NzcyI+CiAgICAucGFuZSB7CiAgICAg IGxlZnQ6IDVweDsKICAgICAgdG9wOiA1cHg7CiAgICAgIGJhY2tncm91bmQtY29sb3I6IHJlZDsK ICAgICAgcG9zaXRpb246IGFic29sdXRlOwogICAgfQoKICAgICNjb250YWluZXIgewogICAgICBw b3NpdGlvbjogYWJzb2x1dGU7CiAgICAgIGxlZnQ6IDIwMDBweDsKICAgICAgaGVpZ2h0OiAxMDAl OwogICAgICB3aWR0aDogMTAwJTsKICAgICAgLXdlYmtpdC10cmFuc2l0aW9uOiAtd2Via2l0LXRy YW5zZm9ybSAycyBlYXNlOwogICAgICAtd2Via2l0LXRyYW5zZm9ybTogc2NhbGUoMSk7CiAgICAg IC13ZWJraXQtdHJhbnNmb3JtLW9yaWdpbi14OiAtMjAwMHB4OwogICAgfQoKICAgIGJvZHkgewog ICAgICBvdmVyZmxvdzogaGlkZGVuOwogICAgfQogIDwvc3R5bGU+CgogIDxzY3JpcHQgdHlwZT0i dGV4dC9qYXZhc2NyaXB0Ij4KICAgIGZ1bmN0aW9uICQobykge3JldHVybiBkb2N1bWVudC5nZXRF bGVtZW50QnlJZChvKTt9CgogICAgZnVuY3Rpb24gY2xpY2ttZSgpIHsKICAgICAgdmFyIGNvbnRh aW5lciA9ICQoJ2NvbnRhaW5lcicpOwogICAgICBjb250YWluZXIuc3R5bGUud2Via2l0VHJhbnNm b3JtID0gd2luZG93LnNjYWxlID8gJycgOiAnc2NhbGUoMC4xKSc7CiAgICAgIHdpbmRvdy5zY2Fs ZSA9ICF3aW5kb3cuc2NhbGU7CiAgICB9CgogICAgd2luZG93Lm9ubG9hZCA9IGZ1bmN0aW9uKCkg ewogICAgICB2YXIgY29udGFpbmVyID0gJCgnY29udGFpbmVyJyk7CiAgICAgIHZhciB3aWR0aCA9 IGNvbnRhaW5lci5vZmZzZXRXaWR0aDsKICAgICAgdmFyIGhlaWdodCA9IGNvbnRhaW5lci5vZmZz ZXRIZWlnaHQ7CiAgICAgIGZvciAodmFyIGkgPSAwOyBpIDwgMjA7IGkrKykgewogICAgICAgIHZh ciBkaXYgPSBkb2N1bWVudC5jcmVhdGVFbGVtZW50KCdkaXYnKTsKICAgICAgICBkaXYuY2xhc3NO YW1lID0gJ3BhbmUnOwogICAgICAgIHdpZHRoIC09IDEwOwogICAgICAgIGhlaWdodCAtPSAxMDsK ICAgICAgICBkaXYuc3R5bGUud2lkdGggPSB3aWR0aDsKICAgICAgICBkaXYuc3R5bGUuaGVpZ2h0 ID0gaGVpZ2h0OwogICAgICAgIGNvbnRhaW5lci5hcHBlbmRDaGlsZChkaXYpOwogICAgICAgIGNv bnRhaW5lciA9IGRpdjsKICAgICAgfQogICAgfQoKICA8L3NjcmlwdD4KPC9oZWFkPgoKPGJvZHk+ CiAgPGJ1dHRvbiBpZD0iY2xpY2ttZSIgdHlwZT0iYnV0dG9uIiBvbmNsaWNrPSJjbGlja21lKCk7 Ij5DbGljayBtZSE8L2J1dHRvbj4KICA8ZGl2IGlkPSJjb250YWluZXIiPgogIDwvZGl2Pgo8L2Jv ZHk+CjwvaHRtbD4K