[CompositeAfterPaint] transition animation in iframe doesn't start |
|||||||||
Issue descriptiontransitions/transition-end-event-destroy-iframe.html With SPv2 enabled, this test's iframe html adds a transitionEnd listener on box1, then sets a translate transform on box1, and the listener is never called. Test's purpose per its opening text is: Tests element removal and hiding in webkitTransitionEnd event handler. Should not crash.
,
Mar 16 2017
virtual/threaded/transitions/extra-transition.html looks like the same issue.
,
Mar 28 2017
,
Dec 7 2017
Still in SPv2 expectations file.
,
Dec 7
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue. Sorry for the inconvenience if the bug really should have been left as Available. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Dec 7
,
Dec 7
Tested in content_shell, and saw that the animation in iframe didn't start. The animation will start if I open the iframe's inner document directly in content_shell.
,
Dec 7
,
Dec 7
The animation just doesn't start with CAP:
<iframe srcdoc="
<div id='target' style='width: 100px; height: 100px; background: blue; transition: transform 2s'></div>
<script>
onload = function() { target.style.transform = 'translate(100px, 0)'; };
</script>"
</iframe>
,
Dec 11
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/f5af1abbbe19b919a32abe86ba0827ffb4250ff1 commit f5af1abbbe19b919a32abe86ba0827ffb4250ff1 Author: Xianzhu Wang <wangxianzhu@chromium.org> Date: Tue Dec 11 16:44:00 2018 [CompositeAfterPaint] Ensure animation timeline is attached to host We need to attach the document animation timeline to the animation host to make animations work. In pre-CompositeAfterPaint, this is done from PaintLayerCompositor. For CompositeAfterPaint, attach animation timeline from Document::Initialize() and WebLocalFrameImpl::SetFrameWidget(). Bug: 702353 Change-Id: I5e19add5196e5b52ebcade42b58e336051b111e6 Reviewed-on: https://chromium-review.googlesource.com/c/1368994 Reviewed-by: Philip Rogers <pdr@chromium.org> Commit-Queue: Xianzhu Wang <wangxianzhu@chromium.org> Cr-Commit-Position: refs/heads/master@{#615554} [modify] https://crrev.com/f5af1abbbe19b919a32abe86ba0827ffb4250ff1/third_party/blink/renderer/core/dom/document.cc [modify] https://crrev.com/f5af1abbbe19b919a32abe86ba0827ffb4250ff1/third_party/blink/renderer/core/dom/document.h [modify] https://crrev.com/f5af1abbbe19b919a32abe86ba0827ffb4250ff1/third_party/blink/renderer/core/frame/web_local_frame_impl.cc [modify] https://crrev.com/f5af1abbbe19b919a32abe86ba0827ffb4250ff1/third_party/blink/renderer/core/page/chrome_client_impl.cc [modify] https://crrev.com/f5af1abbbe19b919a32abe86ba0827ffb4250ff1/third_party/blink/web_tests/FlagExpectations/enable-blink-features=CompositeAfterPaint
,
Dec 11
|
|||||||||
►
Sign in to add a comment |
|||||||||
Comment 1 by schenney@chromium.org
, Mar 16 2017