1354624 - Fix PrintTarget::GetReferenceDrawTarget to not use the cached refDT when recording

Status: RESOLVED FIXED

(Core :: Printing: Output, enhancement)

Description

[Jonathan Watt [:jwatt]]

We call PrintTarget::GetReferenceDrawTarget early while setting up to print in order to do some measuring (or something like that) during Reflow of the print document. We cache the returned DT in mRefDT during this first GetReferenceDrawTarget call. This then causes that method to return that non-recording DT even when a DrawEventRecorder argument is passed to it later during the drawing stage.

If this bug hadn't existed then bug 1347646 would probably never have happened.

Comment 1

[Jonathan Watt [:jwatt]]

Attachment: Fix PrintTarget::GetReferenceDrawTarget and its overrides to honor aRecorder

Comment 2

[Jonathan Watt [:jwatt]]

Attachment: Remove mRefDT member from PrintTargetSkPDF

We don't need this member - it already exists in the base class.

Comment 3

[Jonathan Watt [:jwatt]]

Attachment: Make PrintTarget::CreateRecordingDrawTarget static

I noticed this when trying to remove some of the code duplication (which I'll do properly with a bigger refactoring in another bug).

Comment 4

[Jonathan Watt [:jwatt]]

Attachment: Fix PrintTarget::GetReferenceDrawTarget and its overrides to honor aRecorder

Comment 5

[Bob Owen (:bobowen)]

Comment on attachment 8857462 [details] [diff] [review]
Fix PrintTarget::GetReferenceDrawTarget and its overrides to honor aRecorder

Review of attachment 8857462 [details] [diff] [review]:
-----------------------------------------------------------------

Obviously there's a lot of duplication here, but it sound like you're going to pick that up in another bug.

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/e866a70f9eb6
Fix PrintTarget::GetReferenceDrawTarget and its overrides to honor aRecorder. r=bobowen
https://hg.mozilla.org/integration/mozilla-inbound/rev/f0d866c0600a
Remove mRefDT member from PrintTargetSkPDF. r=bobowen
https://hg.mozilla.org/integration/mozilla-inbound/rev/3a370d9f84a5
Make PrintTarget::CreateRecordingDrawTarget static. r=bobowen

Comment 7

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

https://hg.mozilla.org/mozilla-central/rev/e866a70f9eb6
https://hg.mozilla.org/mozilla-central/rev/f0d866c0600a
https://hg.mozilla.org/mozilla-central/rev/3a370d9f84a5