RESOLVED FIXED56134
PlatformContextSkia should flush GrContext in destructor 
https://bugs.webkit.org/show_bug.cgi?id=56134
Summary PlatformContextSkia should flush GrContext in destructor
Brian Salomon
Reported 2011-03-10 12:49:49 PST
When a PlatformContextSkia is backed by a GrContext it should flush the GrContext in its destructor. Otherwise, the GrRenderTarget that the skia canvas is drawing into refers to a DrawingBuffer's FBO id that may be destroyed before the next time the GrContext is flushed.
Attachments
fix (2.64 KB, patch)
2011-03-10 12:51 PST, Brian Salomon 		no flags
DetailsFormatted DiffDiff
patch (2.32 KB, patch)
2011-03-10 12:56 PST, Brian Salomon 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Brian Salomon
Comment 1 2011-03-10 12:51:33 PST
Created attachment 85377 [details] fix This patch also changes the parameter passed to GrContext::flush from a boolean to a int. A recent change to skia made this parameter a bitfield of flags rather than a boolean.
Brian Salomon
Comment 2 2011-03-10 12:56:35 PST
Created attachment 85378 [details] patch
Kenneth Russell
Comment 3 2011-03-10 13:48:33 PST
Comment on attachment 85378 [details] patch Looks fine.
WebKit Commit Bot
Comment 4 2011-03-11 00:07:04 PST
Comment on attachment 85378 [details] patch Clearing flags on attachment: 85378 Committed r80826: <http://trac.webkit.org/changeset/80826>
WebKit Commit Bot
Comment 5 2011-03-11 00:07:09 PST
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.