Bug 159737

Summary: Full screen ePub embedded video is playing on 1/4 screen, cut off
Product: WebKit Reporter: Jer Noble <jer.noble>
Component: New BugsAssignee: Jer Noble <jer.noble>
Status: RESOLVED FIXED    
Severity: Normal CC: dino, jonlee, thorton, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
URL: InRadar
Attachments:
Description Flags
Patch thorton: review+

Jer Noble
Reported 2016-07-13 14:27:51 PDT
Full screen ePub embedded video is playing on 1/4 screen, cut off
Attachments
Patch (3.70 KB, patch)
2016-07-13 14:34 PDT, Jer Noble 		thorton: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Jer Noble
Comment 1 2016-07-13 14:28:10 PDT
<rdar://problem/26259404>
Radar WebKit Bug Importer
Comment 2 2016-07-13 14:28:33 PDT
<rdar://problem/27332427>
Jer Noble
Comment 3 2016-07-13 14:34:47 PDT
Created attachment 283566 [details] Patch
Tim Horton
Comment 4 2016-07-13 14:38:09 PDT
Comment on attachment 283566 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=283566&action=review > Source/WebKit2/UIProcess/Cocoa/WebViewImpl.mm:1010 > + setFixedLayoutSize(CGSizeZero); Somewhat surprising that WKViewFixedSizeLayoutStrategy doesn't do this in willChangeLayoutStrategy. Maybe it should be there? I don't know.
Tim Horton
Comment 5 2016-07-13 14:38:45 PDT
Also somewhat concerned that something is using the fixed layout size when fixed layout is off... seems bad.
Jon Lee
Comment 6 2016-07-14 14:01:32 PDT
<rdar://problem/26259404>
Dean Jackson
Comment 7 2016-07-15 13:43:34 PDT
Committed r203295: <http://trac.webkit.org/changeset/203295>
Note You need to log in before you can comment on or make changes to this bug.