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

Side by Side Diff: src/core/SkWriteBuffer.cpp

Issue 1199473002: change old picture serialization to really handle images (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: Created 5 years, 6 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
OLDNEW
1 1
2 /* 2 /*
3 * Copyright 2012 Google Inc. 3 * Copyright 2012 Google Inc.
4 * 4 *
5 * Use of this source code is governed by a BSD-style license that can be 5 * Use of this source code is governed by a BSD-style license that can be
6 * found in the LICENSE file. 6 * found in the LICENSE file.
7 */ 7 */
8 8
9 #include "SkWriteBuffer.h" 9 #include "SkWriteBuffer.h"
10 #include "SkBitmap.h" 10 #include "SkBitmap.h"
(...skipping 200 matching lines...) Expand 10 before | Expand all | Expand 10 after
211 write_encoded_bitmap(this, data, SkIPoint::Make(0, 0)); 211 write_encoded_bitmap(this, data, SkIPoint::Make(0, 0));
212 return; 212 return;
213 } 213 }
214 } 214 }
215 } 215 }
216 216
217 this->writeUInt(0); // signal raw pixels 217 this->writeUInt(0); // signal raw pixels
218 SkBitmap::WriteRawPixels(this, bitmap); 218 SkBitmap::WriteRawPixels(this, bitmap);
219 } 219 }
220 220
221 static bool try_write_encoded(SkWriteBuffer* buffer, SkData* encoded, SkIPoint o rigin) {
222 SkPixelSerializer* ps = buffer->getPixelSerializer();
223 // Assumes that if the client did not set a serializer, they are
224 // happy to get the encoded data.
225 if (!ps || ps->useEncodedData(encoded->data(), encoded->size())) {
226 write_encoded_bitmap(buffer, encoded, origin);
227 return true;
228 }
229 return false;
230 }
231
232 void SkWriteBuffer::writeImage(const SkImage* image) {
233 this->writeInt(image->width());
234 this->writeInt(image->height());
235
236 SkIPoint origin;
237 SkAutoTUnref<SkData> encoded(image->refEncodedData(&origin));
238 if (encoded && try_write_encoded(this, encoded, origin)) {
239 return;
240 }
241
242 encoded.reset(image->encode(SkImageEncoder::kPNG_Type, 100));
243 if (encoded && try_write_encoded(this, encoded, SkIPoint::Make(0, 0))) {
244 return;
245 }
246
247 this->writeUInt(0); // signal no pixels
248 }
249
221 void SkWriteBuffer::writeTypeface(SkTypeface* obj) { 250 void SkWriteBuffer::writeTypeface(SkTypeface* obj) {
222 if (NULL == obj || NULL == fTFSet) { 251 if (NULL == obj || NULL == fTFSet) {
223 fWriter.write32(0); 252 fWriter.write32(0);
224 } else { 253 } else {
225 fWriter.write32(fTFSet->add(obj)); 254 fWriter.write32(fTFSet->add(obj));
226 } 255 }
227 } 256 }
228 257
229 SkFactorySet* SkWriteBuffer::setFactoryRecorder(SkFactorySet* rec) { 258 SkFactorySet* SkWriteBuffer::setFactoryRecorder(SkFactorySet* rec) {
230 SkRefCnt_SafeAssign(fFactorySet, rec); 259 SkRefCnt_SafeAssign(fFactorySet, rec);
(...skipping 90 matching lines...) Expand 10 before | Expand all | Expand 10 after
321 // make room for the size of the flattened object 350 // make room for the size of the flattened object
322 (void)fWriter.reserve(sizeof(uint32_t)); 351 (void)fWriter.reserve(sizeof(uint32_t));
323 // record the current size, so we can subtract after the object writes. 352 // record the current size, so we can subtract after the object writes.
324 size_t offset = fWriter.bytesWritten(); 353 size_t offset = fWriter.bytesWritten();
325 // now flatten the object 354 // now flatten the object
326 flattenable->flatten(*this); 355 flattenable->flatten(*this);
327 size_t objSize = fWriter.bytesWritten() - offset; 356 size_t objSize = fWriter.bytesWritten() - offset;
328 // record the obj's size 357 // record the obj's size
329 fWriter.overwriteTAt(offset - sizeof(uint32_t), SkToU32(objSize)); 358 fWriter.overwriteTAt(offset - sizeof(uint32_t), SkToU32(objSize));
330 } 359 }
OLDNEW
« gm/factory.cpp ('K') | « src/core/SkReadBuffer.h ('k') | src/image/SkImage.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698