[Tizen] Fix the issue that some callbacks are not called. 30/297730/2 accepted/tizen/7.0/unified/20230825.185057 accepted/tizen/7.0/unified/20230915.095423
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 09:02:56 +0000 (17:02 +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 3246af6..0539e22 100755 (executable)
@@ -756,6 +756,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());
@@ -764,6 +765,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);
 }
 
@@ -775,6 +777,7 @@ void TizenWebEngineChromium::GetPlainTextAsynchronously(PlainTextReceivedCallbac
 
 void TizenWebEngineChromium::RegisterGeolocationPermissionCallback(GeolocationPermissionCallback callback)
 {
+  mGeolocationPermissionCallback = callback;
   ewk_view_geolocation_permission_callback_set(mWebView, &TizenWebEngineChromium::OnGeolocationPermission, this);
 }