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

Side by Side Diff: xfa/fwl/cfwl_combobox.cpp

Issue 2563693003: Remove unused list data pointer (Closed)
Patch Set: Created 4 years 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 | « no previous file | xfa/fwl/cfwl_listbox.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/cfwl_combobox.h" 7 #include "xfa/fwl/cfwl_combobox.h"
8 8
9 #include <algorithm> 9 #include <algorithm>
10 #include <memory> 10 #include <memory>
(...skipping 144 matching lines...) Expand 10 before | Expand all | Expand 10 after
155 rtTextBk.width -= m_rtBtn.width; 155 rtTextBk.width -= m_rtBtn.width;
156 156
157 CFWL_ThemeBackground param; 157 CFWL_ThemeBackground param;
158 param.m_pWidget = this; 158 param.m_pWidget = this;
159 param.m_iPart = CFWL_Part::Background; 159 param.m_iPart = CFWL_Part::Background;
160 param.m_pGraphics = pGraphics; 160 param.m_pGraphics = pGraphics;
161 if (pMatrix) 161 if (pMatrix)
162 param.m_matrix.Concat(*pMatrix); 162 param.m_matrix.Concat(*pMatrix);
163 param.m_rtPart = rtTextBk; 163 param.m_rtPart = rtTextBk;
164 164
165 if (m_iCurSel >= 0) {
166 if (void* p = m_pListBox->GetItemData(
167 m_pListBox.get(),
168 m_pListBox->GetItem(m_pListBox.get(), m_iCurSel))) {
169 param.m_pData = p;
170 }
171 }
172
173 if (m_pProperties->m_dwStates & FWL_WGTSTATE_Disabled) { 165 if (m_pProperties->m_dwStates & FWL_WGTSTATE_Disabled) {
174 param.m_dwStates = CFWL_PartState_Disabled; 166 param.m_dwStates = CFWL_PartState_Disabled;
175 } else if ((m_pProperties->m_dwStates & FWL_WGTSTATE_Focused) && 167 } else if ((m_pProperties->m_dwStates & FWL_WGTSTATE_Focused) &&
176 (m_iCurSel >= 0)) { 168 (m_iCurSel >= 0)) {
177 param.m_dwStates = CFWL_PartState_Selected; 169 param.m_dwStates = CFWL_PartState_Selected;
178 } else { 170 } else {
179 param.m_dwStates = CFWL_PartState_Normal; 171 param.m_dwStates = CFWL_PartState_Normal;
180 } 172 }
181 pTheme->DrawBackground(&param); 173 pTheme->DrawBackground(&param);
182 174
(...skipping 886 matching lines...) Expand 10 before | Expand all | Expand 10 after
1069 else 1061 else
1070 iCurSel++; 1062 iCurSel++;
1071 } 1063 }
1072 m_iCurSel = iCurSel; 1064 m_iCurSel = iCurSel;
1073 SyncEditText(m_iCurSel); 1065 SyncEditText(m_iCurSel);
1074 return; 1066 return;
1075 } 1067 }
1076 if (m_pEdit) 1068 if (m_pEdit)
1077 m_pEdit->GetDelegate()->OnProcessMessage(pMsg); 1069 m_pEdit->GetDelegate()->OnProcessMessage(pMsg);
1078 } 1070 }
OLDNEW
« no previous file with comments | « no previous file | xfa/fwl/cfwl_listbox.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698