Closed Bug 1490518 Opened 4 years ago Closed 4 years ago

Invalidation broken with scale transforms and SVG


(Core :: Web Painting, defect)

Not set



Tracking Status
firefox-esr60 --- unaffected
firefox62 --- unaffected
firefox63 + verified
firefox64 + verified


(Reporter: mstange, Assigned: mikokm)



(Keywords: regression)


(3 files)

Attached file testcase
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
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
Scale areas after clipping to unscaled clip r=mattwoodrow
Closed: 4 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)
Comment on attachment 9012135 [details] [diff] [review]

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.
1) Open the attach testcase
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]

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.
Flags: qe-verify+
You need to log in before you can comment on or make changes to this bug.