FWL refcounts never incremented (part 1)

BUG=pdfium:282
R=thestig@chromium.org

Review URL: https://codereview.chromium.org/1509203002 .

12 files changed, 16 insertions, 53 deletions

diff --git a/xfa/include/fwl/basewidget/fwl_barcode.h b/xfa/include/fwl/basewidget/fwl_barcode.h
index ac54ab8d39..28b40baad1 100644
--- a/xfa/include/fwl/basewidget/fwl_barcode.h
+++ b/xfa/include/fwl/basewidget/fwl_barcode.h
@@ -49,13 +49,10 @@ class IFWL_BarcodeDP : public IFWL_EditDP {
 };

 class IFWL_Barcode : public IFWL_Edit {

  public:

-  static IFWL_Barcode* Create();

+  IFWL_Barcode();

   FWL_ERR Initialize(const CFWL_WidgetImpProperties& properties,

                      IFWL_Widget* pOuter);

   void SetType(BC_TYPE type);

   FX_BOOL IsProtectedType();

-

- protected:

-  IFWL_Barcode();

 };

 #endif

diff --git a/xfa/include/fwl/basewidget/fwl_caret.h b/xfa/include/fwl/basewidget/fwl_caret.h
index 547f1c9ba8..d2e8a807ce 100644
--- a/xfa/include/fwl/basewidget/fwl_caret.h
+++ b/xfa/include/fwl/basewidget/fwl_caret.h
@@ -16,7 +16,7 @@ class IFWL_Caret;
 #define FWL_PARTSTATE_CAT_HightLight 1

 class IFWL_Caret : public IFWL_Widget {

  public:

-  static IFWL_Caret* Create();

+  IFWL_Caret();

 

   FWL_ERR Initialize(IFWL_Widget* pOuter = NULL);

   FWL_ERR Initialize(const CFWL_WidgetImpProperties& properties,

@@ -26,8 +26,5 @@ class IFWL_Caret : public IFWL_Widget {
   FWL_ERR GetFrequency(FX_DWORD& elapse);

   FWL_ERR SetFrequency(FX_DWORD elapse);

   FWL_ERR SetColor(CFX_Color crFill);

-

- protected:

-  IFWL_Caret();

 };

 #endif

diff --git a/xfa/include/fwl/basewidget/fwl_checkbox.h b/xfa/include/fwl/basewidget/fwl_checkbox.h
index 989430acc1..3e08a63972 100644
--- a/xfa/include/fwl/basewidget/fwl_checkbox.h
+++ b/xfa/include/fwl/basewidget/fwl_checkbox.h
@@ -70,24 +70,19 @@ class IFWL_CheckBoxDP : public IFWL_DataProvider {
 };

 class IFWL_CheckBox : public IFWL_Widget {

  public:

-  static IFWL_CheckBox* Create();

+  IFWL_CheckBox();

   FWL_ERR Initialize(const CFWL_WidgetImpProperties& properties,

                      IFWL_Widget* pOuter);

   int32_t GetCheckState();

   FWL_ERR SetCheckState(int32_t iCheck);

 

- protected:

-  IFWL_CheckBox();

 };

 class IFWL_RadioButton : public IFWL_Widget {

  public:

-  static IFWL_RadioButton* Create();

+  IFWL_RadioButton();

   FWL_ERR Initialize(const CFWL_WidgetImpProperties& properties,

                      IFWL_Widget* pOuter);

   int32_t GetCheckState();

   FWL_ERR SetCheckState(int32_t iCheck);

-

- protected:

-  IFWL_RadioButton();

 };

 #endif

diff --git a/xfa/include/fwl/basewidget/fwl_combobox.h b/xfa/include/fwl/basewidget/fwl_combobox.h
index c3311e5200..184d0107a4 100644
--- a/xfa/include/fwl/basewidget/fwl_combobox.h
+++ b/xfa/include/fwl/basewidget/fwl_combobox.h
@@ -97,7 +97,7 @@ class IFWL_ComboBoxDP : public IFWL_ListBoxDP {
 };

 class IFWL_ComboBox : public IFWL_Widget {

  public:

-  static IFWL_ComboBox* Create();

+  IFWL_ComboBox();

   FWL_ERR Initialize(const CFWL_WidgetImpProperties& properties,

                      IFWL_Widget* pOuter);

   int32_t GetCurSel();

@@ -133,8 +133,5 @@ class IFWL_ComboBox : public IFWL_Widget {
   FWL_ERR GetBBox(CFX_RectF& rect);

   FWL_ERR EditModifyStylesEx(FX_DWORD dwStylesExAdded,

                              FX_DWORD dwStylesExRemoved);

-

- protected:

-  IFWL_ComboBox();

 };

 #endif

diff --git a/xfa/include/fwl/basewidget/fwl_datetimepicker.h b/xfa/include/fwl/basewidget/fwl_datetimepicker.h
index f311185d1d..1f3750aaca 100644
--- a/xfa/include/fwl/basewidget/fwl_datetimepicker.h
+++ b/xfa/include/fwl/basewidget/fwl_datetimepicker.h
@@ -70,7 +70,7 @@ class IFWL_DateTimePickerDP : public IFWL_DataProvider {
 };

 class IFWL_DateTimePicker : public IFWL_Widget {

  public:

-  static IFWL_DateTimePicker* Create();

+  IFWL_DateTimePicker();

   FWL_ERR Initialize(const CFWL_WidgetImpProperties& properties,

                      IFWL_Widget* pOuter);

   FWL_ERR GetCurSel(int32_t& iYear, int32_t& iMonth, int32_t& iDay);

@@ -98,8 +98,5 @@ class IFWL_DateTimePicker : public IFWL_Widget {
   FWL_ERR SetEditLimit(int32_t nLimit);

   FWL_ERR ModifyEditStylesEx(FX_DWORD dwStylesExAdded,

                              FX_DWORD dwStylesExRemoved);

-

- protected:

-  IFWL_DateTimePicker();

 };

 #endif

diff --git a/xfa/include/fwl/basewidget/fwl_edit.h b/xfa/include/fwl/basewidget/fwl_edit.h
index c804af7ae1..207e6ef614 100644
--- a/xfa/include/fwl/basewidget/fwl_edit.h
+++ b/xfa/include/fwl/basewidget/fwl_edit.h
@@ -112,7 +112,8 @@ class IFWL_EditDP : public IFWL_DataProvider {};
 typedef struct _FWL_HEDTFIND { void* pData; } * FWL_HEDTFIND;

 class IFWL_Edit : public IFWL_Widget {

  public:

-  static IFWL_Edit* Create();

+  IFWL_Edit();

+

   FWL_ERR Initialize(const CFWL_WidgetImpProperties& properties,

                      IFWL_Widget* pOuter);

   FWL_ERR SetText(const CFX_WideString& wsText);

@@ -158,8 +159,5 @@ class IFWL_Edit : public IFWL_Widget {
   FX_BOOL GetSuggestWords(CFX_PointF pointf, CFX_ByteStringArray& sSuggest);

   FX_BOOL ReplaceSpellCheckWord(CFX_PointF pointf,

                                 const CFX_ByteStringC& bsReplace);

-

- protected:

-  IFWL_Edit();

 };

 #endif

diff --git a/xfa/include/fwl/basewidget/fwl_listbox.h b/xfa/include/fwl/basewidget/fwl_listbox.h
index 569e00ea0c..cc2fe91feb 100644
--- a/xfa/include/fwl/basewidget/fwl_listbox.h
+++ b/xfa/include/fwl/basewidget/fwl_listbox.h
@@ -109,7 +109,8 @@ class IFWL_ListBoxCompare {
 };

 class IFWL_ListBox : public IFWL_Widget {

  public:

-  static IFWL_ListBox* Create();

+  IFWL_ListBox();

+

   FWL_ERR Initialize(const CFWL_WidgetImpProperties& properties,

                      IFWL_Widget* pOuter);

   int32_t CountSelItems();

@@ -119,8 +120,5 @@ class IFWL_ListBox : public IFWL_Widget {
   FWL_ERR GetItemText(FWL_HLISTITEM hItem, CFX_WideString& wsText);

   FWL_ERR GetScrollPos(FX_FLOAT& fPos, FX_BOOL bVert = TRUE);

   FWL_ERR* Sort(IFWL_ListBoxCompare* pCom);

-

- protected:

-  IFWL_ListBox();

 };

 #endif

diff --git a/xfa/include/fwl/basewidget/fwl_picturebox.h b/xfa/include/fwl/basewidget/fwl_picturebox.h
index 7ca8f526d7..b55d4c30d9 100644
--- a/xfa/include/fwl/basewidget/fwl_picturebox.h
+++ b/xfa/include/fwl/basewidget/fwl_picturebox.h
@@ -42,12 +42,8 @@ class IFWL_PictureBoxDP : public IFWL_DataProvider {
 };

 class IFWL_PictureBox : public IFWL_Widget {

  public:

-  static IFWL_PictureBox* Create();

-

+  IFWL_PictureBox();

   FWL_ERR Initialize(const CFWL_WidgetImpProperties& properties,

                      IFWL_Widget* pOuter);

-

- protected:

-  IFWL_PictureBox();

 };

 #endif

diff --git a/xfa/include/fwl/basewidget/fwl_pushbutton.h b/xfa/include/fwl/basewidget/fwl_pushbutton.h
index 55a650e23e..e0239190ad 100644
--- a/xfa/include/fwl/basewidget/fwl_pushbutton.h
+++ b/xfa/include/fwl/basewidget/fwl_pushbutton.h
@@ -45,11 +45,8 @@ class IFWL_PushButtonDP : public IFWL_DataProvider {
 };

 class IFWL_PushButton : public IFWL_Widget {

  public:

-  static IFWL_PushButton* Create();

+  IFWL_PushButton();

   FWL_ERR Initialize(const CFWL_WidgetImpProperties& properties,

                      IFWL_Widget* pOuter);

-

- protected:

-  IFWL_PushButton();

 };

 #endif

diff --git a/xfa/include/fwl/basewidget/fwl_scrollbar.h b/xfa/include/fwl/basewidget/fwl_scrollbar.h
index 1dd7dd2d5c..10ce5f2688 100644
--- a/xfa/include/fwl/basewidget/fwl_scrollbar.h
+++ b/xfa/include/fwl/basewidget/fwl_scrollbar.h
@@ -47,7 +47,7 @@ enum FWL_SCBCODE {
 class IFWL_ScrollBarDP : public IFWL_DataProvider {};

 class IFWL_ScrollBar : public IFWL_Widget {

  public:

-  static IFWL_ScrollBar* Create();

+  IFWL_ScrollBar();

   FWL_ERR Initialize(const CFWL_WidgetImpProperties& properties,

                      IFWL_Widget* pOuter);

   FX_BOOL IsVertical();

@@ -62,8 +62,5 @@ class IFWL_ScrollBar : public IFWL_Widget {
   FX_FLOAT GetTrackPos();

   FWL_ERR SetTrackPos(FX_FLOAT fTrackPos);

   FX_BOOL DoScroll(FX_DWORD dwCode, FX_FLOAT fPos = 0.0f);

-

- protected:

-  IFWL_ScrollBar();

 };

 #endif

diff --git a/xfa/include/fwl/basewidget/fwl_spinbutton.h b/xfa/include/fwl/basewidget/fwl_spinbutton.h
index 521cfca277..79e3f68b5b 100644
--- a/xfa/include/fwl/basewidget/fwl_spinbutton.h
+++ b/xfa/include/fwl/basewidget/fwl_spinbutton.h
@@ -27,16 +27,13 @@ class IFWL_SpinButton;
 BEGIN_FWL_EVENT_DEF(CFWL_EvtSpbClick, FWL_EVTHASH_SPB_Click)

 FX_BOOL m_bUp;

 END_FWL_EVENT_DEF

+

 class IFWL_SpinButton : public IFWL_Widget {

  public:

-  static IFWL_SpinButton* Create();

-

+  IFWL_SpinButton();

   FWL_ERR Initialize(const CFWL_WidgetImpProperties& properties,

                      IFWL_Widget* pOuter);

   FWL_ERR EnableButton(FX_BOOL bEnable, FX_BOOL bUp = TRUE);

   FX_BOOL IsButtonEnable(FX_BOOL bUp = TRUE);

-

- protected:

-  IFWL_SpinButton();

 };

 #endif

diff --git a/xfa/include/fwl/basewidget/fwl_tooltipctrl.h b/xfa/include/fwl/basewidget/fwl_tooltipctrl.h
index 2dce69383f..ac467556d8 100644
--- a/xfa/include/fwl/basewidget/fwl_tooltipctrl.h
+++ b/xfa/include/fwl/basewidget/fwl_tooltipctrl.h
@@ -31,14 +31,11 @@ class IFWL_ToolTipDP : public IFWL_DataProvider {
 };

 class IFWL_ToolTip : public IFWL_Form {

  public:

-  static IFWL_ToolTip* Create();

+  IFWL_ToolTip();

   FWL_ERR Initialize(const CFWL_WidgetImpProperties& properties,

                      IFWL_Widget* pOuter);

   FWL_ERR SetAnchor(const CFX_RectF& rtAnchor);

   FWL_ERR Show();

   FWL_ERR Hide();

-

- protected:

-  IFWL_ToolTip();

 };

 #endif