Bommels05/Mozilla
Bommels05/Mozilla
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Bug 9489. Remove cliprect push/pop to decrease paint time on titledbuttons. The cliprect was not doing anything useful, and was costing about 36% of titledbutton paint time. r=hyatt

Browse Source 
git-svn-id: svn://10.0.0.236/trunk@55004 18797224-902f-48f8-a5cc-f745e15eee43
This commit is contained in:
waterson%netscape.com 1999-12-02 02:50:29 +00:00
parent fed44b6639
commit 3db2b20a4c
1 changed files with 11 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
mozilla/layout/xul/base/src/nsTitledButtonFrame.cpp
View File

@ -464,16 +464,27 @@ nsTitledButtonFrame::Paint(nsIPresContext* aPresContext,
mRenderer->PaintButton(aPresContext, aRenderingContext, aDirtyRect, aWhichLayer, rect);
#if 0
// Removing this code because it seriously impacts performance of
// titled buttons without providing any visible benefit. Please
// talk to hyatt or waterson if you think this code should be put
// back in. Ideally, we could pessimistically create a clip rect
// only if LayoutTitleAndImage() determines that the drawing would
// spill outside of the available rect.
aRenderingContext.PushState();
PRBool clipState;
aRenderingContext.SetClipRect(rect, nsClipCombine_kIntersect, clipState);
#endif
LayoutTitleAndImage(aPresContext, aRenderingContext, aDirtyRect, aWhichLayer);
PaintTitle(aPresContext, aRenderingContext, aDirtyRect, aWhichLayer);
PaintImage(aPresContext, aRenderingContext, aDirtyRect, aWhichLayer);
#if 0
aRenderingContext.PopState(clipState);
#endif
/*
aRenderingContext.SetColor(NS_RGB(0,128,0));