Open Bug 1737263 Opened 3 years ago Updated 2 years ago

Pages containing SVG images are unsuccessfully printed

Categories

(Core :: Graphics, defect)

defect

Tracking

()

REOPENED
Tracking Status
firefox-esr78 --- unaffected
firefox-esr91 --- wontfix
firefox93 --- wontfix
firefox94 --- wontfix
firefox95 --- wontfix
firefox96 --- wontfix
firefox97 --- wontfix

People

(Reporter: gmoldovan, Assigned: jrmuizel, NeedInfo)

References

(Regression)

Details

(Keywords: regression)

Attachments

(4 files)

Attached file RC.cgi.pdf

Affected versions

  • Latest Nightly 95.0a1
  • Beta 94.0b9
  • RC 93.0

Affected platforms

  • Windows 10x64
  • Ubuntu 20.04
  • macOS 10.15

Steps to reproduce

  1. Launch Firefox.

  2. Access the following link: https://bug1715803.bmoattachments.org/attachment.cgi?id=9226355.

  3. Print preview the page.

  4. Save the page as PDF.

  5. Compare it with print preview.

Expected result

  • The Save to PDF page looks like in the print preview.

Actual result

  • The Save to PDF page adds unwanted background to the image.

Suggested Severity

  • S3 since it only affects certain extensions

Regression range

  • This is not a recent regression since is reproduces on 93.0 Release. I will look for an old regression range as soon as possible.

Additional notes

  • I attached a Print to file PDF with the problem.
  • On macOS 10.15 the issue is more visible. The image is completely ruined.
Has Regression Range: --- → no
Has STR: --- → yes
Component: Printing: Output → Graphics

Hey Jeff, a couple regressions from Matt's work. You reviewed it, so wondering if you might have some ideas.

Flags: needinfo?(jmuizelaar)

Yeah, I'll take a look

Assignee: nobody → jmuizelaar
Flags: needinfo?(jmuizelaar)

Bug 1540737 originally used PushLayerWithBlend here but
PushLayerWithBlend is not implemented for D2D. Since then,
I believe we've stopped using D2D for content drawing so
it should be possible to switch back to PushLayerWithBlend.

Pushed by jmuizelaar@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/4053cb4ba9ad
Switch back to PushLayerWithBlend. r=mstange
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 96 Branch
Attached image SVGimage.png

Verified as fixed on Ubuntu 20.04 and Windows 10x64 on the latest Nightly 96.0a1 (20211105093421).

On macOS it looks a lot better, but the image is blurry. I attached a screenshot with the issue.
@Jeff, do you know if this is expected or should I fill a new bug for this issue?

Flags: needinfo?(jmuizelaar)

(In reply to Giorgia Nichita, Release Desktop QA from comment #7)

Created attachment 9249429 [details]
SVGimage.png

Verified as fixed on Ubuntu 20.04 and Windows 10x64 on the latest Nightly 96.0a1 (20211105093421).

On macOS it looks a lot better, but the image is blurry. I attached a screenshot with the issue.
@Jeff, do you know if this is expected or should I fill a new bug for this issue?

Giorgia can you check what the behaviour was on macOS before bug 1540737 landed?

Flags: needinfo?(jmuizelaar) → needinfo?(giorgia.moldovan)
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Target Milestone: 96 Branch → ---

Backed out for causing Windows graphics crashes (bug 1739661)

Backout link: https://hg.mozilla.org/integration/autoland/rev/2d244932aec645578e28d2572eaef0a4fb7b8ff4

Flags: needinfo?(jmuizelaar)

(In reply to Jeff Muizelaar [:jrmuizel] from comment #8)

Giorgia can you check what the behaviour was on macOS before bug 1540737 landed?

Before Bug 1540737 landed, the image was completely ruined, I attached a screenshot of the behavior.

Flags: needinfo?(giorgia.moldovan)

There's a r+ patch which didn't land and no activity in this bug for 2 weeks.
:jrmuizel, could you have a look please?
For more information, please visit auto_nag documentation.

Flags: needinfo?(mstange.moz)
Flags: needinfo?(jmuizelaar)
Flags: needinfo?(mstange.moz)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: