Bug 162143

Summary: [EFL] ewk_context_menu_item_select test have been failed
Product: WebKit Reporter: Hunseop Jeong <hs85.jeong>
Component: WebKit EFLAssignee: Hunseop Jeong <hs85.jeong>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, gyuyoung.kim, lucas.de.marchi
Priority: P2    
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch none

Description Hunseop Jeong 2016-09-19 01:35:45 PDT
https://build.webkit.org/builders/EFL%20Linux%2064-bit%20Release%20WK2/builds/29782/steps/API%20tests/logs/stdio

[----------] 2 tests from EWK2ContextMenuTest
[ RUN      ] EWK2ContextMenuTest.ewk_context_menu_item_select
ERR<17602>:ecore_evas modules/ecore_evas/engines/x/ecore_evas_x.c:4584 ecore_evas_gl_x11_options_new_internal() evas_engine_info_set() init engine 'opengl_x11' failed.
ERR<17602>:evas_main lib/evas/canvas/evas_gl.c:151 evas_gl_new() Evas GL engine not available.
../../Source/WebKit2/UIProcess/API/efl/tests/test_ewk2_context_menu.cpp:202: Failure
Value of: waitUntilLoadFinished()
  Actual: false
Expected: true
[  FAILED  ] EWK2ContextMenuTest.ewk_context_menu_item_select (10152 ms)
[ RUN      ] EWK2ContextMenuTest.ewk_context_menu_custom_items
ERR<17602>:ecore_evas modules/ecore_evas/engines/x/ecore_evas_x.c:4584 ecore_evas_gl_x11_options_new_internal() evas_engine_info_set() init engine 'opengl_x11' failed.
ERR<17602>:evas_main lib/evas/canvas/evas_gl.c:151 evas_gl_new() Evas GL engine not available.
[       OK ] EWK2ContextMenuTest.ewk_context_menu_custom_items (228 ms)
[----------] 2 tests from EWK2ContextMenuTest (10380 ms total)

[----------] Global test environment tear-down
[==========] 2 tests from 1 test case ran. (10380 ms total)
[  PASSED  ] 1 test.
[  FAILED  ] 1 test, listed below:
[  FAILED  ] EWK2ContextMenuTest.ewk_context_menu_item_select
Comment 1 Hunseop Jeong 2016-09-19 18:57:22 PDT
Created attachment 289298 [details]
Patch
Comment 2 Gyuyoung Kim 2016-09-20 02:21:10 PDT
Comment on attachment 289298 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=289298&action=review

LGTM with my comment.

> Source/WebKit2/ChangeLog:9
> +        to wait to finish the loading. I change the codes to finish the test when "testFinished" is true. 

It seems to me that this description is unclear to me. I revise it as below,

ewk_context_menu_item_select has been waited until test page loading is finished, but it is unnecessary
to wait the loading is finished. It would be good to pass if *testFinished* flag is true.

> Source/WebKit2/UIProcess/API/efl/tests/test_ewk2_context_menu.cpp:200
> +    ewkViewClass()->context_menu_hide = finishTest;

Nice catch. We have missed to do it.

> Source/WebKit2/UIProcess/API/efl/tests/test_ewk2_context_menu.cpp:205
> +    testFinished = false;

ditto.
Comment 3 Hunseop Jeong 2016-09-20 17:37:30 PDT
Created attachment 289413 [details]
Patch
Comment 4 Hunseop Jeong 2016-09-20 17:40:16 PDT
(In reply to comment #2)
> Comment on attachment 289298 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=289298&action=review
> 
> LGTM with my comment.
> 
> > Source/WebKit2/ChangeLog:9
> > +        to wait to finish the loading. I change the codes to finish the test when "testFinished" is true. 
> 
> It seems to me that this description is unclear to me. I revise it as below,
> 
> ewk_context_menu_item_select has been waited until test page loading is
> finished, but it is unnecessary
> to wait the loading is finished. It would be good to pass if *testFinished*
> flag is true.

Thanks Gyuyoung, I will try to write down a clear description.
Comment 5 WebKit Commit Bot 2016-09-20 19:36:28 PDT
Comment on attachment 289413 [details]
Patch

Clearing flags on attachment: 289413

Committed r206194: <http://trac.webkit.org/changeset/206194>
Comment 6 WebKit Commit Bot 2016-09-20 19:36:32 PDT
All reviewed patches have been landed.  Closing bug.