Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(401)

Issue 5101041: Add asserts to distiguish sandbox issues and real GDI corruption. (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
13 years, 1 month ago by arthurhsu
Modified:
13 years, 1 month ago
Reviewers:
Steve VanDeBogart
CC:
skia-review_googlegroups.com
Base URL:
http://skia.googlecode.com/svn/trunk
Visibility:
Public.

Description

Add asserts to distinguish sandbox issues and real GDI corruption. Original code review: http://codereview.appspot.com/4974049. BUG=none TEST=none

Patch Set 1 #

Patch Set 2 : Add assert only, discard other changes. #

Patch Set 3 : Fix patch error #

Unified diffs Side-by-side diffs Delta from patch set Stats (+6 lines, -0 lines) Patch
M src/ports/SkFontHost_win.cpp View 1 2 1 chunk +6 lines, -0 lines 0 comments Download

Messages

Total messages: 3
arthurhsu
13 years, 1 month ago (2011-09-22 19:05:14 UTC) #1
arthurhsu
Changed to add assertion only.
13 years, 1 month ago (2011-09-22 21:08:14 UTC) #2
Steve VanDeBogart
13 years, 1 month ago (2011-09-22 21:17:23 UTC) #3
LGTM
Committed as r2315
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b