[Webkit-unassigned] [Bug 47181] SVGFEMergeNodeElement doesn't support dynamic invalidation

Tue Oct 5 10:00:47 PDT 2010

https://bugs.webkit.org/show_bug.cgi?id=47181

Dirk Schulze <krit at webkit.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #69792|review?, commit-queue?      |review-, commit-queue-
               Flag|                            |

--- Comment #2 from Dirk Schulze <krit at webkit.org>  2010-10-05 10:00:47 PST ---
(From update of attachment 69792)
Not sure, why you need feComposite on the tests. A test like this should be as simple as possible :-) Nevertheless, the patch itself looks great, but you should describe in the ChangeLog what you did to fix this issue. For example why did you ask the parent? why doesn't feMergeNode have a renderer, so that you have to call invalidation via the parent? And so on. As long as you don't have commit privileges, you have to upload a second patch. :-( r- because of the ChangeLog.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.