Last Comment Bug 361377 - Performance regression with this testcase, which continuously reloads a page
: Performance regression with this testcase, which continuously reloads a page
: perf, regression, testcase
Product: Core
Classification: Components
Component: Graphics (show other bugs)
: Trunk
: x86 Windows XP
P4 normal with 1 vote (vote)
: ---
Assigned To: Vladimir Vukicevic [:vlad] [:vladv]
: Milan Sreckovic [:milan]
Depends on:
Blocks: 334719
  Show dependency treegraph
Reported: 2006-11-21 03:01 PST by Martijn Wargers [:mwargers]
Modified: 2010-11-02 23:48 PDT (History)
6 users (show)
dsicore: blocking1.9.1-
vladimir: wanted1.9.1+
vladimir: blocking1.9-
reed: wanted1.9+
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

testcase (2.01 KB, text/html)
2006-11-21 03:04 PST, Martijn Wargers [:mwargers]
no flags Details

Description User image Martijn Wargers [:mwargers] 2006-11-21 03:01:09 PST
I was hoping that I could find a possible regression in performance for bug 360789, but I didn't see that.
But I see with the testcase that I made a performance regression when cairo was turned on on windows.
So that's why I'm filing this bug. It might not be the most useful testcase, but it's probably useful to keep track of it, to ensure that the performance will turn back into pre-cairo days.
Comment 1 User image Martijn Wargers [:mwargers] 2006-11-21 03:04:48 PST
Created attachment 246144 [details]

You need to download the testcase locally to get a good working testcase.
In the statusbar, I get the following results:
2006-02-22 build: 17391ms 
2006-02-25 build: 23735ms 
That is the period when cairo was turned on in windows.

2006-11-20 build: 21000ms
2006-11-20 branch build: 19484ms
Not as noticeable a difference, but still. (It's better to test with a non-cairo trunk build).
Comment 2 User image Martijn Wargers [:mwargers] 2008-06-11 14:52:17 PDT
Current trunk gives 28577ms
A branch build gives 24510ms
So there is still a performance regression visible here.
Note that trunk doesn't show the iframe big anymore (because of some CSS rule), so that should be a reason to be even quicker than branch, perhaps.
Comment 3 User image Damon Sicore (:damons) 2008-06-23 16:45:36 PDT
I don't think this would block 1.9.1.  Please re-nom if you disagree.
Comment 4 User image Vladimir Vukicevic [:vlad] [:vladv] 2008-07-22 13:53:28 PDT
There's a border involved here, so that's almost certinly it; could get faster with an upcoming patch to borders.
Comment 5 User image Vladimir Vukicevic [:vlad] [:vladv] 2008-07-22 13:57:32 PDT
Yeah; Fx 3: 24322 with border patch: 23020.  Will land that when the tree reopens.
Comment 6 User image Mike Shaver (:shaver -- probably not reading bugmail closely) 2010-11-02 23:48:41 PDT
I think vlad landed it, though borders have continued to plague is well into the year 2010!

Note You need to log in before you can comment on or make changes to this bug.