Open Bug 1555878 Opened 5 years ago Updated 2 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: