From db9faec3c8fb3ced3d8340b2b6ae252b8f40d135 Mon Sep 17 00:00:00 2001 From: Tom Sepez Date: Wed, 31 May 2017 16:29:29 -0700 Subject: Put all CCodec_PngModule state into its context. Rename FXPNG_Context and use unowned ptr. Clean up context in its dtor. Then create and destroy using |new|. Change-Id: I7b66e6d0da50a16d3b8d5108ffd931bb01145892 Reviewed-on: https://pdfium-review.googlesource.com/6152 Commit-Queue: Tom Sepez Reviewed-by: Lei Zhang --- core/fxcodec/codec/ccodec_progressivedecoder.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'core/fxcodec/codec/ccodec_progressivedecoder.h') diff --git a/core/fxcodec/codec/ccodec_progressivedecoder.h b/core/fxcodec/codec/ccodec_progressivedecoder.h index f9a67ccee3..5ebb32397d 100644 --- a/core/fxcodec/codec/ccodec_progressivedecoder.h +++ b/core/fxcodec/codec/ccodec_progressivedecoder.h @@ -132,7 +132,7 @@ class CCodec_ProgressiveDecoder : public CCodec_BmpModule::Delegate, // TODO(tsepez): All these contexts probably should be unique_ptrs. FXJPEG_Context* m_pJpegContext; - FXPNG_Context* m_pPngContext; + CFX_UnownedPtr m_pPngContext; std::unique_ptr m_pGifContext; CCodec_BmpModule::Context* m_pBmpContext; CFX_UnownedPtr m_pTiffContext; -- cgit v1.2.3