Bug 158914 - [GTK] Layout test media/restore-from-page-cache.html is failing
Summary: [GTK] Layout test media/restore-from-page-cache.html is failing
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: 83704
Blocks:
  Show dependency tree / graph
 
Reported: 2016-06-19 17:08 PDT by Michael Catanzaro
Modified: 2021-07-02 03:47 PDT (History)
5 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Catanzaro 2016-06-19 17:08:02 PDT 
Layout test media/restore-from-page-cache.html has been flaky since last July. First recorded failure is r187031 "Improve behavior of media elements in page cache" which looks like a smoking gun to me. Updating expectations.

--- /home/slave/webkitgtk/gtk-linux-64-release-tests/build/layout-test-results/media/restore-from-page-cache-expected.txt
+++ /home/slave/webkitgtk/gtk-linux-64-release-tests/build/layout-test-results/media/restore-from-page-cache-actual.txt
@@ -2,6 +2,7 @@
 Make sure we don't reload a <video> element when navigating back to an uncached page.
 
 EVENT(loadstart)
+*** volumechange event fired. This should happen AFTER returning from cache.
 EVENT(canplaythrough)
 EXPECTED (loadCount == '0') OK
 *** Page going into cache
Comment 1 Diego Pino 2021-06-09 00:19:54 PDT 
The only test filed under this bug was:

  - media/restore-from-page-cache.html

GTK post-commit test bot reports the test has been failing consistently for the last 4000 revisions.

Diff:

--- /home/buildbot/worker/gtk-linux-64-release-tests/build/layout-test-results/media/restore-from-page-cache-expected.txt
+++ /home/buildbot/worker/gtk-linux-64-release-tests/build/layout-test-results/media/restore-from-page-cache-actual.txt
@@ -13,4 +13,5 @@
 EXPECTED (videoComputedStyle.height == '240px') OK
 *** ratechange event fired. This should happen AFTER returning from cache.
 *** volumechange event fired. This should happen AFTER returning from cache.
+*** volumechange event fired. This should happen AFTER returning from cache.
Comment 2 Xabier Rodríguez Calvar 2021-07-02 03:47:08 PDT 
Fixed in r279499