For example: <div>Hello, world!</div> Applying style="-webkit-filter: blur(1px);" to this div makes it disappear. Sometimes it flickers into view or shows up depending on the order of rendering. PR 325051
Created attachment 197739 [details] Patch
I have been considering whether to add a new layout test that tries to apply a filter to some text. However, there's no reason to believe that other ports would fail to do that - being able to apply filters to images and rectangles means they will be able to apply them to anything. A test like that would just be duplication of already tested functionality to all other ports, and thus contribute nothing of value. Also, the BlackBerry port doesn't have pixel tests up and running yet, so such a test would not actually prevent regression of this fix.
For these reasons I decided against adding a new layout test. For other ports, this is really already covered by existing layout tests.
Comment on attachment 197739 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=197739&action=review > Source/WebCore/platform/graphics/blackberry/LayerFilterRenderer.cpp:712 > + glVertexAttribPointer(m_positionLocation, 2, GL_FLOAT, GL_FALSE, 0, &layer->getTransformedBounds() ); There's an extra space between &layer->getTransformedBounds() and the last parentheses.
(In reply to comment #4) > (From update of attachment 197739 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=197739&action=review > > > Source/WebCore/platform/graphics/blackberry/LayerFilterRenderer.cpp:712 > > + glVertexAttribPointer(m_positionLocation, 2, GL_FLOAT, GL_FALSE, 0, &layer->getTransformedBounds() ); > > There's an extra space between &layer->getTransformedBounds() and the last parentheses. Thank you, I'll fix that before landing =)
Created attachment 198034 [details] Patch
Comment on attachment 198034 [details] Patch Clearing flags on attachment: 198034 Committed r148423: <http://trac.webkit.org/changeset/148423>
All reviewed patches have been landed. Closing bug.