RESOLVED FIXED 19592
Mismatched GraphicsContext::save()/restore() pairs in shadow painting code 
https://bugs.webkit.org/show_bug.cgi?id=19592
Summary Mismatched GraphicsContext::save()/restore() pairs in shadow painting code
Jonathon Jongsma (jonner)
Reported 2008-06-16 12:12:03 PDT
If you look at the following code, you'll notice two calls to context->save() with only a single restore() at teh end of the function. The context is not modified at all between the two calls to save(), so I think the first one is useless and will lead to mismatched save/restore pairs. http://trac.webkit.org/browser/trunk/WebCore/rendering/InlineFlowBox.cpp#L934
Attachments
remove the extra save() (1.79 KB, patch)
2008-06-16 13:01 PDT, Jonathon Jongsma (jonner) 		mitz: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Jonathon Jongsma (jonner)
Comment 1 2008-06-16 13:01:34 PDT
Created attachment 21743 [details] remove the extra save() I'm not sure if there's a test we can add for this. I haven't been able to actually come up with a rendering failure due to this, it just looks like it may be a bug in waiting.
mitz
Comment 2 2008-06-16 13:07:49 PDT
Comment on attachment 21743 [details] remove the extra save() r=me I am going to land it, tweaking the change log format and removing the reference to GraphicsContextCairo.cpp.
Jonathon Jongsma (jonner)
Comment 3 2008-06-16 13:08:45 PDT
oops, how did that get in there? :P thanks.
mitz
Comment 4 2008-06-16 13:11:30 PDT
Landed in <http://trac.webkit.org/changeset/34604>.
Note You need to log in before you can comment on or make changes to this bug.