Multiple image rendering lead to browser hang

NEW
Unassigned

Status

()

--
major
4 years ago
3 years ago

People

(Reporter: mylith+mozilla, Unassigned)

Tracking

({hang, testcase})

35 Branch
x86_64
Windows 7
hang, testcase
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [gfx-noted])

Attachments

(1 attachment)

(Reporter)

Description

4 years ago
Created attachment 8559041 [details]
testcase.html

User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:35.0) Gecko/20100101 Firefox/35.0
Build ID: 20150122214805

Steps to reproduce:

I wrote small test case script which generates table in 3 different ways:

- cell with text
- cell with div element and background image
- cell with img

FX 35.0.1, Mozilla/5.0 (Windows NT 6.1; WOW64; rv:35.0) Gecko/20100101 Firefox/35.0

table with img: Generation time: 32263 ms (browser hang)
table bg class: Generation time: 565 ms
table textNode: Generation time: 391 ms

Chrome 40.0.2214.93 m, Mozilla/5.0 (Windows NT 6.1; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/40.0.2214.93 Safari/537.36

table with img: Generation time: 2650 ms
table bg class: Generation time: 311 ms
table textNode: Generation time: 201 ms

IE 11, Mozilla/5.0 (Windows NT 6.1; WOW64; Trident/7.0; rv:11.0) like Gecko

table with img: Generation time: ? ms (impossible to render, browser hang)
table bg class: Generation time: 14709 ms
table textNode: Generation time: 11129 ms


Actual results:

Rendering images (same one) lead to browser hang. 


Expected results:

Rendering images should be faster if possible, at least should not hang browser.
(Reporter)

Updated

4 years ago
Keywords: testcase
(Reporter)

Updated

4 years ago
Keywords: hang
(Reporter)

Updated

4 years ago
Summary: Repeatedly image rendering lead to browser hang → Multiple image rendering lead to browser hang
Severity: normal → major
Component: Untriaged → ImageLib
Product: Firefox → Core
Note that I hang with an adblocker (ublock) but the test completes in 10 seconds without an adblocker.

http://people.mozilla.org/~bgirard/cleopatra/#report=c4d03d25c6ac218e2009a740d311ad440ef2736c&invertCallback=true&filter=[{"type"%3A"RangeSampleFilter","start"%3A96486,"end"%3A104143}]&selection="%28total%29",967

We spend a lot of time in 'mozilla::image::ProgressTracker::AddObserver(mozilla::image::IProgressObserver*)' and a lot of time reflowing.

:seth do you expect ProgressTracker::AddObserver to be so costly in this test case?
Flags: needinfo?(seth)

Updated

4 years ago
Whiteboard: [gfx-noted]
Depends on: 1194912
This should be resolved by bug 1194912.
Flags: needinfo?(seth)
You need to log in before you can comment on or make changes to this bug.