Closed Bug 1490518 Opened 6 years ago Closed 6 years ago

Invalidation broken with scale transforms and SVG

Categories

(Core :: Web Painting, defect)

defect
Not set
normal

Tracking

()

VERIFIED FIXED
mozilla64
Tracking Status
firefox-esr60 --- unaffected
firefox62 --- unaffected
firefox63 + verified
firefox64 + verified

People

(Reporter: mstange, Assigned: mikokm)

References

Details

(Keywords: regression)

Attachments

(3 files)

Attached file testcase (deleted) —
Steps to reproduce: 1. Load the attached testcase. 2. Click anywhere on the page. Expected results: Only the green rectangle should remain. Actual results: Parts of the blue rectangle remain as well. This was caused by bug 1462672 (verified with mozregression). This causes glitches on the aDAU dashboard which is not publicly accessible.
Flags: needinfo?(mikokm)
Assignee: nobody → mikokm
Status: NEW → ASSIGNED
Flags: needinfo?(mikokm)
Thank you for the reduced testcase!
I have applied this fix locally and confirmed that it fixes the original issue on the aDAU dashboard. Thanks!
Comment on attachment 9009594 [details] Bug 1490518 - Scale areas after clipping to unscaled clip Matt Woodrow (:mattwoodrow) has approved the revision.
Attachment #9009594 - Flags: review+
Pushed by mikokm@gmail.com: https://hg.mozilla.org/integration/autoland/rev/948e96f2b7d9 Scale areas after clipping to unscaled clip r=mattwoodrow
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla64
Is this ready for uplift to beta?
Flags: needinfo?(mikokm)
(In reply to Julien Cristau [:jcristau] from comment #7) > Is this ready for uplift to beta? The attached patch does not apply cleanly due to some whitespace changes. I will post a rebased patch after the try run finishes.
Flags: needinfo?(mikokm)
Attached patch 1490518-beta.diff (deleted) — Splinter Review
Comment on attachment 9012135 [details] [diff] [review] 1490518-beta.diff Approval Request Comment [Feature/Bug causing the regression]: SVG elements with scale transforms are not invalidating properly, when the transform is flattened. [User impact if declined]: Possible artifacts where the SVG was, if the scale transformed SVG changes. [Is this code covered by automated tests?]: Yes, there is a reftest. [Has the fix been verified in Nightly?]: Yes, I have verified the fix in Nightly. [Needs manual test from QE? If yes, steps to reproduce]: Optional. STR: 1) Open the attach testcase https://bug1490518.bmoattachments.org/attachment.cgi?id=9008250 2) clicking the blue box should completely hide it (with just one click) [List of other uplifts needed for the feature/fix]: None. [Is the change risky?]: Low risk. [Why is the change risky/not risky?]: The fix has been in the Nightly for a week and there is a reftest. The worst case is unnecessary invalidation (and painting) with improved correctness. [String changes made/needed]: None.
Attachment #9012135 - Flags: approval-mozilla-beta?
Comment on attachment 9012135 [details] [diff] [review] 1490518-beta.diff Low risk patch for a tracked regression, approved for 63 beta 10, thanks.
Attachment #9012135 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
Flags: qe-verify+
I managed to reproduce the issue using an older version of Nightly (2018-09-11) on Windows 10 x64. I retested everything using the latest Nightly 64.0a1 on Windows 10 x64, Ubuntu 16.04 x64 and macOS 10.13. The bug is not reproducing anymore. But I can still reproduce the issue on beta 63.0b9. I think I have to wait until beta 63.0b10 is up to verify the fix there.
I verified the fix on beta 63.0b10 using Windows 10 x64, Ubuntu 16.04 x64 and macOS 10.13. The bug is not reproducing anymore.
Status: RESOLVED → VERIFIED
Flags: qe-verify+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: