Summary: | Use is<>() / downcast<>() for all SVG RenderObjects | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Chris Dumez <cdumez> | ||||
Component: | SVG | Assignee: | Chris Dumez <cdumez> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | aboxhall, apinheiro, cfleizach, commit-queue, dino, dmazzoni, d-r, esprehn+autocc, fmalita, glenn, gyuyoung.kim, jcraig, jdiggs, kondapallykalyan, mario, mmaxfield, pdr, samuel_white, schenney, sergio, simon.fraser, zimmermann | ||||
Priority: | P2 | ||||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Bug Depends on: | 137424 | ||||||
Bug Blocks: | |||||||
Attachments: |
|
Description
Chris Dumez
2014-10-17 16:14:35 PDT
Created attachment 240050 [details]
Patch
Comment on attachment 240050 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=240050&action=review > Source/WebCore/rendering/RenderLayerFilterInfo.cpp:131 > + for (const auto& filter : m_internalSVGReferences) { Shouldn't this be const auto*? > Source/WebCore/rendering/svg/SVGRootInlineBox.cpp:-80 > - RenderSVGText* textRoot = toRenderSVGText(&blockFlow()); > - ASSERT(textRoot); ahahahahah Comment on attachment 240050 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=240050&action=review >> Source/WebCore/rendering/RenderLayerFilterInfo.cpp:131 >> + for (const auto& filter : m_internalSVGReferences) { > > Shouldn't this be const auto*? It is a RefPtr type. Comment on attachment 240050 [details] Patch Clearing flags on attachment: 240050 Committed r174854: <http://trac.webkit.org/changeset/174854> All reviewed patches have been landed. Closing bug. |