Efl does not have a proper way to support nested loop so that applications may change values outside showPopupMenu. Like Qt port, WebKit2/Efl should keep activePopupMenu to change value of popup menu after showPopupMenu is finished.
Created attachment 161255 [details] Patch
Comment on attachment 161255 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=161255&action=review > Source/WebKit2/UIProcess/API/efl/tests/test_ewk2_view.cpp:192 > +static Eina_Bool selectItemAfterDelayed(void* data) Please use standard boolean instead of Eina_Bool.
Comment on attachment 161255 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=161255&action=review >> Source/WebKit2/UIProcess/API/efl/tests/test_ewk2_view.cpp:192 >> +static Eina_Bool selectItemAfterDelayed(void* data) > > Please use standard boolean instead of Eina_Bool. callback function needs to use Eina_Bool. please ignore my previous comment.
Comment on attachment 161255 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=161255&action=review > Source/WebKit2/UIProcess/API/efl/tests/test_ewk2_view.cpp:195 > + return false; Is ECORE_CALLBACK_CANCEL more clear ?
Created attachment 161851 [details] Patch
(In reply to comment #4) > (From update of attachment 161255 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=161255&action=review > > > Source/WebKit2/UIProcess/API/efl/tests/test_ewk2_view.cpp:195 > > + return false; > > Is ECORE_CALLBACK_CANCEL more clear ? Yes, it's more clear for efl folks. I fixed.
Comment on attachment 161851 [details] Patch Clearing flags on attachment: 161851 Committed r127397: <http://trac.webkit.org/changeset/127397>
All reviewed patches have been landed. Closing bug.