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

Side by Side Diff: src/ports/SkImageGenerator_skia.cpp

Issue 1263013002: Remove SK_LEGACY_IMAGE_GENERATOR_ENUMS_AND_OPTIONS (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: Created 5 years, 4 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 unified diff | Download patch
« no previous file with comments | « src/images/SkDecodingImageGenerator.cpp ('k') | tests/CachedDecodingPixelRefTest.cpp » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* 1 /*
2 * Copyright 2015 The Android Open Source Project 2 * Copyright 2015 The Android Open Source Project
3 * 3 *
4 * Use of this source code is governed by a BSD-style license that can be 4 * Use of this source code is governed by a BSD-style license that can be
5 * found in the LICENSE file. 5 * found in the LICENSE file.
6 */ 6 */
7 7
8 #include "SkData.h" 8 #include "SkData.h"
9 #include "SkImageDecoder.h" 9 #include "SkImageDecoder.h"
10 #include "SkImageGenerator.h" 10 #include "SkImageGenerator.h"
(...skipping 28 matching lines...) Expand all
39 39
40 public: 40 public:
41 SkImageDecoderGenerator(const SkImageInfo& info, SkImageDecoder* decoder, Sk Data* data) 41 SkImageDecoderGenerator(const SkImageInfo& info, SkImageDecoder* decoder, Sk Data* data)
42 : INHERITED(info), fInfo(info), fDecoder(decoder), fData(SkRef(data)) 42 : INHERITED(info), fInfo(info), fDecoder(decoder), fData(SkRef(data))
43 {} 43 {}
44 44
45 protected: 45 protected:
46 SkData* onRefEncodedData() override { 46 SkData* onRefEncodedData() override {
47 return SkRef(fData.get()); 47 return SkRef(fData.get());
48 } 48 }
49 #ifdef SK_LEGACY_IMAGE_GENERATOR_ENUMS_AND_OPTIONS
50 Result onGetPixels(const SkImageInfo& info, void* pixels, size_t rowBytes,
51 const Options&,
52 SkPMColor ctableEntries[], int* ctableCount) override {
53 #else
54 bool onGetPixels(const SkImageInfo& info, void* pixels, size_t rowBytes, 49 bool onGetPixels(const SkImageInfo& info, void* pixels, size_t rowBytes,
55 SkPMColor ctableEntries[], int* ctableCount) override { 50 SkPMColor ctableEntries[], int* ctableCount) override {
56 #endif
57 SkMemoryStream stream(fData->data(), fData->size(), false); 51 SkMemoryStream stream(fData->data(), fData->size(), false);
58 SkAutoTUnref<BareMemoryAllocator> allocator(SkNEW_ARGS(BareMemoryAllocat or, 52 SkAutoTUnref<BareMemoryAllocator> allocator(SkNEW_ARGS(BareMemoryAllocat or,
59 (info, pixels, ro wBytes))); 53 (info, pixels, ro wBytes)));
60 fDecoder->setAllocator(allocator); 54 fDecoder->setAllocator(allocator);
61 fDecoder->setRequireUnpremultipliedColors(kUnpremul_SkAlphaType == info. alphaType()); 55 fDecoder->setRequireUnpremultipliedColors(kUnpremul_SkAlphaType == info. alphaType());
62 56
63 SkBitmap bm; 57 SkBitmap bm;
64 const SkImageDecoder::Result result = fDecoder->decode(&stream, &bm, inf o.colorType(), 58 const SkImageDecoder::Result result = fDecoder->decode(&stream, &bm, inf o.colorType(),
65 SkImageDecoder::k DecodePixels_Mode); 59 SkImageDecoder::k DecodePixels_Mode);
66 if (SkImageDecoder::kFailure == result) { 60 if (SkImageDecoder::kFailure == result) {
67 #ifdef SK_LEGACY_IMAGE_GENERATOR_ENUMS_AND_OPTIONS
68 return kInvalidInput;
69 #else
70 return false; 61 return false;
71 #endif
72 } 62 }
73 63
74 SkASSERT(info.colorType() == bm.info().colorType()); 64 SkASSERT(info.colorType() == bm.info().colorType());
75 65
76 if (kIndex_8_SkColorType == info.colorType()) { 66 if (kIndex_8_SkColorType == info.colorType()) {
77 SkASSERT(ctableEntries); 67 SkASSERT(ctableEntries);
78 68
79 SkColorTable* ctable = bm.getColorTable(); 69 SkColorTable* ctable = bm.getColorTable();
80 if (NULL == ctable) { 70 if (NULL == ctable) {
81 #ifdef SK_LEGACY_IMAGE_GENERATOR_ENUMS_AND_OPTIONS
82 return kInvalidConversion;
83 #else
84 return false; 71 return false;
85 #endif
86 } 72 }
87 const int count = ctable->count(); 73 const int count = ctable->count();
88 memcpy(ctableEntries, ctable->readColors(), count * sizeof(SkPMColor )); 74 memcpy(ctableEntries, ctable->readColors(), count * sizeof(SkPMColor ));
89 *ctableCount = count; 75 *ctableCount = count;
90 } 76 }
91 #ifdef SK_LEGACY_IMAGE_GENERATOR_ENUMS_AND_OPTIONS
92 if (SkImageDecoder::kPartialSuccess == result) {
93 return kIncompleteInput;
94 }
95 return kSuccess;
96 #else
97 return true; 77 return true;
98 #endif
99 } 78 }
100 79
101 bool onGetYUV8Planes(SkISize sizes[3], void* planes[3], size_t rowBytes[3], 80 bool onGetYUV8Planes(SkISize sizes[3], void* planes[3], size_t rowBytes[3],
102 SkYUVColorSpace* colorSpace) override { 81 SkYUVColorSpace* colorSpace) override {
103 SkMemoryStream stream(fData->data(), fData->size(), false); 82 SkMemoryStream stream(fData->data(), fData->size(), false);
104 return fDecoder->decodeYUV8Planes(&stream, sizes, planes, rowBytes, colo rSpace); 83 return fDecoder->decodeYUV8Planes(&stream, sizes, planes, rowBytes, colo rSpace);
105 } 84 }
106 85
107 private: 86 private:
108 typedef SkImageGenerator INHERITED; 87 typedef SkImageGenerator INHERITED;
109 }; 88 };
110 89
111 SkImageGenerator* SkImageGenerator::NewFromEncodedImpl(SkData* data) { 90 SkImageGenerator* SkImageGenerator::NewFromEncodedImpl(SkData* data) {
112 SkMemoryStream stream(data->data(), data->size(), false); 91 SkMemoryStream stream(data->data(), data->size(), false);
113 SkImageDecoder* decoder = SkImageDecoder::Factory(&stream); 92 SkImageDecoder* decoder = SkImageDecoder::Factory(&stream);
114 if (NULL == decoder) { 93 if (NULL == decoder) {
115 return NULL; 94 return NULL;
116 } 95 }
117 96
118 SkBitmap bm; 97 SkBitmap bm;
119 stream.rewind(); 98 stream.rewind();
120 if (!decoder->decode(&stream, &bm, kUnknown_SkColorType, SkImageDecoder::kDe codeBounds_Mode)) { 99 if (!decoder->decode(&stream, &bm, kUnknown_SkColorType, SkImageDecoder::kDe codeBounds_Mode)) {
121 SkDELETE(decoder); 100 SkDELETE(decoder);
122 return NULL; 101 return NULL;
123 } 102 }
124 103
125 return SkNEW_ARGS(SkImageDecoderGenerator, (bm.info(), decoder, data)); 104 return SkNEW_ARGS(SkImageDecoderGenerator, (bm.info(), decoder, data));
126 } 105 }
OLDNEW
« no previous file with comments | « src/images/SkDecodingImageGenerator.cpp ('k') | tests/CachedDecodingPixelRefTest.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698