Bug 211612

Summary: REGRESSION(r261341): imported/w3c/web-platform-tests/html/semantics/embedded-content/the-video-element/intrinsic_sizes.htm is failing
Product: WebKit Reporter: Lauro Moura <lmoura>
Component: MediaAssignee: Eric Carlson <eric.carlson>
Status: RESOLVED CONFIGURATION CHANGED    
Severity: Normal CC: aakash_jain, ap, bugs-noreply, eric.carlson, webkit-bot-watchers-bugzilla, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
See Also: https://bugs.webkit.org/show_bug.cgi?id=211611
https://bugs.webkit.org/show_bug.cgi?id=211464
Bug Depends on: 211641    
Bug Blocks:    
Attachments:
Description Flags
Patch none

Description Lauro Moura 2020-05-07 21:15:45 PDT
Diff:

--- /home/slave/webkitgtk/gtk-linux-64-release-tests/build/layout-test-results/imported/w3c/web-platform-tests/html/semantics/embedded-content/the-video-element/intrinsic_sizes-expected.txt
+++ /home/slave/webkitgtk/gtk-linux-64-release-tests/build/layout-test-results/imported/w3c/web-platform-tests/html/semantics/embedded-content/the-video-element/intrinsic_sizes-actual.txt
@@ -5,6 +5,6 @@
 PASS default object size is 300x150 
 PASS default height is half the width 
 PASS default width is twice the height 
-FAIL default object size after src is removed assert_equals: expected "300px" but got "320px"
+PASS default object size after src is removed 
 FAIL default object size after poster is removed assert_equals: expected "102px" but got "300px"
Comment 2 Aakash Jain 2020-05-08 05:12:34 PDT
r261341 (Bug 211464) seems like the commit which made this test very flaky.

It did made following changed in imported/w3c/web-platform-tests/html/semantics/embedded-content/the-video-element/intrinsic_sizes-expected.txt

- FAIL default object size after src is removed assert_equals: expected "320px" but got "300px"
+ FAIL default object size after src is removed assert_equals: expected "300px" but got "320px"
Comment 3 Radar WebKit Bug Importer 2020-05-08 09:38:01 PDT
<rdar://problem/63024081>
Comment 4 Eric Carlson 2020-05-08 10:05:11 PDT
Created attachment 398868 [details]
Patch
Comment 5 EWS 2020-05-08 10:30:39 PDT
Committed r261392: <https://trac.webkit.org/changeset/261392>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 398868 [details].
Comment 6 Ryan Haddad 2020-05-08 13:25:43 PDT
Re-opened since this is blocked by bug 211641
Comment 7 Ryan Haddad 2020-05-08 13:27:52 PDT
Reverted r261341 and r261392 for reason:

Caused multiple regression test failures

Committed r261409: <https://trac.webkit.org/changeset/261409>
Comment 8 Alexey Proskuryakov 2020-05-08 15:47:55 PDT
*** Bug 211611 has been marked as a duplicate of this bug. ***