Closed Bug 1678709 Opened 3 years ago Closed 3 years ago

Print preview takes a little longer to render a certain pdf page

Categories

(Toolkit :: Printing, defect)

defect

Tracking

()

VERIFIED FIXED
85 Branch
Tracking Status
firefox-esr78 --- unaffected
firefox83 --- unaffected
firefox84 --- verified
firefox85 --- verified

People

(Reporter: emilghitta, Assigned: iain)

References

(Regression)

Details

(Keywords: regression)

Attachments

(2 files)

Attached image Delay.gif

Affected versions

  • Firefox 84.0b3 (BuildId:20201119195818)
  • Firefox 85.0a1 (BuildId:20201120094511)

Affected platforms

  • Windows 10 64bit
  • macOS 10.13
  • Ubuntu 20.04 64bit ( Unfortunately I don’t have access to an Ubuntu machine at the moment but I will confirm this and update this comment Monday).

Steps to reproduce

  1. Launch Firefox.
  2. Access the following link
  3. Hit Ctrl + P or click the print button to open the print preview.

Expected result

  • The print preview renders the pdf successfully.

Actual result

  • The print preview renders the pdf successfully but takes a little longer than it used to.

Regression Range

Notes

  • For further information please observe the attached screencast.
  • [Suggested Severity] I think that S4 fits for this issue.

This might be a duplicate of bug 1677782. Taking a look.

Assignee: nobody → iireland
Has Regression Range: --- → yes
Has STR: --- → yes

Sometimes TryHoistBoundsCheck modifies an existing node instead of creating a new one. In that case, we still have to set the bailout kind to prevent bailout loops.

Pushed by iireland@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/49d07caeb2f0
Update BailoutKind for modified bounds check r=jandem
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 85 Branch

Hello,

Confirming this issue as verifed fixed . I have tried on 85.0a1(20201203094726), 84.0b7(20201201213706) and the affected build from 85.0a1(20201120163152) with macOS 10.15, Win10x64 and Ubuntu 20x64

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

Attachment

General

Created:
Updated:
Size: