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

Issue 6286157: Test BMP files more thoroughly to be sure they're valid. (Closed)

Created:
9 years, 10 months ago by Bill Richardson
Modified:
9 years, 6 months ago
Reviewers:
Randall Spangler
CC:
chromium-os-reviews_chromium.org, Randall Spangler, gauravsh, Luigi Semenzato, Bill Richardson
Visibility:
Public.

Description

Test BMP files more thoroughly to be sure they're valid. This is work-in-progress. More tests to come... Change-Id: Id2e59fd7d0229be3ad90b29b2d0dc035ceeca666 BUG=chromium-os:11766 TEST=manual Adding an example to the test framework. Use make make runbmptests to ensure it works. Committed: http://chrome-svn/viewvc/chromeos?view=rev&revision=fc05bb8

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+53 lines, -21 lines) Patch
M tests/bitmaps/NotReallyA.bmp View Binary file 0 comments Download
M utility/bmpblk_utility.cc View 5 chunks +53 lines, -21 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
Bill Richardson
9 years, 10 months ago (2011-02-07 23:56:29 UTC) #1
Randall Spangler
9 years, 10 months ago (2011-02-08 18:30:20 UTC) #2
LGTM

Powered by Google App Engine
This is Rietveld 408576698