Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(209)

Issue 8565027: Revert 109461 - (Closed)

Created:
9 years, 1 month ago by Jeff Timanus
Modified:
9 years, 1 month ago
Reviewers:
Jeff Timanus
CC:
chromium-reviews
Visibility:
Public.

Description

Revert 109461 - Re-introduction of release mode checks to determine the cause of CreateDIBSection failures. The crash is happening outside of the expected conditions in which CreateDIBSection would fail (large bitmap size, GDI and virtual memory pressure). To track the cause, this CL attempts to capture the last system error on bitmap allocation failure. See previous checks here: http://codereview.chromium.org/8341090 BUG=101934, 104063 TEST=NONE Review URL: http://codereview.chromium.org/8509030 TBR=twiz@chromium.org Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=109978

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -58 lines) Patch
M skia/ext/bitmap_platform_device_win.cc View 2 chunks +0 lines, -57 lines 0 comments Download
M skia/ext/canvas_paint_win.h View 1 chunk +4 lines, -1 line 0 comments Download

Messages

Total messages: 3 (0 generated)
Jeff Timanus
9 years, 1 month ago (2011-11-14 21:49:13 UTC) #1
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/twiz@chromium.org/8565027/1
9 years, 1 month ago (2011-11-14 21:54:59 UTC) #2
commit-bot: I haz the power
9 years, 1 month ago (2011-11-14 23:22:28 UTC) #3
Change committed as 109978

Powered by Google App Engine
This is Rietveld 408576698