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

Side by Side Diff: xfa/fwl/core/cfwl_widget.cpp

Issue 2464703006: Fold IFWL*::{Initialize|Finalize} into constructor/destructor (Closed)
Patch Set: Rebase to master Created 4 years, 1 month 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/fwl/core/cfwl_pushbutton.cpp ('k') | xfa/fwl/core/fwl_noteimp.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/fwl/core/cfwl_widget.h" 7 #include "xfa/fwl/core/cfwl_widget.h"
8 8
9 #include "xfa/fde/tto/fde_textout.h" 9 #include "xfa/fde/tto/fde_textout.h"
10 #include "xfa/fwl/core/cfwl_themetext.h" 10 #include "xfa/fwl/core/cfwl_themetext.h"
11 #include "xfa/fwl/core/cfwl_widgetmgr.h" 11 #include "xfa/fwl/core/cfwl_widgetmgr.h"
12 #include "xfa/fwl/core/fwl_noteimp.h" 12 #include "xfa/fwl/core/fwl_noteimp.h"
13 #include "xfa/fwl/core/fwl_noteimp.h" 13 #include "xfa/fwl/core/fwl_noteimp.h"
14 #include "xfa/fwl/core/ifwl_app.h" 14 #include "xfa/fwl/core/ifwl_app.h"
15 #include "xfa/fwl/core/ifwl_themeprovider.h" 15 #include "xfa/fwl/core/ifwl_themeprovider.h"
16 16
17 #define FWL_WGT_CalcHeight 2048 17 #define FWL_WGT_CalcHeight 2048
18 #define FWL_WGT_CalcWidth 2048 18 #define FWL_WGT_CalcWidth 2048
19 #define FWL_WGT_CalcMultiLineDefWidth 120.0f 19 #define FWL_WGT_CalcMultiLineDefWidth 120.0f
20 20
21 CFWL_Widget::CFWL_Widget(const IFWL_App* app) 21 CFWL_Widget::CFWL_Widget(const IFWL_App* app)
22 : m_pApp(app), 22 : m_pApp(app),
23 m_pWidgetMgr(app->GetWidgetMgr()), 23 m_pWidgetMgr(app->GetWidgetMgr()),
24 m_pProperties(new CFWL_WidgetProperties) { 24 m_pProperties(new CFWL_WidgetProperties) {
25 ASSERT(m_pWidgetMgr); 25 ASSERT(m_pWidgetMgr);
26 } 26 }
27 27
28 CFWL_Widget::~CFWL_Widget() { 28 CFWL_Widget::~CFWL_Widget() {}
29 if (m_pIface)
30 m_pIface->Finalize();
31 }
32 29
33 void CFWL_Widget::Initialize() { 30 void CFWL_Widget::Initialize() {
34 ASSERT(m_pIface); 31 ASSERT(m_pIface);
35 m_pIface->SetAssociateWidget(this); 32 m_pIface->SetAssociateWidget(this);
36 } 33 }
37 34
38 IFWL_Widget* CFWL_Widget::GetWidget() { 35 IFWL_Widget* CFWL_Widget::GetWidget() {
39 return m_pIface.get(); 36 return m_pIface.get();
40 } 37 }
41 38
(...skipping 89 matching lines...) Expand 10 before | Expand all | Expand 10 after
131 } 128 }
132 129
133 IFWL_WidgetDelegate* CFWL_Widget::GetCurrentDelegate() { 130 IFWL_WidgetDelegate* CFWL_Widget::GetCurrentDelegate() {
134 return m_pIface ? m_pIface->GetCurrentDelegate() : nullptr; 131 return m_pIface ? m_pIface->GetCurrentDelegate() : nullptr;
135 } 132 }
136 133
137 void CFWL_Widget::SetCurrentDelegate(IFWL_WidgetDelegate* pDelegate) { 134 void CFWL_Widget::SetCurrentDelegate(IFWL_WidgetDelegate* pDelegate) {
138 if (m_pIface) 135 if (m_pIface)
139 m_pIface->SetCurrentDelegate(pDelegate); 136 m_pIface->SetCurrentDelegate(pDelegate);
140 } 137 }
OLDNEW
« no previous file with comments | « xfa/fwl/core/cfwl_pushbutton.cpp ('k') | xfa/fwl/core/fwl_noteimp.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698