From: hyun lee Date: Wed, 21 Aug 2013 02:21:32 +0000 (+0900) Subject: Fix for issue about memory leak X-Git-Tag: submit/tizen_2.2/20131107.062229~56^2 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=68bba16f3d0412128ffa66f0dbb826d196517fed;p=framework%2Fosp%2Fweb.git Fix for issue about memory leak Change-Id: Ibe332ae8349e7b80248dea28e32e8eb24dbdaf18 Signed-off-by: hyun lee --- diff --git a/src/controls/FWebCtrl_WebImpl.cpp b/src/controls/FWebCtrl_WebImpl.cpp index 637f8ab..386ef35 100755 --- a/src/controls/FWebCtrl_WebImpl.cpp +++ b/src/controls/FWebCtrl_WebImpl.cpp @@ -4676,14 +4676,13 @@ _WebImpl::ShowUserConfirmPopupAsync(_UserConfirmMode userConfirmMode, void* pPol { case USER_CONFIRM_USERMEDIA: { - __pUserMediaConfirmPopup.reset(); + __pUserMediaConfirmPopup.reset(); __pUserMediaConfirmPopup = std::move(pUserConfirmPopup); r = __pUserMediaConfirmPopup->ShowPopup(); SysTryCatch(NID_WEB_CTRL, r == E_SUCCESS, __pUserMediaConfirmPopup.reset(), r, "[%s] Propagating.", GetErrorMessage(r)); - __pUserMediaConfirmPopup.release(); break; } case USER_CONFIRM_GEOLOCATION: @@ -4695,7 +4694,6 @@ _WebImpl::ShowUserConfirmPopupAsync(_UserConfirmMode userConfirmMode, void* pPol r = __pGeolocationConfirmPopup->ShowPopup(); SysTryCatch(NID_WEB_CTRL, r == E_SUCCESS, __pGeolocationConfirmPopup.reset(), r, "[%s] Propagating.", GetErrorMessage(r)); - __pGeolocationConfirmPopup.release(); break; } case USER_CONFIRM_NOTIFICATION: @@ -4707,7 +4705,6 @@ _WebImpl::ShowUserConfirmPopupAsync(_UserConfirmMode userConfirmMode, void* pPol r = __pNotificationConfirmPopup->ShowPopup(); SysTryCatch(NID_WEB_CTRL, r == E_SUCCESS, __pNotificationConfirmPopup.reset(), r, "[%s] Propagating.", GetErrorMessage(r)); - __pNotificationConfirmPopup.release(); break; } case USER_SCRIPT_ALERT: @@ -4719,7 +4716,6 @@ _WebImpl::ShowUserConfirmPopupAsync(_UserConfirmMode userConfirmMode, void* pPol r = __pScriptAlertConfirmPopup->ShowPopup(); SysTryCatch(NID_WEB_CTRL, r == E_SUCCESS, __pScriptAlertConfirmPopup.reset(), r, "[%s] Propagating.", GetErrorMessage(r)); - __pScriptAlertConfirmPopup.release(); break; } case USER_SCRIPT_CONFIRM: @@ -4731,8 +4727,6 @@ _WebImpl::ShowUserConfirmPopupAsync(_UserConfirmMode userConfirmMode, void* pPol r = __pScriptConfirmPopup->ShowPopup(); SysTryCatch(NID_WEB_CTRL, r == E_SUCCESS, __pScriptConfirmPopup.reset(), r, "[%s] Propagating.", GetErrorMessage(r)); - __pScriptConfirmPopup.release(); - break; } default: