Selecting/deselecting images slower in Cairo builds

RESOLVED WORKSFORME

Status

()

Core
Graphics
RESOLVED WORKSFORME
13 years ago
8 years ago

People

(Reporter: Martijn Wargers (zombie), Unassigned)

Tracking

({perf, testcase})

Trunk
mozilla1.9alpha8
x86
Windows XP
perf, testcase
Points:
---
Dependency tree / graph
Bug Flags:
blocking1.9 -
wanted1.9 +

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(3 attachments)

(Reporter)

Description

13 years ago
See upcoming testcase.
The 2006-02-23 cairo build is considerably slower as the 2006-02-22 non-cairo build in selecting/deselecting.
(Reporter)

Comment 1

13 years ago
Created attachment 212983 [details]
testcase

The testcase currently also suffers from two other cairo bugs, namely bug 324698 and bug 328386.

With the 2006-02-22 non-cairo build I get as result: 7701ms
With the 2006-02-23 cairo build I get as result: 16203ms
(Reporter)

Updated

13 years ago
Depends on: 328367
(Reporter)

Updated

13 years ago
Flags: blocking1.9a1?

Comment 2

13 years ago
I think once 324698 is fixed this will be much less of a performance hit.
Depends on: 324698

Updated

12 years ago
Depends on: 334510

Updated

12 years ago
Blocks: 334510
No longer depends on: 334510
Flags: blocking1.9a1? → blocking1.9+
Testing this with Fx2 and Fx3 shows identical time for both; can you retest?  (Yes, the blue/red issue still exists)
(Reporter)

Comment 4

12 years ago
Created attachment 246142 [details]
testcase2, using animated gif

This uses an animated gif as image.
I now work from a much faster computer, and with a faster computer, the difference can't be seen with the first testcase (but I bet it still does with a slower computer).
With that, I get 5141ms with a 2006-11-20 Firefox2 build, with a trunk build I get 13844ms. Also the animated gif looks different, I guess that's the anti-aliasing or something?
(Reporter)

Comment 5

12 years ago
Created attachment 246143 [details]
Testcase3, using a lot of animated gifs

This testcase uses a lot of animated gifs
Reults:
2006-11-20 1.8.1 branch build: 14656ms
2006-11-20 trunk build: 24062ms
Summary: Selecting/deselecting slower in Cairo builds → Selecting/deselecting images slower in Cairo builds
Target Milestone: --- → mozilla1.9beta1
Flags: blocking1.9+ → blocking1.9-
Whiteboard: [wanted-1.9]
(Reporter)

Comment 6

11 years ago
Ok, this is now worksforme.
All the testcase are now as fast or faster on current trunk build as on branch builds.
However, I want to recheck if bug 381661 gets fixed.
Status: NEW → RESOLVED
Last Resolved: 11 years ago
Depends on: 381661
Resolution: --- → WORKSFORME
If this was affected by that, we have a problem; none of the testcase images are being scaled, I don't think?
(Reporter)

Comment 8

11 years ago
The second testcase uses a scaled image, the rest not.
This is what I get as result:

2007-05-22 build: 
Testcase 1 - 4109ms
Testcase 2 - 6641ms
Testcase 3 - 20939ms

2007-05-23: 
Testcase 1 - 4109ms
Testcase 2 - 4188ms
Testcase 3 - 20808ms

Branch build with the 3rd testcase: 11808ms

Sorry, I was a bit to quick with closing the bug. The 3rd testcase is still quite a bit slower compared to branch builds.
But with the first and second testcase, I don't see a difference anymore.
Status: RESOLVED → REOPENED
Resolution: WORKSFORME → ---
Flags: wanted1.9+
Whiteboard: [wanted-1.9]
(Reporter)

Comment 9

10 years ago
Ok, testcase2 is slower again in current trunk builds, because of the smoothing of scaled images.
The other testcases are quicker in current trunk builds.
I see the same speed for testcases 1 and 2, which makes me believe that this is fixed.
Status: REOPENED → RESOLVED
Last Resolved: 11 years ago8 years ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.