108363 2013-01-30 11:53:54 -0800 REGRESSION (r141265): fullscreen/full-screen-iframe-legacy.html, fullscreen/video-specified-size.html failing 2017-03-11 10:50:23 -0800 1 1 1 Unclassified WebKit WebKitGTK 528+ (Nightly build) Unspecified Unspecified NEW Gtk, LayoutTestFailure, Regression P2 Normal --- 1 zan webkit-unassigned bugs-noreply gustavo pnormand spena oldest_to_newest 820377 0 zan 2013-01-30 11:53:54 -0800 fullscreen/full-screen-iframe-legacy.html and fullscreen/video-specified-size.html tests are failing, most likely due to changes in r141265. http://trac.webkit.org/changeset/141265 The dashboard isn't showing these as failures until r141273, but I see no way how that commit could affect these tests. http://trac.webkit.org/changeset/141273 http://test-results.appspot.com/dashboards/flakiness_dashboard.html#group=%40ToT%20-%20webkit.org&tests=fullscreen%2Ffull-screen-iframe-legacy.html%2Cfullscreen%2Fvideo-specified-size.html --- /home/slave/webkitgtk/gtk-linux-64-release/build/layout-test-results/fullscreen/full-screen-iframe-legacy-expected.txt +++ /home/slave/webkitgtk/gtk-linux-64-release/build/layout-test-results/fullscreen/full-screen-iframe-legacy-actual.txt @@ -3,6 +3,6 @@ To test manually, click the video "full screen" button - the page should enter full screen mode. EVENT(webkitfullscreenchange) -TEST(document.getElementById('frame').contentDocument.width==document.width) OK +TEST(document.getElementById('frame').contentDocument.width==document.width) FAIL END OF TEST --- /home/slave/webkitgtk/gtk-linux-64-release/build/layout-test-results/fullscreen/video-specified-size-expected.txt +++ /home/slave/webkitgtk/gtk-linux-64-release/build/layout-test-results/fullscreen/video-specified-size-actual.txt @@ -1,3 +1,3 @@ -EVENT(webkitfullscreenchange) TEST(video.clientWidth==document.body.clientWidth) OK +EVENT(webkitfullscreenchange) TEST(video.clientWidth==document.body.clientWidth) FAIL END OF TEST 821123 1 pnormand 2013-01-30 23:28:23 -0800 Yes this is a regression in the sense that when NATIVE_FULLSCREEN_VIDEO is enabled at build time then a fullscreen-ing a video will not use all the FULLSCREEN_API code paths. So it's expected that these two tests fail now. We should unflag them once we have a working Video AC implementation. When that happens we can disable the NATIVE_FULLSCREEN_VIDEO feature at build time. 904487 2 spena 2013-06-28 01:29:40 -0700 Hi, fast/regions/full-screen-video-from-region.html fullscreen/full-screen-crash-offsetLeft.html fullscreen/full-screen-no-style-sharing.html fullscreen/full-screen-stacking-context.html fullscreen/video-controls-override.html fullscreen/video-specified-size.html are no longer timing out, although fullscreen/video-specified-size.html is slow. 904498 3 pnormand 2013-06-28 01:58:12 -0700 Because the commit that introduced this regression was rolled out I suppose :) 904500 4 pnormand 2013-06-28 01:58:45 -0700 Oh sorry, ignore this comment, wrong bug 904501 5 pnormand 2013-06-28 01:59:39 -0700 So, NATIVE_FULLSCREEN_VIDEO was actually disabled recently by Calvaris.