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

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

Issue 2435603003: Cleanup unused methods and return values in FWL code. (Closed)
Patch Set: Fix typo Created 4 years, 2 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « xfa/fwl/core/ifwl_datetimepicker.h ('k') | xfa/fwl/core/ifwl_edit.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: xfa/fwl/core/ifwl_datetimepicker.cpp
diff --git a/xfa/fwl/core/ifwl_datetimepicker.cpp b/xfa/fwl/core/ifwl_datetimepicker.cpp
index 04290b62d15c7ee2409136d16c48ad6316d63e62..ef13d71cf1a6fbb0e348817551a44ae2efaba324 100644
--- a/xfa/fwl/core/ifwl_datetimepicker.cpp
+++ b/xfa/fwl/core/ifwl_datetimepicker.cpp
@@ -36,11 +36,6 @@ IFWL_DateTimePicker::IFWL_DateTimePicker(
IFWL_DateTimePicker::~IFWL_DateTimePicker() {}
-FWL_Error IFWL_DateTimePicker::GetClassName(CFX_WideString& wsClass) const {
- wsClass = FWL_CLASS_DateTimePicker;
- return FWL_Error::Succeeded;
-}
-
FWL_Type IFWL_DateTimePicker::GetClassID() const {
return FWL_Type::DateTimePicker;
}
@@ -75,20 +70,18 @@ FWL_Error IFWL_DateTimePicker::Initialize() {
return FWL_Error::Succeeded;
}
-FWL_Error IFWL_DateTimePicker::Finalize() {
- if (m_pEdit) {
+void IFWL_DateTimePicker::Finalize() {
+ if (m_pEdit)
m_pEdit->Finalize();
- }
- if (m_pMonthCal) {
+ if (m_pMonthCal)
m_pMonthCal->Finalize();
- }
- if (m_pForm) {
+ if (m_pForm)
m_pForm->Finalize();
- }
+
UnregisterEventTarget();
delete m_pDelegate;
m_pDelegate = nullptr;
- return IFWL_Widget::Finalize();
+ IFWL_Widget::Finalize();
}
FWL_Error IFWL_DateTimePicker::GetWidgetRect(CFX_RectF& rect,
« no previous file with comments | « xfa/fwl/core/ifwl_datetimepicker.h ('k') | xfa/fwl/core/ifwl_edit.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698