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

Side by Side Diff: include/codec/SkCodec.h

Issue 1212593003: Destroy SkScanlineDecoder in the SkCodec subclass destructors (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: Created 5 years, 5 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 | « no previous file | include/codec/SkScanlineDecoder.h » ('j') | src/codec/SkJpegCodec.cpp » ('J')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* 1 /*
2 * Copyright 2015 Google Inc. 2 * Copyright 2015 Google Inc.
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 #ifndef SkCodec_DEFINED 8 #ifndef SkCodec_DEFINED
9 #define SkCodec_DEFINED 9 #define SkCodec_DEFINED
10 10
(...skipping 159 matching lines...) Expand 10 before | Expand all | Expand 10 after
170 170
171 /* 171 /*
172 * 172 *
173 * Get method for the input stream 173 * Get method for the input stream
174 * 174 *
175 */ 175 */
176 SkStream* stream() { 176 SkStream* stream() {
177 return fStream.get(); 177 return fStream.get();
178 } 178 }
179 179
180 /*
181 *
182 * Get method for the scanline decoder
reed1 2015/06/29 21:21:49 nit: lets have the comment be slightly more useful
msarett 2015/06/29 21:59:43 Done.
183 *
184 */
185 SkScanlineDecoder* scanlineDecoder() {
186 return fScanlineDecoder.get();
187 }
188
180 private: 189 private:
181 SkAutoTDelete<SkStream> fStream; 190 SkAutoTDelete<SkStream> fStream;
182 bool fNeedsRewind; 191 bool fNeedsRewind;
183 SkAutoTDelete<SkScanlineDecoder> fScanlineDecoder; 192 SkAutoTDelete<SkScanlineDecoder> fScanlineDecoder;
184 193
185 typedef SkImageGenerator INHERITED; 194 typedef SkImageGenerator INHERITED;
186 }; 195 };
187 #endif // SkCodec_DEFINED 196 #endif // SkCodec_DEFINED
OLDNEW
« no previous file with comments | « no previous file | include/codec/SkScanlineDecoder.h » ('j') | src/codec/SkJpegCodec.cpp » ('J')

Powered by Google App Engine
This is Rietveld 408576698