Closed Bug 1565235 Opened 5 years ago Closed 4 years ago

Remove flushApzRepaints hack from inline options browser

Categories

(Toolkit :: Add-ons Manager, task, P2)

task

Tracking

()

VERIFIED FIXED
mozilla74
Tracking Status
firefox70 --- wontfix
firefox74 --- verified
firefox75 --- verified

People

(Reporter: mstriemer, Assigned: mstriemer)

References

Details

Attachments

(1 file)

We needed to call windowUtils.flushApzRepaints() to fix an issue with the inline browser. That has since been fixed in bug 1548687 and the workaround [1] can now be removed.

[1] https://searchfox.org/mozilla-central/search?q=Bug+1548687&path=

Priority: P3 → P2
Assignee: nobody → mstriemer
Status: NEW → ASSIGNED
Pushed by mstriemer@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/67ed349b3786
Remove flushApzRepaints() hack from about:addons inline options r=rpl
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla74

Hello,
Are there any particular scenarios that require verification after the removal of these hacks? Please provide some more context and steps in order for QA to manually verify if so. If that is not necessary please employ the "qe- verify" tag.
Thank you.

Flags: needinfo?(lgreco)

Hi Miruna,
you should be able to verify that the behavior is now still fixed without the workaround by using a subset of the steps (the ones from 3 to 7) from the STR Mark provided in Bug 1548687 comment 0.

Flags: needinfo?(lgreco)

Great. Thank you.
Verified on FF Nightly 74.0a1 (20200204214324) and 75.0a1 (20200212093201) on Windows Pro 10 64-bit and MacOS Catalina 10.15.
Installed Tree Style Tab extension and from it's Options/Preferences Context menu section I've checked/ unchecked the context menu items
and additional check boxes. Clicking on a checkbox highlights it until clicking outside it and adds a checkmark in the selected box.
Also tested to see if modifications remained saved after existing/entering the extension Options menu or after browser restart.

Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.