Bug 170766

Summary: Provide a viewport parameter to disable clipping to the safe area
Product: WebKit Reporter: Tim Horton <thorton>
Component: New BugsAssignee: Tim Horton <thorton>
Status: RESOLVED FIXED    
Severity: Normal CC: bdakin, dino, eoconnor, jonlee, sam, simon.fraser
Priority: P2    
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch
none
add a second test
none
Patch
none
Patch
none
Patch
none
Patch
none
Patch
none
Patch
none
Patch bdakin: review+

Description Tim Horton 2017-04-12 01:23:24 PDT
Provide a viewport parameter to disable clipping to the document rect
Comment 1 Tim Horton 2017-04-12 01:27:08 PDT
Created attachment 306894 [details]
Patch
Comment 2 Tim Horton 2017-04-12 01:27:55 PDT
<rdar://problem/31564634>
Comment 3 Tim Horton 2017-04-12 01:28:49 PDT
We should also separately consider improving the margin tile mechanism -- we could extend the normal tiles into the insets, for example, to reduce memory overhead.
Comment 4 Simon Fraser (smfr) 2017-04-12 01:49:55 PDT
Comment on attachment 306894 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=306894&action=review

> Source/WebCore/dom/ViewportArguments.h:54
>      float shrinkToFit;
> +    float clipToDocument;

Can we stop this madness?
Comment 5 Sam Weinig 2017-04-12 05:58:58 PDT
Comment on attachment 306894 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=306894&action=review

>> Source/WebCore/dom/ViewportArguments.h:54
>> +    float clipToDocument;
> 
> Can we stop this madness?

Agreed. Why this float?
Comment 6 Tim Horton 2017-04-12 16:46:17 PDT
Created attachment 306950 [details]
Patch
Comment 7 Tim Horton 2017-04-12 17:06:39 PDT
Created attachment 306953 [details]
add a second test
Comment 8 Tim Horton 2017-04-12 17:50:14 PDT
Created attachment 306956 [details]
Patch
Comment 9 Tim Horton 2017-04-13 13:58:28 PDT
Repaints in the margins are getting clipped out in GraphicsLayerCA::setNeedsDisplayInRect...
Comment 10 Tim Horton 2017-04-13 14:05:39 PDT
(In reply to Tim Horton from comment #9)
> Repaints in the margins are getting clipped out in
> GraphicsLayerCA::setNeedsDisplayInRect...

... because the RenderView's layer isn't extended into the margins.
Comment 11 Tim Horton 2017-04-13 14:32:11 PDT
Created attachment 307025 [details]
Patch
Comment 12 Tim Horton 2017-04-14 16:12:35 PDT
Created attachment 307164 [details]
Patch
Comment 13 Tim Horton 2017-04-14 17:46:39 PDT
Created attachment 307172 [details]
Patch
Comment 14 Tim Horton 2017-04-14 17:48:55 PDT
Created attachment 307173 [details]
Patch
Comment 15 Tim Horton 2017-04-17 10:10:42 PDT
Comment on attachment 307173 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=307173&action=review

> Source/WebCore/page/ViewportConfiguration.cpp:341
> +        m_configuration.clipToSafeArea = m_viewportArguments.clipToSafeArea != 0.;

This can just be assignment now.
Comment 16 Tim Horton 2017-04-17 10:46:52 PDT
Created attachment 307280 [details]
Patch
Comment 17 Tim Horton 2017-04-17 10:52:11 PDT
Created attachment 307281 [details]
Patch
Comment 18 Tim Horton 2017-04-17 13:25:24 PDT
https://trac.webkit.org/changeset/215425/webkit