Show context menu for email and phone link
authorRobert Jadczak <r.jadczak@samsung.com>
Fri, 13 Feb 2015 15:28:37 +0000 (16:28 +0100)
committerYoungsoo Choi <kenshin.choi@samsung.com>
Tue, 10 Jul 2018 06:57:09 +0000 (06:57 +0000)
Added context menu with proper options for tap event on email or phone link.

Bug: http://web.sec.samsung.net/bugzilla/show_bug.cgi?id=11229
Reviewed by: Antonio Gomes, Marcin Kolibabka, Piotr Grad, Piotr Tworek

Change-Id: Ia67069cf6ee76a5f26b21c897b9b4848d6d5b7ef
Signed-off-by: Robert Jadczak <r.jadczak@samsung.com>
tizen_src/impl/browser/renderer_host/render_widget_host_view_efl.cc

index 1d7fedb..512e677 100644 (file)
@@ -1294,14 +1294,18 @@ void HandleTap(
   if (!settings)
     return;
 
-  if (settings->linkEffectEnabled() && hit_test &&
-      (hit_test->GetResultContext() &
-      tizen_webview::TW_HIT_TEST_RESULT_CONTEXT_LINK)) {
-    sound_effect::playLinkEffect();
+  if (!hit_test)
+    return;
+
+  if (hit_test->GetResultContext() &
+      tizen_webview::TW_HIT_TEST_RESULT_CONTEXT_LINK) {
+    if (settings->linkEffectEnabled())
+      sound_effect::playLinkEffect();
 #if defined(TIZEN_CONTENTS_DETECTION)
-  } else if (settings->detectContentsAutomatically() && hit_test &&
-             (hit_test->GetResultContext() &
-             tizen_webview::TW_HIT_TEST_RESULT_CONTEXT_TEXT)) {
+    rwhv->OnContentsDetected(hit_test->GetLinkUri());
+  } else if (settings->detectContentsAutomatically() &&
+      (hit_test->GetResultContext() &
+          tizen_webview::TW_HIT_TEST_RESULT_CONTEXT_TEXT)) {
     rwhv->DetectContentOnPosition(x, y);
 #endif
   }