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

Side by Side Diff: xfa/fde/fde_gedevice.cpp

Issue 2208423002: Use smart pointers for class owned pointers under xfa/fde (Closed) Base URL: https://pdfium.googlesource.com/pdfium.git@master
Patch Set: one more change Created 4 years, 4 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 | « xfa/fde/fde_gedevice.h ('k') | xfa/fde/fde_render.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2014 PDFium Authors. All rights reserved. 1 // Copyright 2014 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 #include "xfa/fde/fde_gedevice.h" 7 #include "xfa/fde/fde_gedevice.h"
8 8
9 #include <algorithm> 9 #include <algorithm>
10 10
11 #include "xfa/fde/cfde_path.h" 11 #include "xfa/fde/cfde_path.h"
12 #include "xfa/fde/fde_object.h" 12 #include "xfa/fde/fde_object.h"
13 #include "xfa/fgas/font/fgas_font.h" 13 #include "xfa/fgas/font/fgas_font.h"
14 #include "xfa/fgas/font/fgas_gefont.h" 14 #include "xfa/fgas/font/fgas_gefont.h"
15 15
16 CFDE_RenderDevice::CFDE_RenderDevice(CFX_RenderDevice* pDevice, 16 CFDE_RenderDevice::CFDE_RenderDevice(CFX_RenderDevice* pDevice,
17 FX_BOOL bOwnerDevice) 17 FX_BOOL bOwnerDevice)
18 : m_pDevice(pDevice), 18 : m_pDevice(pDevice),
19 m_bOwnerDevice(bOwnerDevice), 19 m_bOwnerDevice(bOwnerDevice),
20 m_pCharPos(nullptr),
21 m_iCharCount(0) { 20 m_iCharCount(0) {
22 ASSERT(pDevice); 21 ASSERT(pDevice);
23 22
24 FX_RECT rt = m_pDevice->GetClipBox(); 23 FX_RECT rt = m_pDevice->GetClipBox();
25 m_rtClip.Set((FX_FLOAT)rt.left, (FX_FLOAT)rt.top, (FX_FLOAT)rt.Width(), 24 m_rtClip.Set((FX_FLOAT)rt.left, (FX_FLOAT)rt.top, (FX_FLOAT)rt.Width(),
26 (FX_FLOAT)rt.Height()); 25 (FX_FLOAT)rt.Height());
27 } 26 }
28 27
29 CFDE_RenderDevice::~CFDE_RenderDevice() { 28 CFDE_RenderDevice::~CFDE_RenderDevice() {
30 FX_Free(m_pCharPos);
31 if (m_bOwnerDevice) 29 if (m_bOwnerDevice)
32 delete m_pDevice; 30 delete m_pDevice;
33 } 31 }
34 int32_t CFDE_RenderDevice::GetWidth() const { 32 int32_t CFDE_RenderDevice::GetWidth() const {
35 return m_pDevice->GetWidth(); 33 return m_pDevice->GetWidth();
36 } 34 }
37 int32_t CFDE_RenderDevice::GetHeight() const { 35 int32_t CFDE_RenderDevice::GetHeight() const {
38 return m_pDevice->GetHeight(); 36 return m_pDevice->GetHeight();
39 } 37 }
40 void CFDE_RenderDevice::SaveState() { 38 void CFDE_RenderDevice::SaveState() {
(...skipping 280 matching lines...) Expand 10 before | Expand all | Expand 10 after
321 const CFX_Matrix* pMatrix) { 319 const CFX_Matrix* pMatrix) {
322 CFDE_Path* pGePath = (CFDE_Path*)pPath; 320 CFDE_Path* pGePath = (CFDE_Path*)pPath;
323 if (!pGePath) 321 if (!pGePath)
324 return FALSE; 322 return FALSE;
325 if (!pBrush) 323 if (!pBrush)
326 return FALSE; 324 return FALSE;
327 return m_pDevice->DrawPath(&pGePath->m_Path, pMatrix, nullptr, 325 return m_pDevice->DrawPath(&pGePath->m_Path, pMatrix, nullptr,
328 pBrush->GetColor(), 0, FXFILL_WINDING); 326 pBrush->GetColor(), 0, FXFILL_WINDING);
329 } 327 }
330 328
OLDNEW
« no previous file with comments | « xfa/fde/fde_gedevice.h ('k') | xfa/fde/fde_render.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698