[webkit-reviews] review granted: [Bug 107701] Canvas and DOM go out of sync : [Attachment 188179] Patch

[bugzilla-daemon at webkit.org]

Simon Fraser (smfr) <simon.fraser at apple.com> has granted sugoi at chromium.org's
request for review:
Bug 107701: Canvas and DOM go out of sync
https://bugs.webkit.org/show_bug.cgi?id=107701

Attachment 188179: Patch
https://bugs.webkit.org/attachment.cgi?id=188179&action=review

------- Additional Comments from Simon Fraser (smfr) <simon.fraser at apple.com>
View in context: https://bugs.webkit.org/attachment.cgi?id=188179&action=review


>> Source/WebCore/rendering/RenderLayerBacking.cpp:465
>> +		   
layerCompositor->updateCompositingDescendantGeometry(stackingContainer,
stackingContainer, flags & CompositingChildrenOnly);
> 
> So for the root layer, this will do both the
updateCompositingDescendantGeometry() a few lines above, and then do another
update from the SC ancestor?

Actually I think this is fine.