nsFrameImageLoader::IsSameImageRequest is 5% of paint time for modern skin

VERIFIED FIXED

Status

()

Core
Layout
VERIFIED FIXED
17 years ago
17 years ago

People

(Reporter: Mike Pinkerton (not reading bugmail), Assigned: Stuart Parmenter)

Tracking

({perf})

Trunk
PowerPC
Mac System 9.x
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

17 years ago
profiling the painting that happens on a window update event on mac,
nsFrameImageLoader::IsSameImageRequest() is 5% of the total paint time.

thought i'd filed this already, but guess not.
(Reporter)

Updated

17 years ago
Blocks: 48274
(Reporter)

Updated

17 years ago
Keywords: nsmac1, perf

Comment 1

17 years ago
You'll find that the parent of this call is nsPresContext::StartLoadImage()
(bug 64438).
Depends on: 64438

Comment 2

17 years ago
Reassigning to Attinasi.
Assignee: karnaze → attinasi

Comment 3

17 years ago
I think that once the dependency is fixed, this won't be a problem. The code for
IsSameImageRequest is not that bad.
Status: NEW → ASSIGNED
(Assignee)

Comment 4

17 years ago
i should probably take this since nsFrameImageLoader is going away.

Comment 5

17 years ago
Stuart, feel free to take it, but please point me to more info on the
frameImageloader's impending demise... Thanks.

Updated

17 years ago
Assignee: attinasi → pavlov
Status: ASSIGNED → NEW

Comment 6

17 years ago
Over to pavlov, as he suggested (Comments From Stuart Parmenter 2001-02-20 11:40)
(Assignee)

Comment 7

17 years ago
nsFrameImageLoader isn't used anymore.
Status: NEW → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → FIXED

Comment 8

17 years ago
Marking verified per last comments. 
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.