RESOLVED FIXED 96801
[CSS Shaders] Refactor FECustomFilter to make it usable from platform dependent implementations 
https://bugs.webkit.org/show_bug.cgi?id=96801
Summary [CSS Shaders] Refactor FECustomFilter to make it usable from platform depende...
Alexandru Chiculita
Reported 2012-09-14 09:56:37 PDT
Refactoring the code in FECustomFilter to make it easy to use from FECustomFilterSkia.
Attachments
Patch V1 (16.18 KB, patch)
2012-09-14 10:08 PDT, Alexandru Chiculita 		no flags
DetailsFormatted DiffDiff
Patch V2 (16.12 KB, patch)
2012-09-14 10:10 PDT, Alexandru Chiculita 		no flags
DetailsFormatted DiffDiff
Patch V3 (16.18 KB, patch)
2012-09-14 10:25 PDT, Alexandru Chiculita 		dino: review+
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Alexandru Chiculita
Comment 1 2012-09-14 10:08:27 PDT
Created attachment 164176 [details] Patch V1
Alexandru Chiculita
Comment 2 2012-09-14 10:10:47 PDT
Created attachment 164178 [details] Patch V2 I forgot to remove the Skia method in FECustomFilter.h. Fixed it now.
Alexandru Chiculita
Comment 3 2012-09-14 10:25:52 PDT
Created attachment 164184 [details] Patch V3 Rebased.
Dean Jackson
Comment 4 2012-09-14 11:28:40 PDT
Comment on attachment 164184 [details] Patch V3 View in context: https://bugs.webkit.org/attachment.cgi?id=164184&action=review I didn't look through this so carefully since I'd already done it in the original bug :) > Source/WebCore/platform/graphics/filters/FECustomFilter.cpp:185 > + // Only allocate a texture if the program needs one and the caller doesn't allocate one by itself. > + if (programNeedsInputTexture() && (filterDrawType == NEEDS_INPUT_TEXTURE) && !ensureInputTexture()) > + return false; > + if (!ensureFrameBuffer()) > + return false; Any reason why we don't merge that into one test?
Alexandru Chiculita
Comment 5 2012-09-14 11:54:34 PDT
Comment on attachment 164184 [details] Patch V3 Thanks Dean!
Alexandru Chiculita
Comment 6 2012-09-14 12:35:17 PDT
Landed in http://trac.webkit.org/changeset/128641 .
Note You need to log in before you can comment on or make changes to this bug.