ImageLoader::RegisterCSSImage doesn't need to be called on loading document when printing

RESOLVED FIXED in Firefox 64

Status

()

enhancement
P3
normal
RESOLVED FIXED
8 months ago
8 months ago

People

(Reporter: heycam, Assigned: heycam)

Tracking

unspecified
mozilla64
Points:
---

Firefox Tracking Flags

(firefox64 fixed)

Details

Attachments

(1 attachment)

Assignee

Description

8 months ago
For a print document, we load any CSS images using the original document's ImageLoader.  We also register the CSS image with that ImageLoader but since that only has the effect of finding those image when a test is asked to flush use counters for them, I don't think we need to do that.
Assignee

Comment 1

8 months ago
I tried removing the loading document registration and printing a document that had some images that were only used in a `@media print` rule and it seemed to work fine.

Comment 3

8 months ago
Pushed by cmccormack@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/3b7ba9bbad14
Don't register CSS images in the loading document when printing r=emilio

Comment 4

8 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/3b7ba9bbad14
Status: ASSIGNED → RESOLVED
Last Resolved: 8 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla64
You need to log in before you can comment on or make changes to this bug.