Bug 177899

Summary: Elements are laid out incorrectly in full screen mode when _minimumLayoutWidth is nonzero
Product: WebKit Reporter: mitz
Component: Layout and RenderingAssignee: Nobody <webkit-unassigned>
Status: NEW    
Severity: Normal CC: adele, bfulgham, fred.wang, simon.fraser, thorton, webkit-bug-importer, zalan
Priority: P2 Keywords: InRadar
Version: Other   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
fullscreen test
none
wip patch none

mitz
Reported 2017-10-04 14:32:06 PDT
To reproduce: 1. Change MiniBrowser to set its WKWebView’s _minimumLayoutWidth to some nonzero value, such as 400 2. Load a webpage with a video element with controls 3. Click the full screen control Results: The video enters full-screen mode, but doesn’t lay out to fit in the full-screen window.
Attachments
fullscreen test (5.92 KB, text/html)
2018-01-05 11:28 PST, Adele Peterson 		no flags
Details
wip patch (3.37 KB, patch)
2018-01-08 10:17 PST, Adele Peterson 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
mitz
Comment 1 2017-10-04 14:33:26 PDT
<rdar://problem/34820953>
Adele Peterson
Comment 2 2018-01-03 10:38:47 PST
Also occurs for other fullscreen elements, like the ones in this test page: https://davidwalsh.name/demo/fullscreen.php
Adele Peterson
Comment 3 2018-01-05 11:28:30 PST
Created attachment 330562 [details] fullscreen test
Adele Peterson
Comment 4 2018-01-08 10:17:20 PST
Created attachment 330713 [details] wip patch Adding a work in progress patch. Still trying to decide how to write a test - whether to write an API test or add something to DRT (or the WK2 equivalent).
Frédéric Wang (:fredw)
Comment 5 2018-05-23 06:59:42 PDT
Comment on attachment 330713 [details] wip patch View in context: https://bugs.webkit.org/attachment.cgi?id=330713&action=review > Adding a work in progress patch. Still trying to decide how to write a test - whether to write an API test or add something to DRT (or the WK2 equivalent). Not sure if that can really help here, but there is this bug to add fullscreen support to WebDriver: bug 180398 > Source/WebKit/UIProcess/Cocoa/WebViewImpl.h:638 > + nit: remove extra whitespace.
Note You need to log in before you can comment on or make changes to this bug.