Closed Bug 61469 Opened 19 years ago Closed 12 years ago
collect total reflows that occur during system test
Per kmcclusk's suggestion, collect rods's ``reflow statistics'' and report daily, or even better, display in a slot on tinderbox. Here's info from rods on what needs to be done to make this work: First, you need to [build with] MOZ_REFLOW_PERF=1 Then there are two small fixes: Here are the two diffs (that should be checked in ASAP): Index: nsPresShell.cpp =================================================================== RCS file: /cvsroot/mozilla/layout/html/base/src/nsPresShell.cpp,v retrieving revision 3.360 diff -r3.360 nsPresShell.cpp 6052c6052 < //mReflowCountMgr->Add(aName, (nsReflowReason)aType); --- > mReflowCountMgr->Add(aName, (nsReflowReason)aType); Index: nsTableOuterFrame.cpp =================================================================== RCS file: /cvsroot/mozilla/layout/html/table/src/nsTableOuterFrame.cpp,v retrieving revision 3.173 diff -r3.173 nsTableOuterFrame.cpp 1392c1392 < DO_GLOBAL_REFLOW_COUNT("nsTableOuterFrame", aReflowState.reason); --- > DO_GLOBAL_REFLOW_COUNT("nsTableOuterFrame", aOuterRS.reason); I'll attach a patch shortly that enables MOZ_REFLOW_PERF via an autoconf option so it'll work on Linux.
patch looks good. firstname.lastname@example.org
rods patch looks good to me to, sr=waterson. cls, can you take a look at my patch to configure.in?
curt, another test that we want to add to daily verfication run. we should figure out some set of urls to hit (start small then go big) hit those urls with builds enabled to do reflow counting, and then spit the data out somewhere in daily and longer term trend reports. It would be good to attach a report, and to provide some idea of how to get the data to dump on a test run or url page load so we can start thinking about the best way to show the trend. does the data just show the number of reflows per session, or does it show by pages loaded? chris h.
assigned this one to curt.
Assignee: chofmann → curt
r=cls on the configure.in patch.
sr=leaf on the configure.in patch
Curt, I'll check in the changes to nsPresShell.cpp and nsTableOuterFrame - rods
checked in configure.in changes. this should be ready for a bit of automation work now...
This bug has not been touched since 2000-11-30. What are the current status of this bug. Is it being kept open for the mentioned automation work or can it be closed.
No reply in 5½ years, no dependencies => INCOMPLETE
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → INCOMPLETE
You need to log in before you can comment on or make changes to this bug.