Bug 95366 – flexitem-stretch-image.html is flaky

• Component: Tools / Tests
• Status: RESOLVED DUPLICATE

[Tony Chang]

I haven't added this test yet, but it's moving a test that's flaky in flexitem.html to this file.  This should help us identify the flakiness and allow us to get some test coverage back while we diagnose.

[Tony Chang]

Created attachment 161340 [details]
Patch

[Ojan Vafai]

Comment on attachment 161340 [details]
Patch

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

> LayoutTests/ChangeLog:11
> +

Maybe mention that you made it only check the one element by using the ID and when we fix the bug, we can delete all the other extraneous junk.

[Jessie Berlin]

All the Mac platforms are now reporting this css3/flexbox/flexitem-stretch-image.html as an unexpected pass:
http://build.webkit.org/results/Apple%20MountainLion%20Debug%20WK1%20(Tests)/r127166%20(374)/results.html
http://build.webkit.org/results/Apple%20MountainLion%20Release%20WK1%20(Tests)/r127171%20(433)/results.html
http://build.webkit.org/results/Apple%20MountainLion%20Debug%20WK2%20(Tests)/r127166%20(447)/results.html
http://build.webkit.org/results/Apple%20MountainLion%20Release%20WK2%20(Tests)/r127171%20(439)/results.html
http://build.webkit.org/results/Apple%20Lion%20Debug%20WK1%20(Tests)/r127166%20(2386)/results.html
http://build.webkit.org/results/Apple%20Lion%20Release%20WK1%20(Tests)/r127171%20(3007)/results.html
http://build.webkit.org/results/Apple%20Lion%20Debug%20WK2%20(Tests)/r127167%20(2977)/results.html
http://build.webkit.org/results/Apple%20Lion%20Release%20WK2%20(Tests)/r127165%20(2770)/results.html

I originally posted this information in https://bugs.webkit.org/show_bug.cgi?id=94720 (since that was the bug associated with adding this test to TestExpecations). I was going remove it from TestExpecations, but seeing this title makes me unsure.

Should I just mark it as TEXT PASS?

[Ojan Vafai]

As expected, this started failing again on the Chromium Mac debug bots after http://trac.webkit.org/changeset/127050.

[Tony Chang]

Comment on attachment 161340 [details]
Patch

Looks like this test started failing on Chromium Mac Debug, so we don't need this change.

[Tony Chang]

I think Jessie is going to remove the test from mac/TestExpectations.  I think I can remove it in a few other places too.

[Jessie Berlin]

(In reply to comment #6)
> I think Jessie is going to remove the test from mac/TestExpectations.  I think I can remove it in a few other places too.

Removed the test from mac/TestExpectations in http://trac.webkit.org/changeset/127184

[Tony Chang]

Removed from efl/TestExpectations in http://trac.webkit.org/changeset/127196.  The test is only flaky on debug chromium builds, but it may just be a race that is triggered by the slower builds/machines.

[Tony Chang]

The fix for the flaky test is in bug 98611.

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