1316750 - mozPrintCallback stopped producing vector output again for non-remote printing

Status: RESOLVED FIXED

(Core :: Printing: Output, defect)

Description

[Haik Aftandilian [:haik]]

Attachment: Test case

The problem fixed by bug 1308259 "mozPrintCallback stoped producing vector output" is reproducible again.

mozregression led me to bug 1280324 "Provide a way to get a reference DrawTarget from PrintTargets, and fix various nsDeviceContext::CreateRenderingContext consumers to get a reference context".

  $ mozregression -g 2016-10-17 --pref "print.print_via_parent:false" "security.sandbox.content.level:0"

There's a known issue with mozPrintCallback on Nightly with Mac remote printing (1310165), but this issue affects non-remote printing which is used in Aurora/Beta/Release right now and that is what was fixed by 1308259. The above prefs need to be set to test non-remote printing on Nightly.

I've only tested on Mac so far.

To use the test case, print to PDF, and then check whether or not the added "Test" text under "data: loading print" and "api loading print" is selectable or a single bitmap. It should be selectable. By selectable, I mean that individual characters can be highlighted, not just the whole word as an image.

Comment 1

[Haik Aftandilian [:haik]]

This reproduces for me on Windows 10 Nightly on a fresh profile with no prefs set.

Comment 2

[Jonathan Watt [:jwatt]]

When we're under the stack:

  PrintTarget::GetReferenceDrawTarget
  nsDeviceContext::CreateRenderingContextCommon
  nsDeviceContext::CreateReferenceRenderingContext
  nsSimplePageSequenceFrame::PrePrintNextPage
  nsPrintEngine::PrePrintPage

we call cairo_surface_create_similar. That doesn't do the correct thing for CAIRO_SURFACE_TYPE_QUARTZ and CAIRO_SURFACE_TYPE_WIN32 surfaces. We need to special case these surface types to make sure that the new "similar" surface that we create is also a vector backed surface.

Comment 3

[Jonathan Watt [:jwatt]]

Attachment: patch

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/9508c116ecd7
Add special case handling for CAIRO_SURFACE_TYPE_WIN32 and CAIRO_SURFACE_TYPE_QUARTZ in PrintTarget::GetReferenceDrawTarget. r=edwin

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/732a96ceaf9f
Back out the CAIRO_SURFACE_TYPE_WIN32 code to fix bustage. r=me CLOSED TREE

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/fb46a1dc34f9
Fix CAIRO_SURFACE_TYPE_WIN32 warnings as errors case. r=me CLOSED TREE

Comment 7

[Jonathan Watt [:jwatt]]

Attachment: part 2 - Add CairoContentToCairoFormat helper to HelpersCairo.h

Comment 8

[Jonathan Watt [:jwatt]]

Attachment: part 3 - Add special case code to handle CAIRO_SURFACE_TYPE_WIN32 in PrintTarget::GetReferenceDrawTarget

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/e222819961af
part 2 - Add CairoContentToCairoFormat helper to HelpersCairo.h. r=edwin
https://hg.mozilla.org/integration/mozilla-inbound/rev/c0d0caa11f5a
part 3 - Add special case code to handle CAIRO_SURFACE_TYPE_WIN32 in PrintTarget::GetReferenceDrawTarget. r=edwin

Comment 10

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

I had to back this out (along with your other bugs pushed today) for windows vm debug failures like https://treeherder.mozilla.org/logviewer.html#?job_id=39173435&repo=mozilla-inbound

https://hg.mozilla.org/integration/mozilla-inbound/rev/91b1add28ef64ca507a1c1c14db2031ae37c992b

Comment 11

[Jonathan Watt [:jwatt]]

The same intermittent failure showed up on your backout commit too:

https://treeherder.mozilla.org/#/jobs?repo=mozilla-inbound&revision=91b1add28ef64ca507a1c1c14db2031ae37c992b

It seems to have mysteriously disappeared some time after that though. I've triggered some more rebuilds, but based on current evidence I'd be inclined to think the failure isn't related to the changes I landed and reland these patches.

Comment 12

[Jonathan Watt [:jwatt]]

Yeah, it really looks like my patches had nothing to do with that failure. I'll reland part 3 separately from parts 1 and 2 though.

Comment 13

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/c313bf4cfde9
part 1 - Add special case handling for CAIRO_SURFACE_TYPE_QUARTZ to PrintTarget::GetReferenceDrawTarget. r=edwin
https://hg.mozilla.org/integration/mozilla-inbound/rev/8fedd88f6488
part 2 - Add CairoContentToCairoFormat helper to HelpersCairo.h. r=edwin

Comment 14

[Jonathan Watt [:jwatt]]

(In reply to Jonathan Watt [:jwatt] from comment #12)
> Yeah, it really looks like my patches had nothing to do with that failure.
> I'll reland part 3 separately from parts 1 and 2 though.

Here's another example of a job where the failure shows up long after the backouts:

https://treeherder.mozilla.org/#/jobs?repo=mozilla-inbound&revision=be0a409b178acc10fc89b0d528c78a03755d89e4

And another where there is an OOM crash (OOM being a likely cause of the failure to create the DrawTarget with Size(800,1000)):

https://treeherder.mozilla.org/#/jobs?repo=mozilla-inbound&revision=55ceefe2f5aa83f8bf08ba43be938ad92c2bee22

Comment 15

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/ba22cd08d123
part 3 - Add special case code to handle CAIRO_SURFACE_TYPE_WIN32 in PrintTarget::GetReferenceDrawTarget. r=edwin

Comment 16

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/c313bf4cfde9
https://hg.mozilla.org/mozilla-central/rev/8fedd88f6488
https://hg.mozilla.org/mozilla-central/rev/ba22cd08d123