Bug 137525
Summary: | REGRESSION (r174402): media/track/track-forced-subtitles-in-band.html fails | ||
---|---|---|---|
Product: | WebKit | Reporter: | Andy Estes <aestes> |
Component: | Media | Assignee: | Jer Noble <jer.noble> |
Status: | RESOLVED FIXED | ||
Severity: | Normal | CC: | ap, darin, jer.noble |
Priority: | P2 | ||
Version: | 528+ (Nightly build) | ||
Hardware: | Mac | ||
OS: | All | ||
Bug Depends on: | 137472 | ||
Bug Blocks: | 137505 |
Andy Estes
After r174403, media/track/track-forced-subtitles-in-band.html has started failing on the Mac wk1 and wk2 test bots like so:
--- /Volumes/Data/slave/mavericks-release-tests-wk2/build/layout-test-results/media/track/track-forced-subtitles-in-band-expected.txt
+++ /Volumes/Data/slave/mavericks-release-tests-wk2/build/layout-test-results/media/track/track-forced-subtitles-in-band-actual.txt
@@ -10,7 +10,7 @@
** Forced tracks should be in .textTracks, but not in the menu
EXPECTED (video.textTracks.length == '9') OK
-EXPECTED (trackMenuItems.length == '6') OK
+EXPECTED (trackMenuItems.length == '6'), OBSERVED '3' FAIL
** Only the 'fr' forced track should be showing
EXPECTED (video.textTracks[0].language == 'en') OK
See https://build.webkit.org/results/Apple%20Mavericks%20Release%20WK2%20(Tests)/r174451%20(8513)/results.html for more details.
Attachments | ||
---|---|---|
Add attachment proposed patch, testcase, etc. |
Andy Estes
Updated TestExpectations in r174458: <http://trac.webkit.org/changeset/174458>
Chris Dumez
Ok, I'll take a look thanks
Andy Estes
r174402 is the culprit, not r174403. Off-by-one error on my part :)
Alexey Proskuryakov
This text has stopped failing on the next day, and is "only" flaky once again.
Jer, do you know what fixed this? Could it be <http://trac.webkit.org/changeset/174460>?
Jer Noble
(In reply to comment #4)
> This text has stopped failing on the next day, and is "only" flaky once
> again.
>
> Jer, do you know what fixed this? Could it be
> <http://trac.webkit.org/changeset/174460>?
No, it was probably <http://trac.webkit.org/changeset/174472>.
Alexey Proskuryakov
Indeed - there was a large gap in coverage, so I didn't see that at first. Historical data agrees with r174472 being the fix.
So, the fix for the "test is flaky" bug addressed it failing every time, but did not address the flakiness.