Fixed sometimes focus ring is hidden when context menu is shown by longTap
authorYuni Jeong <yhnet.jung@samsung.com>
Wed, 17 Jul 2013 13:16:02 +0000 (22:16 +0900)
committerYuni Jeong <yhnet.jung@samsung.com>
Wed, 24 Jul 2013 06:12:09 +0000 (15:12 +0900)
[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

index af456ca..7684077 100755 (executable)
@@ -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) {