From f4eba48a1ec3230899b1ce075766fdef49d8529b Mon Sep 17 00:00:00 2001 From: Yuni Jeong Date: Wed, 17 Jul 2013 22:16:02 +0900 Subject: [PATCH] Fixed sometimes focus ring is hidden when context menu is shown by longTap [Title] Fixed sometimes focus ring is hidden when context menu is shown by longTap [Issue#] P130709-3174 [Problem] Sometimes focus ring is hidden when context menu is shown by longTap [Cause] Focus ring is hidden when gesture move event occur. If minute movement occur when longTap, gesture move event occur. So, focus ring is hidden. [Solution] When context menu is showing, focus ring should not be hidden. So, added condition to check whether context menu is visible or invisible when gesture move event occur. Change-Id: I491068689b79aa9d2ef9b1b2c6f1fc555a9162a2 --- Source/WebKit2/UIProcess/API/efl/ewk_view.cpp | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/Source/WebKit2/UIProcess/API/efl/ewk_view.cpp b/Source/WebKit2/UIProcess/API/efl/ewk_view.cpp index af456ca..7684077 100755 --- a/Source/WebKit2/UIProcess/API/efl/ewk_view.cpp +++ b/Source/WebKit2/UIProcess/API/efl/ewk_view.cpp @@ -586,8 +586,12 @@ static Eina_Bool _ewk_view_smart_gesture_move(Ewk_View_Smart_Data* smartData, co #endif #if ENABLE(TIZEN_WEBKIT2_FOCUS_RING) - if (impl->focusRing && !(event->type == EWK_GESTURE_PAN && !impl->exceedTouchMoveThreshold)) - impl->focusRing->requestToHide(true); + if (impl->focusRing && !(event->type == EWK_GESTURE_PAN && !impl->exceedTouchMoveThreshold)) { +#if ENABLE(TIZEN_CONTEXT_MENU_WEBKIT_2) + if (!impl->pageClient->isContextMenuVisible()) +#endif + impl->focusRing->requestToHide(true); + } #endif switch (event->type) { -- 2.7.4