Summary: | Content disappears on CSS parallax example | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Simon Fraser (smfr) <simon.fraser> | ||||||||
Component: | New Bugs | Assignee: | Simon Fraser (smfr) <simon.fraser> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | darin, simon.fraser, webkit-bug-importer | ||||||||
Priority: | P2 | Keywords: | InRadar | ||||||||
Version: | WebKit Nightly Build | ||||||||||
Hardware: | Unspecified | ||||||||||
OS: | Unspecified | ||||||||||
URL: | https://googlechromelabs.github.io/ui-element-samples/parallax/ | ||||||||||
Attachments: |
|
Description
Simon Fraser (smfr)
2020-04-19 22:09:58 PDT
Created attachment 396942 [details]
Patch
Created attachment 396943 [details]
Patch
That new patch is a two-for-one! Oh whoops. I wish webkit-patch supported --git-index like prepare-Changelogs does. Created attachment 396945 [details]
Patch
Found 2 new test failures: compositing/tiling/coverage-adjustment-secondary-quad-mapping.html, fast/layoutformattingcontext/simple-absolute-positioned-replaced-inline-element-with-percentage-height.html Comment on attachment 396942 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=396942&action=review > Source/WebCore/platform/graphics/transforms/TransformState.cpp:223 > + if (direction == ApplyTransformDirection) { > quad = m_accumulatedTransform->mapQuad(quad); > + return; > + } > > quad = m_accumulatedTransform->inverse().valueOr(TransformationMatrix()).projectQuad(quad, wasClamped); I should have suggested else instead of return. I used return to mirror the mapPoint() code above. (In reply to Simon Fraser (smfr) from comment #10) > I used return to mirror the mapPoint() code above. I suggested else to mirror translateTransform above. We just happened to look at different functions. |