Open Bug 1555878 Opened 7 years ago Updated 3 years ago

Firefox performed poorly when rendering pages that contain masking image

Categories

(Core :: Graphics: ImageLib, defect, P3)

67 Branch
defect

Tracking

()

People

(Reporter: tungweic, Unassigned)

Details

(Keywords: perf, Whiteboard: [gfx-noted])

Attachments

(2 files)

Attached file index.html —

User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.14; rv:67.0) Gecko/20100101 Firefox/67.0

Steps to reproduce:

  1. Open index.html
  2. Click Load button
  3. Observe each individual page is rendered slowly

Actual results:

The pages are loaded and rendered slowly.

Expected results:

The pages should be loaded and rendered faster since the pdf file is just 1.2 MB.

Attached file imagefill.pdf —

Added the pdf file for repro.

For verifying the bug, put index.html and imagefill.pdf in the same web directory and follow the steps to reproduce.

Keywords: perf

Performance profile: https://perfht.ml/2W7NFL9

Status: UNCONFIRMED → NEW
Component: Untriaged → Graphics
Ever confirmed: true
Product: Firefox → Core

The profile looks like it is showing just a lot of time spent in codecs to decode the embedded jpegs and such. Andrew?

Flags: needinfo?(aosmond)
Priority: -- → P3
Whiteboard: [gfx-noted]

This is a flaw in our sync decoding logic. Small images may be sync decoded as it will be fast. If we request in a loop many many small images to be decoded, it will sync decode them all. It should probably have a maximum budget which we can reset on a timer, and if we exceed the budget, we always async decode. We already track the time to decode in Decoder::mDecodeTime; ideally we would even sample this time for images that did not finish decoding due to missing data from the network.

Component: Graphics → ImageLib
Flags: needinfo?(aosmond)

The component has been changed since the priority was decided, so we're resetting it.
For more information, please visit auto_nag documentation.

Priority: P3 → --
Priority: -- → P3
Severity: normal normal → S3 S3
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: