Added routine to delete m_moveAnimator and m_showTimer in the destructor of TextSelec...
authorYuni Jeong <yhnet.jung@samsung.com>
Mon, 15 Apr 2013 15:07:44 +0000 (00:07 +0900)
committerGerrit Code Review <gerrit2@kim11>
Tue, 16 Apr 2013 09:46:38 +0000 (18:46 +0900)
[Title] Added routine to delete m_moveAnimator and m_showTimer in the destructor of TextSelection.
[Issue#] N/A
[Problem] Crash can occur when deleting webview while m_moveAnimator or m_showTimer is working
[Cause] There is no routine to delete m_moveAnimator and m_showTimer in the destructor of TextSelection.
[Solution] Added routine to delete m_moveAnimator and m_showTimer in the destructor of TextSelection.

Change-Id: I468045249174aab5ade1b8e2345b071f9dca7f59

Source/WebKit2/UIProcess/API/efl/tizen/TextSelection.cpp

index 3b2cfc1..6f8e030 100755 (executable)
@@ -78,6 +78,15 @@ TextSelection::~TextSelection()
 
     delete m_magnifier;
 
+    if (m_moveAnimator) {
+        ecore_animator_del(m_moveAnimator);
+        m_moveAnimator = 0;
+    }
+
+    if (m_showTimer) {
+        ecore_timer_del(m_showTimer);
+        m_showTimer = 0;
+    }
     evas_object_event_callback_del(m_viewImpl->view(), EVAS_CALLBACK_MOUSE_UP, onMouseUp);
 }