Chromium Code Reviews (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out

Issue 9340004: Merge 104501 - [chromium win] Creating lots of temporary canvas contexts will crash. (Closed)

8 years, 10 months ago by Stephen White
8 years, 10 months ago
Justin Novosad
Base URL:


Merge 104501 - [chromium win] Creating lots of temporary canvas contexts will crash. When using the skia port, the allocation of 2d canvas backing stores no longer needs to be done through a platform API (GDI/CG) because canvases now use skia for drawing text. Removing the allocation through GDI on windows prevents resource exhaustion due to unreferenced canvases that are awaiting garbage collection. Patch by Justin Novosad <>; on 2012-01-09 Reviewed by Stephen White. No new tests: Relying on existing canvas layout tests. * html/HTMLCanvasElement.cpp: (WebCore::HTMLCanvasElement::createImageBuffer): * platform/graphics/ImageBuffer.h: * platform/graphics/skia/ImageBufferSkia.cpp: (WebCore::createNonPlatformCanvas): (WebCore::ImageBuffer::ImageBuffer): Committed:

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+16 lines, -1 line) Patch
M Source/WebCore/html/HTMLCanvasElement.cpp View 1 chunk +6 lines, -1 line 0 comments Download
M Source/WebCore/platform/graphics/ImageBuffer.h View 1 chunk +1 line, -0 lines 0 comments Download
M Source/WebCore/platform/graphics/skia/ImageBufferSkia.cpp View 2 chunks +9 lines, -0 lines 0 comments Download


Total messages: 2 (0 generated)
Stephen White
8 years, 10 months ago (2012-02-06 21:43:18 UTC) #1
Justin Novosad
8 years, 10 months ago (2012-02-06 21:56:17 UTC) #2
On 2012/02/06 21:43:18, Stephen White wrote:


Powered by Google App Engine
This is Rietveld 408576698