Bug 133517 - [GTK] 4 of the css3/viewport-percentage-lengths/viewport-percentage-lengths-.*.html tests time out
Summary: [GTK] 4 of the css3/viewport-percentage-lengths/viewport-percentage-lengths-....
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on: 133993
Blocks:
  Show dependency treegraph
 
Reported: 2014-06-04 10:47 PDT by Carlos Alberto Lopez Perez
Modified: 2014-06-17 15:27 PDT (History)
9 users (show)

See Also:


Attachments
Patch (10.66 KB, patch)
2014-06-17 09:53 PDT, Carlos Alberto Lopez Perez
no flags Details | Formatted Diff | Diff
Patch (14.04 KB, patch)
2014-06-17 14:48 PDT, Carlos Alberto Lopez Perez
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Carlos Alberto Lopez Perez 2014-06-04 10:47:25 PDT
The following tests ( added on http://trac.webkit.org/changeset/169407 ) time out on GTK:

css3/viewport-percentage-lengths/viewport-percentage-lengths-anonymous-block.html
css3/viewport-percentage-lengths/viewport-percentage-lengths-percent-size-child.html
css3/viewport-percentage-lengths/viewport-percentage-lengths-relative-font-size.html
css3/viewport-percentage-lengths/viewport-percentage-lengths-resize.html


The tests timeout always when the tests are run inside Xvfb (this is the default, and what the GTK bots do).

But when you run them on a real X server (this is done by exporting the environment variable USE_NATIVE_XDISPLAY) some of them pass:

$ USE_NATIVE_XDISPLAY=1 Tools/Scripts/run-webkit-tests --release --webkit-test-runner --gtk css3/viewport-percentage-lengths/viewport-percentage-lengths-anonymous-block.html css3/viewport-percentage-lengths/viewport-percentage-lengths-percent-size-child.html css3/viewport-percentage-lengths/viewport-percentage-lengths-relative-font-size.html css3/viewport-percentage-lengths/viewport-percentage-lengths-resize.html
Using port 'gtk-wk2'
Test configuration: <, x86, release>
Placing test results in /home/clopez/webkit/webkit/WebKitBuild/Release/layout-test-results
Baseline search path: gtk-wk2 -> wk2 -> gtk -> generic
Using Release build
Pixel tests disabled
Regular timeout: 6000, slow test timeout: 30000
Command line: /home/clopez/webkit/webkit/WebKitBuild/Release/bin/WebKitTestRunner -

Found 4 tests; running 4, skipping 0.
Running 1 WebKitTestRunner.     

[1/4] css3/viewport-percentage-lengths/viewport-percentage-lengths-anonymous-block.html failed unexpectedly (test timed out)
[3/4] css3/viewport-percentage-lengths/viewport-percentage-lengths-relative-font-size.html failed unexpectedly (test timed out)

Retrying 2 unexpected failure(s) ...

Running 1 WebKitTestRunner.

[2/2] css3/viewport-percentage-lengths/viewport-percentage-lengths-relative-font-size.html failed unexpectedly (test timed out)

2 tests ran as expected, 2 didn't:


Unexpected flakiness: timeouts (1)
  css3/viewport-percentage-lengths/viewport-percentage-lengths-anonymous-block.html [ Timeout Pass ]


Regressions: Unexpected timeouts (1)
  css3/viewport-percentage-lengths/viewport-percentage-lengths-relative-font-size.html [ Timeout ]


Maybe this information can be useful.

I tried to run the tests with --no-timeout and also with --time-out-ms=30000 (30 secs), but the behaviour is the same. They timeout always with Xvfb.
Comment 1 Bem Jones-Bey 2014-06-04 11:03:12 PDT
I'm betting those are timing out because the onresize event doesn't fire in DRT/WTR on the GTK+ port. Darin fixed DRT/WTR for the Mac port in Bug 133351 when he changed the tests to use the onresize event. He made an attempt to fix the other ports as well, but it is possible that fix didn't work for the GTK port. Hopefully it's an easy fix.
Comment 2 Carlos Alberto Lopez Perez 2014-06-17 09:53:22 PDT
Created attachment 233235 [details]
Patch
Comment 3 Martin Robinson 2014-06-17 10:11:45 PDT
Comment on attachment 233235 [details]
Patch

Awesome!
Comment 4 WebKit Commit Bot 2014-06-17 10:49:56 PDT
Comment on attachment 233235 [details]
Patch

Clearing flags on attachment: 233235

Committed r170058: <http://trac.webkit.org/changeset/170058>
Comment 5 WebKit Commit Bot 2014-06-17 10:50:00 PDT
All reviewed patches have been landed.  Closing bug.
Comment 6 WebKit Commit Bot 2014-06-17 12:50:34 PDT
Re-opened since this is blocked by bug 133993
Comment 7 Carlos Alberto Lopez Perez 2014-06-17 14:34:55 PDT
(In reply to comment #6)
> Re-opened since this is blocked by bug 133993

Ups! Forgot to update the resolution for the unit tests of Xvfb. I will upload an updated patch.
Comment 8 Carlos Alberto Lopez Perez 2014-06-17 14:48:09 PDT
Created attachment 233261 [details]
Patch
Comment 9 WebKit Commit Bot 2014-06-17 15:27:45 PDT
Comment on attachment 233261 [details]
Patch

Clearing flags on attachment: 233261

Committed r170078: <http://trac.webkit.org/changeset/170078>
Comment 10 WebKit Commit Bot 2014-06-17 15:27:49 PDT
All reviewed patches have been landed.  Closing bug.