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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: tests/GifTest.cpp

Issue 132403008: Remove unnamed namespace usage from tests/ (Closed) Base URL: https://skia.googlecode.com/svn/trunk
Patch Set: SK_OVERRIDE Created 6 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « tests/DrawBitmapRectTest.cpp ('k') | tests/ImageDecodingTest.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tests/GifTest.cpp
diff --git a/tests/GifTest.cpp b/tests/GifTest.cpp
index c2b8a8803212ed77539efb2dfd812314498370b9..7d5d56cc13037c16bd483a79935fe0fffda47b76 100644
--- a/tests/GifTest.cpp
+++ b/tests/GifTest.cpp
@@ -21,43 +21,36 @@
__SK_FORCE_IMAGE_DECODER_LINKING;
-namespace {
- unsigned char gifData[] = {
- 0x47, 0x49, 0x46, 0x38, 0x37, 0x61, 0x03, 0x00,
- 0x03, 0x00, 0xe3, 0x08, 0x00, 0x00, 0x00, 0x00,
- 0x00, 0x00, 0xff, 0xff, 0x00, 0x00, 0xff, 0x00,
- 0xff, 0x80, 0x80, 0x80, 0x00, 0xff, 0x00, 0x00,
- 0xff, 0xff, 0xff, 0xff, 0x00, 0xff, 0xff, 0xff,
- 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
- 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
- 0xff, 0xff, 0xff, 0xff, 0xff, 0x2c, 0x00, 0x00,
- 0x00, 0x00, 0x03, 0x00, 0x03, 0x00, 0x00, 0x04,
- 0x07, 0x50, 0x1c, 0x43, 0x40, 0x41, 0x23, 0x44,
- 0x00, 0x3b};
- unsigned char gifDataNoColormap[] = {
- 0x47, 0x49, 0x46, 0x38, 0x39, 0x61, 0x01, 0x00,
- 0x01, 0x00, 0x00, 0x00, 0x00, 0x21, 0xf9, 0x04,
- 0x01, 0x0a, 0x00, 0x01, 0x00, 0x2c, 0x00, 0x00,
- 0x00, 0x00, 0x01, 0x00, 0x01, 0x00, 0x00, 0x02,
- 0x02, 0x4c, 0x01, 0x00, 0x3b};
- unsigned char interlacedGif[] = {
- 0x47, 0x49, 0x46, 0x38, 0x37, 0x61, 0x09, 0x00,
- 0x09, 0x00, 0xe3, 0x08, 0x00, 0x00, 0x00, 0x00,
- 0x00, 0x00, 0xff, 0xff, 0x00, 0x00, 0xff, 0x00,
- 0xff, 0x80, 0x80, 0x80, 0x00, 0xff, 0x00, 0x00,
- 0xff, 0xff, 0xff, 0xff, 0x00, 0xff, 0xff, 0xff,
- 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
- 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
- 0xff, 0xff, 0xff, 0xff, 0xff, 0x2c, 0x00, 0x00,
- 0x00, 0x00, 0x09, 0x00, 0x09, 0x00, 0x40, 0x04,
- 0x1b, 0x50, 0x1c, 0x23, 0xe9, 0x44, 0x23, 0x60,
- 0x9d, 0x09, 0x28, 0x1e, 0xf8, 0x6d, 0x64, 0x56,
- 0x9d, 0x53, 0xa8, 0x7e, 0xa8, 0x65, 0x94, 0x5c,
- 0xb0, 0x8a, 0x45, 0x04, 0x00, 0x3b};
-}; // namespace
+static unsigned char gGIFData[] = {
+ 0x47, 0x49, 0x46, 0x38, 0x37, 0x61, 0x03, 0x00, 0x03, 0x00, 0xe3, 0x08,
+ 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0xff, 0xff, 0x00, 0x00, 0xff, 0x00,
+ 0xff, 0x80, 0x80, 0x80, 0x00, 0xff, 0x00, 0x00, 0xff, 0xff, 0xff, 0xff,
+ 0x00, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
+ 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
+ 0xff, 0x2c, 0x00, 0x00, 0x00, 0x00, 0x03, 0x00, 0x03, 0x00, 0x00, 0x04,
+ 0x07, 0x50, 0x1c, 0x43, 0x40, 0x41, 0x23, 0x44, 0x00, 0x3b
+};
+
+static unsigned char gGIFDataNoColormap[] = {
+ 0x47, 0x49, 0x46, 0x38, 0x39, 0x61, 0x01, 0x00, 0x01, 0x00, 0x00, 0x00, 0x00,
+ 0x21, 0xf9, 0x04, 0x01, 0x0a, 0x00, 0x01, 0x00, 0x2c, 0x00, 0x00, 0x00, 0x00,
+ 0x01, 0x00, 0x01, 0x00, 0x00, 0x02, 0x02, 0x4c, 0x01, 0x00, 0x3b
+};
+
+static unsigned char gInterlacedGIF[] = {
+ 0x47, 0x49, 0x46, 0x38, 0x37, 0x61, 0x09, 0x00, 0x09, 0x00, 0xe3, 0x08, 0x00,
+ 0x00, 0x00, 0x00, 0x00, 0x00, 0xff, 0xff, 0x00, 0x00, 0xff, 0x00, 0xff, 0x80,
+ 0x80, 0x80, 0x00, 0xff, 0x00, 0x00, 0xff, 0xff, 0xff, 0xff, 0x00, 0xff, 0xff,
+ 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff,
+ 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff, 0x2c, 0x00, 0x00, 0x00,
+ 0x00, 0x09, 0x00, 0x09, 0x00, 0x40, 0x04, 0x1b, 0x50, 0x1c, 0x23, 0xe9, 0x44,
+ 0x23, 0x60, 0x9d, 0x09, 0x28, 0x1e, 0xf8, 0x6d, 0x64, 0x56, 0x9d, 0x53, 0xa8,
+ 0x7e, 0xa8, 0x65, 0x94, 0x5c, 0xb0, 0x8a, 0x45, 0x04, 0x00, 0x3b
+};
static void test_gif_data_no_colormap(skiatest::Reporter* r,
- void* data, size_t size) {
+ void* data,
+ size_t size) {
SkBitmap bm;
bool imageDecodeSuccess = SkImageDecoder::DecodeMemory(
data, size, &bm);
@@ -149,57 +142,57 @@ static void test_gif_data_short(skiatest::Reporter* r,
*/
DEF_TEST(Gif, reporter) {
// test perfectly good images.
- test_gif_data(reporter, static_cast<void *>(gifData), sizeof(gifData));
- test_interlaced_gif_data(reporter, static_cast<void *>(interlacedGif),
- sizeof(interlacedGif));
+ test_gif_data(reporter, static_cast<void *>(gGIFData), sizeof(gGIFData));
+ test_interlaced_gif_data(reporter, static_cast<void *>(gInterlacedGIF),
+ sizeof(gInterlacedGIF));
- unsigned char badData[sizeof(gifData)];
+ unsigned char badData[sizeof(gGIFData)];
/* If you set the environment variable
skia_images_gif_suppressDecoderWarnings to 'false', you will
see warnings on stderr. This is a feature. */
- memcpy(badData, gifData, sizeof(gifData));
+ memcpy(badData, gGIFData, sizeof(gGIFData));
badData[6] = 0x01; // image too wide
- test_gif_data(reporter, static_cast<void *>(badData), sizeof(gifData));
+ test_gif_data(reporter, static_cast<void *>(badData), sizeof(gGIFData));
// "libgif warning [image too wide, expanding output to size]"
- memcpy(badData, gifData, sizeof(gifData));
+ memcpy(badData, gGIFData, sizeof(gGIFData));
badData[8] = 0x01; // image too tall
- test_gif_data(reporter, static_cast<void *>(badData), sizeof(gifData));
+ test_gif_data(reporter, static_cast<void *>(badData), sizeof(gGIFData));
// "libgif warning [image too tall, expanding output to size]"
- memcpy(badData, gifData, sizeof(gifData));
+ memcpy(badData, gGIFData, sizeof(gGIFData));
badData[62] = 0x01; // image shifted right
- test_gif_data(reporter, static_cast<void *>(badData), sizeof(gifData));
+ test_gif_data(reporter, static_cast<void *>(badData), sizeof(gGIFData));
// "libgif warning [shifting image left to fit]"
- memcpy(badData, gifData, sizeof(gifData));
+ memcpy(badData, gGIFData, sizeof(gGIFData));
badData[64] = 0x01; // image shifted down
- test_gif_data(reporter, static_cast<void *>(badData), sizeof(gifData));
+ test_gif_data(reporter, static_cast<void *>(badData), sizeof(gGIFData));
// "libgif warning [shifting image up to fit]"
- memcpy(badData, gifData, sizeof(gifData));
+ memcpy(badData, gGIFData, sizeof(gGIFData));
badData[62] = 0xff; // image shifted left
badData[63] = 0xff; // 2's complement -1 short
- test_gif_data(reporter, static_cast<void *>(badData), sizeof(gifData));
+ test_gif_data(reporter, static_cast<void *>(badData), sizeof(gGIFData));
// "libgif warning [shifting image left to fit]"
- memcpy(badData, gifData, sizeof(gifData));
+ memcpy(badData, gGIFData, sizeof(gGIFData));
badData[64] = 0xff; // image shifted up
badData[65] = 0xff; // 2's complement -1 short
- test_gif_data(reporter, static_cast<void *>(badData), sizeof(gifData));
+ test_gif_data(reporter, static_cast<void *>(badData), sizeof(gGIFData));
// "libgif warning [shifting image up to fit]"
- test_gif_data_no_colormap(reporter, static_cast<void *>(gifDataNoColormap),
- sizeof(gifDataNoColormap));
+ test_gif_data_no_colormap(reporter, static_cast<void *>(gGIFDataNoColormap),
+ sizeof(gGIFDataNoColormap));
// "libgif warning [missing colormap]"
// test short Gif. 80 is missing a few bytes.
- test_gif_data_short(reporter, static_cast<void *>(gifData), 80);
+ test_gif_data_short(reporter, static_cast<void *>(gGIFData), 80);
// "libgif warning [DGifGetLine]"
- test_interlaced_gif_data(reporter, static_cast<void *>(interlacedGif),
+ test_interlaced_gif_data(reporter, static_cast<void *>(gInterlacedGIF),
100); // 100 is missing a few bytes
// "libgif warning [interlace DGifGetLine]"
}
« no previous file with comments | « tests/DrawBitmapRectTest.cpp ('k') | tests/ImageDecodingTest.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698