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

Side by Side Diff: core/fpdfapi/page/cpdf_image.h

Issue 2420743002: Make CPDF_Image() constructors saner. (Closed)
Patch Set: Fix leak Created 4 years, 2 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 // Copyright 2016 PDFium Authors. All rights reserved. 1 // Copyright 2016 PDFium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 // Original code copyright 2014 Foxit Software Inc. http://www.foxitsoftware.com 5 // Original code copyright 2014 Foxit Software Inc. http://www.foxitsoftware.com
6 6
7 #ifndef CORE_FPDFAPI_PAGE_CPDF_IMAGE_H_ 7 #ifndef CORE_FPDFAPI_PAGE_CPDF_IMAGE_H_
8 #define CORE_FPDFAPI_PAGE_CPDF_IMAGE_H_ 8 #define CORE_FPDFAPI_PAGE_CPDF_IMAGE_H_
9 9
10 #include "core/fpdfapi/parser/cpdf_stream.h" 10 #include "core/fpdfapi/parser/cpdf_stream.h"
(...skipping 10 matching lines...) Expand all
21 class CPDF_Dictionay; 21 class CPDF_Dictionay;
22 class CPDF_Document; 22 class CPDF_Document;
23 class CPDF_Page; 23 class CPDF_Page;
24 class IFX_FileRead; 24 class IFX_FileRead;
25 class IFX_FileWrite; 25 class IFX_FileWrite;
26 class IFX_Pause; 26 class IFX_Pause;
27 27
28 class CPDF_Image { 28 class CPDF_Image {
29 public: 29 public:
30 explicit CPDF_Image(CPDF_Document* pDoc); 30 explicit CPDF_Image(CPDF_Document* pDoc);
31 CPDF_Image(CPDF_Document* pDoc, CPDF_Stream* pStream, bool bInline); 31 CPDF_Image(CPDF_Document* pDoc, UniqueStream pStream);
32 CPDF_Image(CPDF_Document* pDoc, uint32_t dwStreamObjNum);
32 ~CPDF_Image(); 33 ~CPDF_Image();
33 34
34 CPDF_Image* Clone(); 35 CPDF_Image* Clone();
35 36
36 CPDF_Dictionary* GetInlineDict() const { return m_pInlineDict; } 37 CPDF_Dictionary* GetInlineDict() const { return m_pDict; }
37 CPDF_Stream* GetStream() const { return m_pStream; } 38 CPDF_Stream* GetStream() const { return m_pStream; }
38 CPDF_Dictionary* GetDict() const { 39 CPDF_Dictionary* GetDict() const {
39 return m_pStream ? m_pStream->GetDict() : nullptr; 40 return m_pStream ? m_pStream->GetDict() : nullptr;
40 } 41 }
41 CPDF_Dictionary* GetOC() const { return m_pOC; } 42 CPDF_Dictionary* GetOC() const { return m_pOC; }
42 CPDF_Document* GetDocument() const { return m_pDocument; } 43 CPDF_Document* GetDocument() const { return m_pDocument; }
43 44
44 int32_t GetPixelHeight() const { return m_Height; } 45 int32_t GetPixelHeight() const { return m_Height; }
45 int32_t GetPixelWidth() const { return m_Width; } 46 int32_t GetPixelWidth() const { return m_Width; }
46 47
47 bool IsInline() const { return m_bInline; } 48 bool IsInline() const { return !!m_pOwnedStream; }
48 bool IsMask() const { return m_bIsMask; } 49 bool IsMask() const { return m_bIsMask; }
49 bool IsInterpol() const { return m_bInterpolate; } 50 bool IsInterpol() const { return m_bInterpolate; }
50 51
51 CFX_DIBSource* LoadDIBSource(CFX_DIBSource** ppMask = nullptr, 52 CFX_DIBSource* LoadDIBSource(CFX_DIBSource** ppMask = nullptr,
52 uint32_t* pMatteColor = nullptr, 53 uint32_t* pMatteColor = nullptr,
53 FX_BOOL bStdCS = FALSE, 54 FX_BOOL bStdCS = FALSE,
54 uint32_t GroupFamily = 0, 55 uint32_t GroupFamily = 0,
55 FX_BOOL bLoadMask = FALSE) const; 56 FX_BOOL bLoadMask = FALSE) const;
56 57
57 void SetInlineDict(CPDF_Dictionary* pDict) { m_pInlineDict = pDict; }
58 void SetImage(const CFX_DIBitmap* pDIBitmap, int32_t iCompress); 58 void SetImage(const CFX_DIBitmap* pDIBitmap, int32_t iCompress);
59 void SetJpegImage(IFX_FileRead* pFile); 59 void SetJpegImage(IFX_FileRead* pFile);
60 60
61 void ResetCache(CPDF_Page* pPage, const CFX_DIBitmap* pDIBitmap); 61 void ResetCache(CPDF_Page* pPage, const CFX_DIBitmap* pDIBitmap);
62 62
63 FX_BOOL StartLoadDIBSource(CPDF_Dictionary* pFormResource, 63 FX_BOOL StartLoadDIBSource(CPDF_Dictionary* pFormResource,
64 CPDF_Dictionary* pPageResource, 64 CPDF_Dictionary* pPageResource,
65 FX_BOOL bStdCS = FALSE, 65 FX_BOOL bStdCS = FALSE,
66 uint32_t GroupFamily = 0, 66 uint32_t GroupFamily = 0,
67 FX_BOOL bLoadMask = FALSE); 67 FX_BOOL bLoadMask = FALSE);
68 FX_BOOL Continue(IFX_Pause* pPause); 68 FX_BOOL Continue(IFX_Pause* pPause);
69 CFX_DIBSource* DetachBitmap(); 69 CFX_DIBSource* DetachBitmap();
70 CFX_DIBSource* DetachMask(); 70 CFX_DIBSource* DetachMask();
71 71
72 CFX_DIBSource* m_pDIBSource; 72 CFX_DIBSource* m_pDIBSource = nullptr;
73 CFX_DIBSource* m_pMask; 73 CFX_DIBSource* m_pMask = nullptr;
74 uint32_t m_MatteColor; 74 uint32_t m_MatteColor = 0;
75 75
76 private: 76 private:
77 void FinishInitialization();
77 CPDF_Dictionary* InitJPEG(uint8_t* pData, uint32_t size); 78 CPDF_Dictionary* InitJPEG(uint8_t* pData, uint32_t size);
78 79
79 CPDF_Stream* m_pStream; 80 int32_t m_Height = 0;
80 const bool m_bInline; 81 int32_t m_Width = 0;
81 CPDF_Dictionary* m_pInlineDict; 82 bool m_bIsMask = false;
82 int32_t m_Height; 83 bool m_bInterpolate = false;
83 int32_t m_Width;
84 bool m_bIsMask;
85 bool m_bInterpolate;
86 CPDF_Document* const m_pDocument; 84 CPDF_Document* const m_pDocument;
87 CPDF_Dictionary* m_pOC; 85 CPDF_Stream* m_pStream = nullptr;
86 CPDF_Dictionary* m_pDict = nullptr;
87 UniqueStream m_pOwnedStream;
88 UniqueDictionary m_pOwnedDict;
89 CPDF_Dictionary* m_pOC = nullptr;
88 }; 90 };
89 91
90 #endif // CORE_FPDFAPI_PAGE_CPDF_IMAGE_H_ 92 #endif // CORE_FPDFAPI_PAGE_CPDF_IMAGE_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698