Fix for issue about memory leak
authorhyun lee <hyunn.lee@samsung.com>
Wed, 21 Aug 2013 02:21:32 +0000 (11:21 +0900)
committerhyun lee <hyunn.lee@samsung.com>
Wed, 21 Aug 2013 02:21:32 +0000 (11:21 +0900)
Change-Id: Ibe332ae8349e7b80248dea28e32e8eb24dbdaf18
Signed-off-by: hyun lee <hyunn.lee@samsung.com>
src/controls/FWebCtrl_WebImpl.cpp

index 637f8ab..386ef35 100755 (executable)
@@ -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: