Remove Release() from IFX_Edit_UndoItem.

Also delete unused CFFL_Edit_UndoItem class.

BUG=https://code.google.com/p/pdfium/issues/detail?id=140
R=thestig@chromium.org

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

3 files changed, 9 insertions, 30 deletions

diff --git a/fpdfsdk/include/formfiller/FFL_TextField.h b/fpdfsdk/include/formfiller/FFL_TextField.h
index e31a9e5462..58ac5943eb 100644
--- a/fpdfsdk/include/formfiller/FFL_TextField.h
+++ b/fpdfsdk/include/formfiller/FFL_TextField.h
@@ -15,19 +15,6 @@
 
 class CBA_FontMap;
 
-
-class CFFL_EditUndoItem
-{
-public:
-	CFFL_EditUndoItem(CPWL_Edit* pEdit);
-	virtual ~CFFL_EditUndoItem();
-	
-	virtual void					Undo();
-	virtual void					Redo();
-	virtual CFX_WideString			GetDescr();
-	virtual void					Release();
-};
-
 struct FFL_TextFieldState
 {
 	int nStart;
@@ -79,9 +66,8 @@ public:
 	
 private:
 	CBA_FontMap*				m_pFontMap;
-//	CBA_SpellCheck*				m_pSpellCheck;
 	FFL_TextFieldState			m_State;
-//	CFFL_IM_BOX					m_IMBox;
+
 };
 
 #endif // !defined(AFX_FFL_EDIT_H__8E0C9456_CBA2_4EFB_9F31_53C6D8C1A8AC__INCLUDED_)
diff --git a/fpdfsdk/include/fxedit/fx_edit.h b/fpdfsdk/include/fxedit/fx_edit.h
index a842fd6284..bf00995ea9 100644
--- a/fpdfsdk/include/fxedit/fx_edit.h
+++ b/fpdfsdk/include/fxedit/fx_edit.h
@@ -164,13 +164,11 @@ public:
 class IFX_Edit_UndoItem
 {
 public:
-	virtual void							Release() = 0;
+	virtual ~IFX_Edit_UndoItem() { }
+
 	virtual void							Undo() = 0;
 	virtual void							Redo() = 0;
 	virtual CFX_WideString					GetUndoTitle() = 0;
-
-protected:
-        ~IFX_Edit_UndoItem() { }
 };
 
 class FXET_CLASS IFX_Edit
diff --git a/fpdfsdk/include/fxedit/fxet_edit.h b/fpdfsdk/include/fxedit/fxet_edit.h
index 8aa44a4bed..9e5c40f5ff 100644
--- a/fpdfsdk/include/fxedit/fxet_edit.h
+++ b/fpdfsdk/include/fxedit/fxet_edit.h
@@ -325,12 +325,9 @@ class CFX_Edit_UndoItem : public IFX_Edit_UndoItem
 {
 public:
 	CFX_Edit_UndoItem() : m_bFirst(TRUE), m_bLast(TRUE) {}
-	virtual ~CFX_Edit_UndoItem(){}
 
-	virtual CFX_WideString					GetUndoTitle() {return L"";}
-	virtual void							Release(){delete this;}
+	CFX_WideString GetUndoTitle() override { return L""; }
 
-public:
 	void									SetFirst(FX_BOOL bFirst){m_bFirst = bFirst;}
 	FX_BOOL									IsFirst(){return m_bFirst;}
 	void									SetLast(FX_BOOL bLast){m_bLast = bLast;}
@@ -345,17 +342,15 @@ class CFX_Edit_GroupUndoItem : public IFX_Edit_UndoItem
 {
 public:
 	CFX_Edit_GroupUndoItem(const CFX_WideString& sTitle);
-	virtual ~CFX_Edit_GroupUndoItem();
+	~CFX_Edit_GroupUndoItem() override;
+
+	void							Undo() override;
+	void							Redo() override;
+	CFX_WideString					GetUndoTitle() override;
 
 	void									AddUndoItem(CFX_Edit_UndoItem* pUndoItem);
 	void									UpdateItems();
 
-public:
-	virtual void							Undo();
-	virtual void							Redo();
-	virtual CFX_WideString					GetUndoTitle();
-	virtual void							Release();
-
 private:
 	CFX_WideString							m_sTitle;
 	CFX_ArrayTemplate<CFX_Edit_UndoItem*>	m_Items;