[webkit-reviews] review granted: [Bug 95110] [Chromium] Fixing infinite recursion in Canvas2DLayerManager : [Attachment 160767] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon Aug 27 12:08:02 PDT 2012


Stephen White <senorblanco at chromium.org> has granted Justin Novosad
<junov at google.com>'s request for review:
Bug 95110: [Chromium] Fixing infinite recursion in Canvas2DLayerManager
https://bugs.webkit.org/show_bug.cgi?id=95110

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

------- Additional Comments from Stephen White <senorblanco at chromium.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=160767&action=review


r=me

> Source/WebCore/platform/graphics/chromium/Canvas2DLayerBridge.cpp:128
>      m_bytesAllocated -= bytesFreed;

Micronit:  this could be in the if-clause too.

> Source/WebKit/chromium/tests/Canvas2DLayerManagerTest.cpp:62
>	   m_bytesAllocated -= bytesFreed;

As above.


More information about the webkit-reviews mailing list