Summary: | [TexMap] Composited CSS shaders crash when using non-GL TextureMapper | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Allan Sandfeld Jensen <allan.jensen> | ||||||
Component: | Layout and Rendering | Assignee: | Allan Sandfeld Jensen <allan.jensen> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | dino, eric, noam, ojan.autocc, webkit.review.bot | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Attachments: |
|
Description
Allan Sandfeld Jensen
2013-01-14 07:43:46 PST
Created attachment 182739 [details]
Patch
Comment on attachment 182739 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=182739&action=review > Source/WebCore/rendering/FilterEffectRenderer.cpp:424 > + if (!effect) > + return; By looking at the rest of FilterEffectRenderer, it looks like lastEffect is expected not to be null, so I'm not sure if this is the best fix longer term. It feels like none of this code makes sense if the list of effect is empty. Is there a way you could fix it at the caller? (In reply to comment #2) > (From update of attachment 182739 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=182739&action=review > > > Source/WebCore/rendering/FilterEffectRenderer.cpp:424 > > + if (!effect) > > + return; > > By looking at the rest of FilterEffectRenderer, it looks like lastEffect is expected not to be null, so I'm not sure if this is the best fix longer term. > It feels like none of this code makes sense if the list of effect is empty. Is there a way you could fix it at the caller? The rest of the patch is fixing it at the caller, so this change on it is own is not needed to fix the crash (though it was here it crashed), but since a FilterEffectRenderer can be left "invalid" after a failed build() call, I felt it would be a good idea not to crash on a state that FilterEffectRenderer has left itself in. Created attachment 182782 [details]
Patch
Committed r139864: <http://trac.webkit.org/changeset/139864> |