Summary: | [ Mac ] REGRESSION (r237587) Layout Test compositing/visible-rect/animated-from-none.html is flaky text diff failure | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Shawn Roberts <sroberts> | ||||
Component: | Tools / Tests | Assignee: | Antoine Quint <graouts> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | dino, graouts, lforschler, simon.fraser, thorton, webkit-bot-watchers-bugzilla, webkit-bug-importer, wenson_hsieh | ||||
Priority: | P2 | Keywords: | InRadar | ||||
Version: | WebKit Nightly Build | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Attachments: |
|
Description
Shawn Roberts
2019-02-18 15:59:14 PST
(In reply to Shawn Roberts from comment #0) > The following layout test is flaky on Mac > > compositing/visible-rect/animated-from-none.html > > Probable cause: > > Test started to become flaky after R237924. Prior revision passes. I haven't investigated this to any depth yet, but it seems somewhat unlikely that <https://trac.webkit.org/r237924> would cause a layout test on macOS to become flaky. Is it possible that another change (which might've landed around the same time as r237924) caused this? It is definitely possible. Partnering with another bot watcher they thought perhaps the Rect it was referring to was it attempting to draw a rectangle and it was off. 237925 has a similar reference but I don't think anything had been changed in that regard. The dumping code is: if (m_transform && !m_transform->isIdentity()) { ts << indent << "(transform "; ts << "[" << m_transform->m11() << " " << m_transform->m12() << " " << m_transform->m13() << " " << m_transform->m14() << "] "; ts << "[" << m_transform->m21() << " " << m_transform->m22() << " " << m_transform->m23() << " " << m_transform->m24() << "] "; ts << "[" << m_transform->m31() << " " << m_transform->m32() << " " << m_transform->m33() << " " << m_transform->m34() << "] "; ts << "[" << m_transform->m41() << " " << m_transform->m42() << " " << m_transform->m43() << " " << m_transform->m44() << "])\n"; } So this transform must be slightly non-identity. Figure out why! I think the reason is that the animation's progress can be a very small number larger than 0. I don't know whether that is a bug or a shortcoming of the test. I think for the Web Animations engine the right thing to use is the "ready" promise and not an "animationstart" event where the event delivery is async and progress may not be 0 at the time it's delivered. Created attachment 367047 [details]
Patch
Comment on attachment 367047 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=367047&action=review > LayoutTests/ChangeLog:11 > + than 1 and yield an animated transform that isn't quite the identity matrix. Don't you mean 0? Committed r244082: <https://trac.webkit.org/changeset/244082> |