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

Unified Diff: xfa/fwl/core/ifwl_combobox.cpp

Issue 2509283003: Unify the three CFWL_EventType::SelectChange classes. (Closed)
Patch Set: 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 side-by-side diff with in-line comments
Download patch
Index: xfa/fwl/core/ifwl_combobox.cpp
diff --git a/xfa/fwl/core/ifwl_combobox.cpp b/xfa/fwl/core/ifwl_combobox.cpp
index 9110b8ae8c7d92b1789640be7db9040053aa5f7f..985bc6f8fa3cf1afa81fcb3ee70457c4c21b9aaa 100644
--- a/xfa/fwl/core/ifwl_combobox.cpp
+++ b/xfa/fwl/core/ifwl_combobox.cpp
@@ -12,6 +12,7 @@
#include "xfa/fwl/core/cfwl_evteditchanged.h"
#include "xfa/fwl/core/cfwl_evtpostdropdown.h"
#include "xfa/fwl/core/cfwl_evtpredropdown.h"
+#include "xfa/fwl/core/cfwl_evtselectchanged.h"
#include "xfa/fwl/core/cfwl_evttextchanged.h"
#include "xfa/fwl/core/cfwl_msgkey.h"
#include "xfa/fwl/core/cfwl_msgkillfocus.h"
@@ -562,10 +563,9 @@ void IFWL_ComboBox::ProcessSelChanged(bool bLButtonUp) {
m_pEdit->SetSelected();
}
- CFWL_EvtCmbSelChanged ev;
+ CFWL_EvtSelectChanged ev;
ev.bLButtonUp = bLButtonUp;
ev.m_pSrcTarget = this;
- ev.iArraySels.Add(m_iCurSel);
npm 2016/11/17 18:07:02 Is iArraySels never used, or why is this deleted?
dsinclair 2016/11/17 18:13:19 Removed from class, compiler is happy.
DispatchEvent(&ev);
}

Powered by Google App Engine
This is Rietveld 408576698