Bug 121742

Summary: [GTK] Remove media/context-menu-actions.html from LayoutTests/platform/gtk-wk2/TestExpectations
Product: WebKit Reporter: Anton Obzhirov <obzhirov>
Component: WebKitGTKAssignee: Anton Obzhirov <obzhirov>
Status: RESOLVED DUPLICATE    
Severity: Normal CC: bugs-noreply, eocanha, pnormand
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Other   
OS: Linux   
Attachments:
Description Flags
Patch mario: review-

Description Anton Obzhirov 2013-09-21 02:25:37 PDT
[GTK] Remove media/context-menu-actions.html from LayoutTests/platform/gtk-wk2/TestExpectations
Comment 1 Anton Obzhirov 2013-09-21 02:30:07 PDT
Created attachment 212270 [details]
Patch
Comment 2 Mario Sanchez Prada 2013-09-23 03:36:32 PDT
Comment on attachment 212270 [details]
Patch

This test is still failing in the WK2 bot (see [1]), so this gardening is not correct.

However, it's worth mentioning that it's not crashing anymore, just timing out:

  01:39:37.250 26788 [180/386] media/context-menu-actions.html failed unexpectedly (test timed out)
  01:39:37.250 26788 worker/0 killing driver
  01:39:37.251 26788 worker/0 media/context-menu-actions.html failed:


Curiously enough, I manually run it locally (against SVN r156267) to see if I could reproduce the error and found that the test actually works, it just takes too much time to sucessfully finish, for some reason.

Check what happens if I increase timeout up to 10 secs:

  $ time Tools/Scripts/run-webkit-tests --gtk -2 --time-out-ms=10000 media/context-menu-actions.html
  Using port 'gtk-wk2'
  Test configuration: <, x86, release>
  Placing test results in /home/mario/work/WebKit/WebKitBuild/Release/layout-test-results
  Baseline search path: gtk-wk2 -> wk2 -> gtk -> generic
  Using Release build
  Pixel tests disabled
  Regular timeout: 10000, slow test timeout: 50000
  Command line: /home/mario/work/WebKit/WebKitBuild/Release/Programs/WebKitTestRunner -

  Found 1 test; running 1, skipping 0.
  Running 1 WebKitTestRunner.     

  The test ran as expected. 

  real   0m5.114s
  user   0m1.408s
  sys    0m0.436s


And if I remove the --time-out-ms parameter:

  [...]
  [1/1] media/context-menu-actions.html failed unexpectedly (test timed out)

  Retrying 1 unexpected failure(s) ...

  Running 1 WebKitTestRunner.

  [1/1] media/context-menu-actions.html^C
  Interrupted, exiting ...
  Failed to execute Tools/Scripts/new-run-webkit-tests at Tools/Scripts/run-webkit-tests line 100.

  real  0m7.021s
  user  0m1.352s
  sys   0m0.296s


Setting r- because of the explanation above.

[1] http://build.webkit.org/builders/GTK%20Linux%2064-bit%20Release%20WK2%20%28Tests%29/builds/8164/steps/layout-test/logs/stdio
Comment 3 Anton Obzhirov 2013-09-23 04:00:29 PDT
After running it on two different machines - passes on one and times out on another. From what I can see and it takes time to get video element initially.
It could probably problem for other video tests as well. Philippe, could
you comment on this please? I can try to investigate it further or may be we can just increase time out for the video tests.
Comment 4 Philippe Normand 2013-09-25 13:27:04 PDT
Do you know what takes so long?
5 seconds to play a local video is not normal...
Comment 5 Anton Obzhirov 2013-10-01 03:25:31 PDT
(In reply to comment #4)
> Do you know what takes so long?
> 5 seconds to play a local video is not normal...
Not really, I can try to investigate though.
Comment 6 Enrique OcaƱa 2020-06-16 04:26:41 PDT

*** This bug has been marked as a duplicate of bug 183611 ***