Bug 178053 – usemap-casing.html test passes in browser, fails in LayoutTests because it is taller than 600px

• Status: RESOLVED
• Resolution: FIXED
• Component: Tools / Tests

[Darin Adler]

To reproduce:

% open-layout-test LayoutTests/imported/w3c/web-platform-tests/html/semantics/embedded-content/the-img-element/usemap-casing.html

Note that all 8 tests pass. Same thing if you just open <http://w3c-test.org/html/semantics/embedded-content/the-img-element/usemap-casing.html>.

% run-webkit-tests imported/w3c/web-platform-tests/html/semantics/embedded-content/the-img-element/usemap-casing.html

It says "The test ran as expected."

But "as expected" means 4 tests passing and 4 tests failing. You can see that in the expected results file LayoutTests/imported/w3c/web-platform-tests/html/semantics/embedded-content/the-img-element/usemap-casing-expected.txt and can see that the failure is because element.parentElement is null. To me seems to indicate that the images did not load.

[youenn fablet]

Testing on Chrome and Firefox, the last 4 tests are also failing.

Testing on Safari, the test seems flaky when opening the same test on different tabs.
Last two tests sometimes fail with the same error message.

Since Chrome, Firefox and Safari all exhibit different behaviors, we might want to look at the test and see what is causing this issue.
Maybe an issue with the way the test is written and/or some interop issue.

[Chris Dumez]

Oh, this is because of our 800x600 restriction in layout tests. elementFromPoint() returns null because the element is outside the viewport.

[Chris Dumez]

(In reply to Chris Dumez from comment #2)
> Oh, this is because of our 800x600 restriction in layout tests.
> elementFromPoint() returns null because the element is outside the viewport.

I guess we could make the test behave nicely with DRT / WKTR by using something smaller than 300x300 for each element.

[youenn fablet]

I think wpt ref tests expect 800x600 viewport so changing the test should be fine.

[Ryosuke Niwa]

WPT tests are supposed to fit within a smaller viewport than 800x600 so this is just a bug that needs to be fixed upstream.

[Chris Dumez]

Upstream PR: https://github.com/w3c/web-platform-tests/pull/7644

[Chris Dumez]

Created attachment 323225 [details]
Patch

[WebKit Commit Bot]

The commit-queue encountered the following flaky tests while processing attachment 323225 [details]:

media/modern-media-controls/seek-backward-support/seek-backward-support.html bug 174916 (authors: [e-mail], [e-mail], and [e-mail])
The commit-queue is continuing to process your patch.

[WebKit Commit Bot]

The commit-queue encountered the following flaky tests while processing attachment 323225 [details]:

The commit-queue is continuing to process your patch.

[WebKit Commit Bot]

Comment on attachment 323225 [details]
Patch

Clearing flags on attachment: 323225

Committed r223087: <http://trac.webkit.org/changeset/223087>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[Radar WebKit Bug Importer]

<rdar://problem/34898968>