Bug 142489

Summary:

[GTK] Layout Test http/tests/media/video-play-waiting.html is timing out

Product:

WebKit

Reporter:

Marcos Chavarría Teijeiro (irc: chavaone) <chavarria1991>

Component:

Tools / Tests

Assignee:

Enrique Ocaña <eocanha>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

bugs-noreply, chavarria1991, clopez, eocanha, eric.carlson, ews-watchlist, glenn, jer.noble, philipj, pnormand, sergio, webkit-bug-importer

Priority:

Keywords:

InRadar

Version:

528+ (Nightly build)

Hardware:

Unspecified

OS:

Unspecified

See Also:

https://bugs.webkit.org/show_bug.cgi?id=140639

Attachments:

Description	Flags
Patch	none
Patch	none
Patch	none
Patch	none

Marcos Chavarría Teijeiro (irc: chavaone)

Reported 2015-03-09 05:48:56 PDT

The http/tests/media/video-play-waiting.html layout test is timing out since it was added on r179220 (https://trac.webkit.org/changeset/r179220).

Attachments
Patch (2.03 KB, patch) 2021-11-24 11:08 PST, Enrique Ocaña	no flags	Details Formatted Diff Diff
Patch (2.92 KB, patch) 2021-11-24 11:26 PST, Enrique Ocaña	no flags	Details Formatted Diff Diff
Patch (4.45 KB, patch) 2021-11-25 01:42 PST, Enrique Ocaña	no flags	Details Formatted Diff Diff
Patch (4.59 KB, patch) 2021-11-25 03:06 PST, Enrique Ocaña	no flags	Details Formatted Diff Diff
Show Obsolete (3) View All Add attachment proposed patch, testcase, etc.

Enrique Ocaña

Comment 1 2021-11-24 11:08:33 PST

Created attachment 445102 [details] Patch

Philippe Normand

Comment 2 2021-11-24 11:16:15 PST

Comment on attachment 445102 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=445102&action=review > LayoutTests/ChangeLog:16 > + * http/tests/media/video-play-waiting.html: Set the video src to empty. this is not flagged in TestExpectations?

Enrique Ocaña

Comment 3 2021-11-24 11:21:35 PST

Comment on attachment 445102 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=445102&action=review >> LayoutTests/ChangeLog:16 >> + * http/tests/media/video-play-waiting.html: Set the video src to empty. > > this is not flagged in TestExpectations? I always forget to unflag, sorry. Thanks for catching it!

Enrique Ocaña

Comment 4 2021-11-24 11:26:55 PST

Created attachment 445103 [details] Patch

Philippe Normand

Comment 5 2021-11-24 12:59:35 PST

Comment on attachment 445103 [details] Patch Good catch, this indeed times out on all platforms, https://results.webkit.org/?suite=layout-tests&test=http%2Ftests%2Fmedia%2Fvideo-play-waiting.html maybe you can un-flag in the top-level TestExpectatoins too?

Enrique Ocaña

Comment 6 2021-11-25 01:42:56 PST

Created attachment 445125 [details] Patch

Enrique Ocaña

Comment 7 2021-11-25 03:06:08 PST

Created attachment 445129 [details] Patch