1694189 - Talos test pdfpaint Error: Expected page 1 got 2

Status: RESOLVED FIXED

(Firefox :: PDF Viewer, defect)

Description

[Brendan Dahl [:bdahl]]

As mentioned in https://bugzilla.mozilla.org/show_bug.cgi?id=1220362#c288

Comment 1

[Brendan Dahl [:bdahl]]

Attachment: Bug 1694189 - Don't wait for timeout to add pagerendered event listener. r=emilio

It appears we were missing some pagerendered events because we weren't
attaching the listener until after a setTimeout in contentLoadHandlerCallback.
Further, the test wasn't really waiting until idle to load the next round, so we
can remove that part for now. This may be something we want to fix in the
future though.

Comment 2

[Brendan Dahl [:bdahl]]

Unfortunately, I haven't been able to reproduce this locally, but I have an idea on how it could happen (see patch).

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/2c2cb0eb3e5f
Don't wait for timeout to add pagerendered event listener. r=emilio,perftest-reviewers,sparky

Comment 4

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/2c2cb0eb3e5f