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

Unified Diff: core/src/fxcodec/codec/codec_int.h

Issue 1305223010: Merge to XFA: Cleanup dead code in CPDF_DIBSource::LoadJpxBitmap() and friends. (Closed) Base URL: https://pdfium.googlesource.com/pdfium.git@xfa
Patch Set: Created 5 years, 3 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 side-by-side diff with in-line comments
Download patch
Index: core/src/fxcodec/codec/codec_int.h
diff --git a/core/src/fxcodec/codec/codec_int.h b/core/src/fxcodec/codec/codec_int.h
index d172d7157e3664f0a7d1ada078d123c532975e34..75e1fd141bbb27739466b57ef1834917e204147b 100644
--- a/core/src/fxcodec/codec/codec_int.h
+++ b/core/src/fxcodec/codec/codec_int.h
@@ -311,20 +311,21 @@ class CCodec_IccModule : public ICodec_IccModule {
class CCodec_JpxModule : public ICodec_JpxModule {
public:
CCodec_JpxModule();
- void* CreateDecoder(const uint8_t* src_buf,
- FX_DWORD src_size,
- FX_BOOL useColorSpace = FALSE);
- void GetImageInfo(void* ctx,
- FX_DWORD& width,
- FX_DWORD& height,
- FX_DWORD& codestream_nComps,
- FX_DWORD& output_nComps);
- FX_BOOL Decode(void* ctx,
+ ~CCodec_JpxModule() override;
+
+ // ICodec_JpxModule:
+ CJPX_Decoder* CreateDecoder(const uint8_t* src_buf,
+ FX_DWORD src_size,
+ bool use_colorspace) override;
+ void GetImageInfo(CJPX_Decoder* pDecoder,
+ FX_DWORD* width,
+ FX_DWORD* height,
+ FX_DWORD* components) override;
+ FX_BOOL Decode(CJPX_Decoder* pDecoder,
uint8_t* dest_data,
int pitch,
- FX_BOOL bTranslateColor,
- uint8_t* offsets);
- void DestroyDecoder(void* ctx);
+ uint8_t* offsets) override;
+ void DestroyDecoder(CJPX_Decoder* pDecoder) override;
};
class CCodec_TiffModule : public ICodec_TiffModule {

Powered by Google App Engine
This is Rietveld 408576698