Bug 163514 - REGRESSION(r199017): [GTK] Layout test media/media-source/media-source-sequence-timestamps.html is failing
Summary: REGRESSION(r199017): [GTK] Layout test media/media-source/media-source-sequen...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Media (show other bugs)
Version: Other
Hardware: PC Linux
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on: 164022
Blocks:
  Show dependency treegraph
 
Reported: 2016-10-16 18:45 PDT by Michael Catanzaro
Modified: 2016-12-26 03:37 PST (History)
4 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Catanzaro 2016-10-16 18:45:19 PDT
Layout test media/media-source/media-source-sequence-timestamps.html is failing since r199017 "Regression(r196145): Crash in getOwnPropertyDescriptor on ​http://www.history.com/shows/vikings"

That commit looks completely unrelated to media-source, but our layout test bot does not lie, the test is not flaky at all and is is blamed to either 199016 (which is definitely not it) or r199017. Huh. Text diff:

--- /home/slave/webkitgtk/gtk-linux-64-release-tests/build/layout-test-results/media/media-source/media-source-sequence-timestamps-expected.txt
+++ /home/slave/webkitgtk/gtk-linux-64-release-tests/build/layout-test-results/media/media-source/media-source-sequence-timestamps-actual.txt
@@ -6,12 +6,6 @@
 EVENT(updateend)
 RUN(sourceBuffer.appendBuffer(samples))
 EVENT(updateend)
-EXPECTED (bufferedSamples.length == '6') OK
-{PTS({0/1, 0.000000}), DTS({0/1, 0.000000}), duration({1000/1000, 1.000000}), flags(1), generation(0)}
-{PTS({1000/1000, 1.000000}), DTS({1000/1000, 1.000000}), duration({1000/1000, 1.000000}), flags(0), generation(0)}
-{PTS({2000/1000, 2.000000}), DTS({2000/1000, 2.000000}), duration({1000/1000, 1.000000}), flags(0), generation(0)}
-{PTS({3000/1000, 3.000000}), DTS({3000/1000, 3.000000}), duration({1000/1000, 1.000000}), flags(1), generation(0)}
-{PTS({4000/1000, 4.000000}), DTS({4000/1000, 4.000000}), duration({1000/1000, 1.000000}), flags(0), generation(0)}
-{PTS({5000/1000, 5.000000}), DTS({5000/1000, 5.000000}), duration({1000/1000, 1.000000}), flags(0), generation(0)}
+EXPECTED (bufferedSamples.length == '6'), OBSERVED '0' FAIL
 END OF TEST

Updating expectations accordingly.
Comment 1 Michael Catanzaro 2016-10-16 19:03:55 PDT
Note this looks really similar to bug #163516, but these are regressions from different commits.
Comment 2 Enrique Ocaña 2016-12-26 03:37:10 PST
Fixed by #164022 on http://svn.webkit.org/repository/webkit/trunk@209797