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

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

Issue 2459423003: Rename IFWL_Widget::SetDelegate (Closed)
Patch Set: Review feedback 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_widget.h ('k') | xfa/fwl/core/cfwl_widgetmgr.cpp » ('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_pDelegate(nullptr),
24 m_pWidgetMgr(app->GetWidgetMgr()), 23 m_pWidgetMgr(app->GetWidgetMgr()),
25 m_pProperties(new CFWL_WidgetProperties) { 24 m_pProperties(new CFWL_WidgetProperties) {
26 ASSERT(m_pWidgetMgr); 25 ASSERT(m_pWidgetMgr);
27 } 26 }
28 27
29 CFWL_Widget::~CFWL_Widget() { 28 CFWL_Widget::~CFWL_Widget() {
30 if (m_pIface) 29 if (m_pIface)
31 m_pIface->Finalize(); 30 m_pIface->Finalize();
32 } 31 }
33 32
(...skipping 90 matching lines...) Expand 10 before | Expand all | Expand 10 after
124 return m_pIface->HitTest(fx, fy); 123 return m_pIface->HitTest(fx, fy);
125 } 124 }
126 125
127 FWL_Error CFWL_Widget::DrawWidget(CFX_Graphics* pGraphics, 126 FWL_Error CFWL_Widget::DrawWidget(CFX_Graphics* pGraphics,
128 const CFX_Matrix* pMatrix) { 127 const CFX_Matrix* pMatrix) {
129 if (!m_pIface) 128 if (!m_pIface)
130 return FWL_Error::Indefinite; 129 return FWL_Error::Indefinite;
131 return m_pIface->DrawWidget(pGraphics, pMatrix); 130 return m_pIface->DrawWidget(pGraphics, pMatrix);
132 } 131 }
133 132
134 IFWL_WidgetDelegate* CFWL_Widget::SetDelegate(IFWL_WidgetDelegate* pDelegate) { 133 IFWL_WidgetDelegate* CFWL_Widget::GetCurrentDelegate() {
135 if (!m_pIface) 134 return m_pIface ? m_pIface->GetCurrentDelegate() : nullptr;
136 return nullptr; 135 }
137 136
138 m_pDelegate = m_pIface->SetDelegate(pDelegate); 137 void CFWL_Widget::SetCurrentDelegate(IFWL_WidgetDelegate* pDelegate) {
139 return m_pDelegate; 138 if (m_pIface)
139 m_pIface->SetCurrentDelegate(pDelegate);
140 } 140 }
OLDNEW
« no previous file with comments | « xfa/fwl/core/cfwl_widget.h ('k') | xfa/fwl/core/cfwl_widgetmgr.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698