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

Issue 387056: Avoid double freeing image list in TableView.... (Closed)

Created:
11 years, 1 month ago by dank
Modified:
9 years, 7 months ago
Reviewers:
James Hawkins
CC:
chromium-reviews_googlegroups.com, ben+cc_chromium.org
Visibility:
Public.

Description

Avoid double freeing image list in TableView. BUG=27729 TEST=run NullModelTableViewTest.NullModel in app verifier or wine

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -13 lines) Patch
M views/controls/table/table_view.h View 1 chunk +0 lines, -3 lines 0 comments Download
M views/controls/table/table_view.cc View 1 chunk +0 lines, -10 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
dank
Windows protects itself from this problem (usually?), but it's still an error, and the fix ...
11 years, 1 month ago (2009-11-14 19:59:05 UTC) #1
James Hawkins
LGTM
11 years, 1 month ago (2009-11-16 18:47:36 UTC) #2
James Hawkins
11 years ago (2009-12-04 21:38:49 UTC) #3
Issue can be closed?

Powered by Google App Engine
This is Rietveld 408576698