Closed Bug 1650351 Opened 3 years ago Closed 3 months ago

Land a reftest for bug 1552923

Categories

(Core :: Graphics: WebRender, task, P3)

task

Tracking

()

RESOLVED FIXED
106 Branch
Tracking Status
firefox106 --- fixed

People

(Reporter: hiro, Assigned: hiro)

References

(Blocks 1 open bug)

Details

Attachments

(2 files)

I wrote a reftest bug 1552923, but apparently it needs bug 1646629 (a change to make reftest-no-flush work properly), so I am going to defer adding the reftest into this bug.

Hiro, is this still relevant?

Flags: needinfo?(hikezoe.birchill)

Yes, coincidently Boris is now touching some relevant stuff in bug 1776077.

I will try to see if this reftest now works or not since bug 1646629 landed.

Assignee: nobody → hikezoe.birchill
Status: NEW → ASSIGNED
Flags: needinfo?(hikezoe.birchill)

Backed out for causing reftest failures on replace-with-new-positive-delay-animation.html

Flags: needinfo?(hikezoe.birchill)

Hmm, it failed on Android. And I suppose just waiting for two frames isn't sufficient for Android. :/

I am going to skip the reftest on Android. On Android even with waiting for 1s the test fails, the green box doesn't appear, it should be triggered by setting the second animation. When I open the reftest in question with some tweak (using window.onload instead of MozReftestInvalidate) on GeckoView example, the green box appears properly. So I suppose it's an issue in our reftest harness (Android specific, it's weird though) rather than a real issue.

Flags: needinfo?(hikezoe.birchill)
Blocks: 1787682

Filed bug 1787682 for the Android failure.

Status: ASSIGNED → RESOLVED
Closed: 3 months ago
Resolution: --- → FIXED
Target Milestone: --- → 106 Branch
You need to log in before you can comment on or make changes to this bug.