Fix the issue that some callbacks are not called. 02/297702/1
authorhuayong.xu <huayong.xu@samsung.com>
Wed, 23 Aug 2023 06:00:00 +0000 (14:00 +0800)
committerhuayong.xu <huayong.xu@samsung.com>
Wed, 23 Aug 2023 06:00:00 +0000 (14:00 +0800)
Callbacks are missing in the patch
https://review.tizen.org/gerrit/#/c/platform/core/uifw/dali-extension/+/278793/

Change-Id: I421d0c3a01986e7625a2de91693e62b603e7df9a

dali-extension/web-engine-chromium/tizen-web-engine-chromium.cpp

index 7c2639a..512e03a 100755 (executable)
@@ -772,6 +772,7 @@ Dali::PixelData TizenWebEngineChromium::GetScreenshot(Dali::Rect<int32_t> viewAr
 
 bool TizenWebEngineChromium::GetScreenshotAsynchronously(Dali::Rect<int32_t> viewArea, float scaleFactor, ScreenshotCapturedCallback callback)
 {
+  mScreenshotCapturedCallback = callback;
   Eina_Rectangle rect;
   EINA_RECTANGLE_SET(&rect, viewArea.x, viewArea.y, viewArea.width, viewArea.height);
   Evas* evas = ecore_evas_get(WebEngineManager::Get().GetWindow());
@@ -780,6 +781,7 @@ bool TizenWebEngineChromium::GetScreenshotAsynchronously(Dali::Rect<int32_t> vie
 
 bool TizenWebEngineChromium::CheckVideoPlayingAsynchronously(VideoPlayingCallback callback)
 {
+  mVideoPlayingCallback = callback;
   return ewk_view_is_video_playing(mWebView, &TizenWebEngineChromium::OnVideoPlaying, this);
 }
 
@@ -791,6 +793,7 @@ void TizenWebEngineChromium::GetPlainTextAsynchronously(PlainTextReceivedCallbac
 
 void TizenWebEngineChromium::RegisterGeolocationPermissionCallback(GeolocationPermissionCallback callback)
 {
+  mGeolocationPermissionCallback = callback;
   ewk_view_geolocation_permission_callback_set(mWebView, &TizenWebEngineChromium::OnGeolocationPermission, this);
 }