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

Side by Side Diff: src/lazy/SkCachingPixelRef.cpp

Issue 54203006: Break up SkLazyPixelRef functionally into class hierarchy. (Closed) Base URL: https://skia.googlecode.com/svn/trunk
Patch Set: last change Created 7 years, 1 month 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 | Annotate | Revision Log
« no previous file with comments | « src/lazy/SkCachingPixelRef.h ('k') | src/lazy/SkLazyCachingPixelRef.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
(Empty)
1 /*
2 * Copyright 2013 Google Inc.
3 *
4 * Use of this source code is governed by a BSD-style license that can be
5 * found in the LICENSE file.
6 */
7
8 #include "SkCachingPixelRef.h"
9 #include "SkScaledImageCache.h"
10
11 SkCachingPixelRef::SkCachingPixelRef()
12 : fErrorInDecoding(false)
13 , fScaledCacheId(NULL) {
14 memset(&fInfo, 0xFF, sizeof(fInfo));
15 }
16 SkCachingPixelRef::~SkCachingPixelRef() {
17 SkASSERT(NULL == fScaledCacheId);
18 // Assert always unlock before unref.
19 }
20
21 bool SkCachingPixelRef::getInfo(SkImageInfo* info) {
22 SkASSERT(info != NULL);
23 if (fErrorInDecoding) {
24 return false; // Don't try again.
25 }
26 if (fInfo.fWidth < 0) {
27 SkImageInfo tmp;
28 if (!this->onDecodeInfo(&tmp)) {
29 fErrorInDecoding = true;
30 return false;
31 }
32 SkASSERT(tmp.fWidth >= 0);
33 fInfo = tmp;
34 }
35 *info = fInfo;
36 return true;
37 }
38
39 bool SkCachingPixelRef::configure(SkBitmap* bitmap) {
40 SkASSERT(bitmap != NULL);
41 SkImageInfo info;
42 if (!this->getInfo(&info)) {
43 return false;
44 }
45 return bitmap->setConfig(info, 0);
46 }
47
48 void* SkCachingPixelRef::onLockPixels(SkColorTable** colorTable) {
49 (void)colorTable;
50 SkImageInfo info;
51 if (!this->getInfo(&info)) {
52 return NULL;
53 }
54 SkBitmap bitmap;
55
56 fScaledCacheId = SkScaledImageCache::FindAndLock(this->getGenerationID(),
57 info.fWidth,
58 info.fHeight,
59 &bitmap);
60 if (NULL == fScaledCacheId) {
61 // Cache has been purged, must re-decode.
62 if (!this->onDecodeInto(0, &bitmap)) {
63 return NULL;
64 }
65 fScaledCacheId = SkScaledImageCache::AddAndLock(this->getGenerationID(),
66 info.fWidth,
67 info.fHeight,
68 bitmap);
69 SkASSERT(fScaledCacheId != NULL);
70 }
71 // Now bitmap should contain a concrete PixelRef of the decoded
72 // image.
73 SkAutoLockPixels autoLockPixels(bitmap);
74 void* pixels = bitmap.getPixels();
75 SkASSERT(pixels != NULL);
76 // At this point, the autoLockPixels will unlockPixels()
77 // to remove bitmap's lock on the pixels. We will then
78 // destroy bitmap. The *only* guarantee that this pointer
79 // remains valid is the guarantee made by
80 // SkScaledImageCache that it will not destroy the *other*
81 // bitmap (SkScaledImageCache::Rec.fBitmap) that holds a
82 // reference to the concrete PixelRef while this record is
83 // locked.
84 return pixels;
85 }
86
87 void SkCachingPixelRef::onUnlockPixels() {
88 if (fScaledCacheId != NULL) {
89 SkScaledImageCache::Unlock(
90 static_cast<SkScaledImageCache::ID*>(fScaledCacheId));
91 fScaledCacheId = NULL;
92 }
93 }
94
95 bool SkCachingPixelRef::onDecodeInto(int pow2, SkBitmap* bitmap) {
96 SkASSERT(bitmap != NULL);
97 SkBitmap tmp;
98 SkImageInfo info;
99 // TODO(halcanary) - Enable SkCachingPixelRef to use a custom
100 // allocator. `tmp.allocPixels(fAllocator, NULL)`
101 if (!(this->configure(&tmp) && tmp.allocPixels())) {
102 return false;
103 }
104 SkAssertResult(this->getInfo(&info)); // since configure() succeeded.
105 if (!this->onDecodePixels(info, tmp.getPixels(), tmp.rowBytes())) {
106 fErrorInDecoding = true;
107 return false;
108 }
109 *bitmap = tmp;
110 return true;
111 }
112
OLDNEW
« no previous file with comments | « src/lazy/SkCachingPixelRef.h ('k') | src/lazy/SkLazyCachingPixelRef.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698